Hey Nguyen and Edwin,

I've done some tinkering on the problem and managed to replicate your results. There is indeed a subtle weakness in the solution.
When numbers are entered, no problem arises. When you enter a string, the print('That is not a number! Numbers only please:> ') statement fires and afterwards a
new value is received as user_input. However, you need to enter a number in order to escape the nested while-loop. This is a bit unreasonable, because the user
would want to type 'exit' and be done with the program, right? Instead the user is asked for a number...
--------------------------------------------------
In short - if you input a string (not an 'int'), you get stuck in the nested while-loop until an integer has been typed. The only real fixing required is to treat 'exit' as a special string,
which needs to be typed only one time - and the program finishes. Currently 'exit' is treated as all the other strings and that's why we remain in the inner loop.
--------------------------------------------------
FIX: In general, I would suggest restructuring the code -- take a look at the image attached. We can use one while-loop and if-else conditions to navigate our way through various
user inputs. If we check user_input.isdigit() at one place, we can separate the inputted strings. However, since there is only one loop, at each iteration we check whether 'exit' has been typed. If you really insist on using two nested while-loops, then maybe two break-statements are required. However, this is not a very clean solution and I would recommend avoiding it.
Hope this has illuminated the issue. I believe this would aid Edwin as well!
A., The 365 Team.

Python Programmer Bootcamp- section 6

Submit an answer

related questions