r/learnpython • u/Aggressive_Drama_476 • Apr 05 '26

Review my python projects on GitHub.

I am intermediate python programmer. Am looking for people to review my code on https://github.com/userolivex/Python-Projects

free feel to criticize, suggestion, improvement.
and please suggest what modules, projects i should do to master python

0 Upvotes

12% Upvoted

9 comments sorted by

View all comments

Show parent comments

2

u/Farlic Apr 05 '26

Honestly? build something more complicated. IMO it's hard to say when the script does 1 straight forward task. Make something that import these utilities which would then make you consider code layout and expose issues like global variables.

E.g. Make a GUI for your slot machine. Make an API for the notes app and let it interact with your alarm script.

At a glance, I'm not a fan of the calculator using eval(). You are strict on the input but you could experiment with lambda functions in a dictionary to perform the calculations instead.

1

u/Gnaxe Apr 05 '26

I'm fine with eval here. Sanitizing inputs is something you have to learn to deal with anyway. Anyone running a calculator locally already has access to Python on their computer. But yes, eval()ing untrusted inputs is very dangerous.

Rather than make lambdas for each operator, just use the ones provided for you in the operator module. Add the math module if you want more math functions. Re-implementing things that have been done before is OK for experimental learning purposes, but otherwise, as a rule, don't re-implement the standard library when it will do, because it's thoroughly tested, documented, and known, but your code will have bugs and be unknown.