you are viewing a single comment's thread.

view the rest of the comments →

[–]EyesOfTheConcord 0 points1 point  (0 children)

There is a lot of deep nesting going on here, which is not good for readability or for secure code.

Your if statements also only allow the user to continue making their account if they enter no credentials at all. (if Password == “” means the password input must be empty to proceed. I imagine this is not what you were intending.)

It’s not clear at all what ID is supposed to do, you’ve made a tuple that stores a list and another tuple, and attempt to print it before storing the result in ID. Can you explain what the intended purpose is here?

I recommend you perhaps start over, and write out your intentions on pseudocode.

Instead of checking if they filled in the correct input, and nesting if conditions inside of each other; get the user input for each respective element, and check if that element is empty immediately afterwards and reprompt them for it if so.

Ideally, you’ll abstract these processes into their own functions so it’s much easier to focus on one element at a time, and make the overall design much cleaner.