29
votes
Accepted
Python - Tkinter - periodic table of chemical elements
I hope you enjoy this bit of code.
I did!
Your symbols, keywords and values should have capitalised variable names since they're global constants. However, life would be easier if your symbols were ...
- 71.1k
27
votes
Python - Tkinter - periodic table of chemical elements
Don't optimize the wrong thing
You have an excellent review already, so I'll comment on a narrow topic: cramped
code layout. For example:
...
- 13.1k
24
votes
Accepted
2048 with GUI in C
Well done. This is not a complete review, but instead a (short) list of possible improvements I found when I skimmed your code.
Documentation
First of all: thank you! It's great to have ...
- 19.7k
10
votes
A GUI Youtube Player (2)
Thanks for bravely offering up your code.
I'm sure it will emerge in better shape.
Each development project has its own standards.
If they aren't written down in the ReadMe,
then we tend to assume ...
- 42.3k
9
votes
Accepted
C++ "Dear ImGui" file browser
One of the things I'm specifically wondering, is it ok/proper to have static helper functions in the .cpp file if they are not member functions? (should they be member functions and should they even ...
- 11.7k
7
votes
First Java Program: A Basic GUI Library Management System with JavaFX
Separations of Concerns
You've mixed UI with the business logic. The same components are responsible for interacting with the user and communicating with the database.
By doing so, you're violating ...
- 3,369
6
votes
Accepted
AppJar number pad and keyboard
Python has an official style guide, PEP8. It won't hurt getting familiar with it, since it helps keeping your code readable, consistent and ready for others to mess with.
Combining Italian and ...
Mast♦
- 13.8k
6
votes
GUI Number Generator in QT C++
I see a number of things that may help you improve your program.
Use proper case for file names
We all know that Windows is not case sensitive with respect to file names, but pretty much every other ...
- 67.2k
6
votes
Dynamic image gallery with filters pulling from a JSON file
It's been a while since I looked too much at native JS (I use TS most of the time), so forgive me if any of my suggestions feel dated.
CSS
I have no real feedback on the CSS. I personally prefer using ...
- 8,049
5
votes
Accepted
Qt Number Generator v2
Seems like you've made a lot of improvements since your previous post! Let's get into the review!
1. General Overview
a. Use the ...
- 437
5
votes
Accepted
GUI to organize images, chapter titles, animation steps, and instructions
If there's a list, make it a list
Do not represent RxCx variables as separate variables. Represent them as nested lists and it will greatly simplify your code. ...
- 71.1k
5
votes
Accepted
College Billing System - Java
Absolute positioning
This:
l1.setBounds(550, 100, 250, 20);
and lines like it are non-ideal. Absolute positioning will fail to scale the controls and their ...
- 71.1k
5
votes
Accepted
Python PyQt6 populate QTreeWidget with a list of named tuples
You have a pile of unstructured, global code; this needs to be organized into functions and maybe classes
Close your cursor once you're done with it, ideally via context management
Order your query ...
- 71.1k
5
votes
ConnectFourFX.java - A Java FX GUI app for playing Connect Four against AI
The functions has*Strike() are basically the same function, but with different starting points and different strides through the ...
- 88.3k
5
votes
First Java Program: A Basic GUI Library Management System with JavaFX
Alexander Ivanchenko treated the most points. That you were aware of the need of password hashing is clear, so okay.
In general I find the code advanced, assuming some non-java background, and ...
- 4,656
5
votes
First Java Program: A Basic GUI Library Management System with JavaFX
All of the Buttons go unused:
...
- 10.1k
4
votes
Autocomplete for tkinter Entry widgets
Very, very, very nice code! It's very difficult to find something for an improvement. So only some not very important things:
In your list comprehension
...
- 1,956
4
votes
Accepted
Buttons labelled with the number of times they have been clicked
Here are a few things I'd recommend changing:
There's no need to store the sum of a and b. You can always compute it in the view ...
- 291
4
votes
Accepted
Lightshot Print Screen key linux handler
Instruct shell how to treat unset variables
set -o nounset
In this script, there should not be possible to encounter an unbound variable, so setting this will ...
- 2,116
4
votes
Accepted
Basic Countdown Timer
Very clean code indeed, there's not much to be said here.
PEP8
I'm sure you've been lectured on this before, but there's an official style guide for Python called PEP8. It states that module-level ...
- 4,632
4
votes
Accepted
Enhancing a Conversion calculator
One of the smartest programmer's I've ever known once told me that code is read way more often than it is written, so optimize for the reader1
While it's true that code should be efficient, it also ...
- 2,169
4
votes
Accepted
Platform-agnostic windowing library
I wouldn't use #define macros. They expose the internal naming convention of your OS dependent functions to the world. Once you send out your library to others [as ...
- 838
4
votes
2048 with GUI in C
Since the other reviews have already hit most points, I'll just mention a few not already covered.
Avoid relative paths in #includes
Generally it's better to ...
- 67.2k
4
votes
Accepted
C++ Qt simple GUI game
When dealing with Qt 5+ prefer the new connect syntax:
QObject::connect(ui->tiles01, &QPushButton::clicked, this, &MainWindow::tileCliked);
Instead of ...
- 13k
4
votes
Accepted
GUI Number Generator in QT C++
your first statement is that you want advice in naming conventions and in your second line you state
I know that you should NOT use _ prefix, but I really like it and m_ looks ugly to me.
Some ...
- 1,675
4
votes
Accepted
C++ Snake game using Qt-framework
Here are some things that may help you improve your program.
Use consistent formatting
It appears that your code uses tabs in some places and spaces in others, making the code appear badly formatted ...
- 67.2k
4
votes
Accepted
labelpix, image annotation tool for object detection GUI python
setup_toolbar from settings.py
Maintaining a several lists in parallel is tedious. ...
- 7,363
Only top scored, non community-wiki answers of a minimum length are eligible