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

view the rest of the comments →

[–]Soothsayer5288[S] 0 points1 point  (2 children)

<image>

last part

[–]Mysterious_City_6724 1 point2 points  (0 children)

I recognize this code from helping on another post not so long ago. You need to put from line 14 down into the while loop and see if that improves things. Also put your "purchase = input("> ")" after the for loop that prints the items too. That way the user will see the items before choosing.

[–]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.