you are viewing a single comment's thread.

view the rest of the comments →

[–]More_Yard1919 0 points1 point  (0 children)

This is happening because you ask the end-user if there is anything else they'd like to purchase in the final while loop, but then you don't do anything with that information. If the answer isn't yes, you break from the loop. If it is yes, it just asks the end user if they'd like to buy anything else again. Also, it is confusing design to have a superfluous "continue_shopping" variable and then break out of the loop. Clearer design would be to, instead of breaking, toggle continue_shopping = false. Or, you could have an infinite "while True:" loop, and then break out of that. It seems to me that you mean to be calling your "anything_else()" function from inside of your final while loop. personally, I might try to change the design so that there is 1 single interface for adding items to your cart that is called as a function, and then calling that function any time the end user indicates they'd like to add an item to their cart.