Short answer: Old code was not optimized. It had many redundant lines of code.
Long answer: Here's a list of things that were addressed:
- Declaring the variables at the start but then re-declaring them in each of the if statements. This is highly inefficient and can be fixed by putting the lines of code into a single function.
- Executing the print commands by each column. Lines of text DO NOT need to be printed one-by-one. Use "\n" to indicate a new line.
- The print commands are put into each and every one of the if statements. Again highly inefficient and can be fixed by putting them into a single function.
- It seems that you did not learn from my edit on S-Spense. Multiple if statements on the same level can be replaced by else if or "elif". This is a good habit and will save you from some trouble in certain applications.
- On the last part which detects for invalid inputs, the second part where it detects if both X and Y inputs are too high is not needed. In fact, a single elif statement will do for detecting invalid inputs.