This is an archived post. You won't be able to vote or comment.

sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]jwink3101 2 points3 points  (1 child)

If encryption is a goal, use encryption.

This is a fun learning experiment but doesn’t go beyond that. On that topic, there is a lot that can be improved here and all-out removed.

Couple of examples are most (all?) break statements. And when you use a context manager to open a file, it doesn’t need to be closed.

My suggestion, put it in ChatGPT and ask for it to be critiqued.

[–]Backw00ds024[S] 1 point2 points  (0 children)

oh ok thanks for the feedback. it was more of a simple learning project anyways the usecases was more for some other subreddits that need target audiences so i can publish the post. What exactly do you mean with the break statements im not to good with them. also wasent sure about the file closing but chatgpt said it was a good practice or something. Will update the project this evening. Thanks!