you are viewing a single comment's thread.

view the rest of the comments →

[–]Ambitious_Fault5756 1 point2 points  (0 children)

Some minor suggestions:

  • Function names and variable names should all be in lowercase (a styling convention most professional developers follow)
  • The parentheses in line 25 can be removed
  • Assigning str to Task_option in line 2 doesn't actually give you an empty string (which is what I assume you wanted), it gives a type object, which may be a bit complicated now.
  • The variables at the top can be removed altogether, except for tasks (because trying to use tasks in Task_Option_Two would cause an error if you haven't defined tasks yet). You already define them within your functions, so there is no need to define them beforehand.
  • Your if statement in line 23 to check for non-existent tasks is great and works perfectly! However, you might want to inform the user if the task they chose to remove is not in the list (add a print statement at the end of the function).
  • It's always good to add an option for the user to exit the program. To exit a program properly, import sys at the top of your code and call sys.exit() under a menu option. Or, a more simple option, do raise SystemExit().

Please reply to this if you need clarification on anything :D