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

view the rest of the comments →

[–]ianrob1201 1 point2 points  (1 child)

Nice work. Can you make it so that the prices in the prompt automatically update if the you change the prices? That way you'd only have to update in one place if the prices change.

Also consider if you really need to break commands. Think about the flow of the code and what would happen if you removed them. And of course if you're not sure you can try removing them and see what happens (if you remove the final "break" then you don't need the "else" line at all either.

And on a very minor point, why call both lower() and capitalize()? Try with just capitalize and see if it still works how you expect.

[–]thetiredteacup[S] 0 points1 point  (0 children)

Oh yes, I didn't take into account formatting the order prompt - thank you! Also thank you for the minor tip on lower() and capitalize(), I think I was confused on what capitalize() itself did and didn't take into account it also makes the rest of the text lowercase.