• Create Account

## Any suggestions for better code? Python "Guess the number"

Old topic!

Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.

4 replies to this topic

### #1DavitosanX  Members

328
Like
0Likes
Like

Posted 15 January 2013 - 02:23 PM

Hello everyone! I haven't been able to spend time programming for some months now, so I decided to brush up my very few skills by writing simple programs. I opted for "Guess the number" since it's pretty basic stuff (or so i thought). My main trouble was that I wanted to let the player quit by typing "quit", instead of asking for a number, for example "0".

If any other newbie has tried out this program, the problem is when the user tries to input anything other than a number. Python (not without reason) doesn't type-cast it into anything with int(), so the program crashes. The following is my solution to this problem, but it doesn't look very elegant at all.

I would appreciate any suggestions on making the code better. Thanks!



from random import *

print("\n* * * Welcome! * * *\n\nGo ahead and try to guess a number from 1 to 10. If you feel like quitting, type QUIT\n")

seed()
playing = True
validating = True
number = randint(1,10)

while playing:

while validating:
try:
int(guess)
break
except:
if str.upper(guess) == "QUIT":
print ("Goodbye!")
validating = False
playing = False
else:
print("\nI don't think that was a number\n")

if playing == False:
break

if int(guess) == 0:
print ("Goodbye!")
playing = False
elif int(guess) < number:
print("\nA little higher...\n")
elif int(guess) > number:
print("\nA little lower...\n")
else:
print("\nYou got it!\n")
playing = False



Starting out in game programming? Me too! Check out my blog, written by a newbie, for the newbies. http://myowngamejourney.blogspot.mx/

Also, if you don't mind a 5 seconds ad, here's a great free ebook for beginners on the subject of pygame: http://bit.ly/19Bem2q

### #2ApochPiQ  Moderators

21388
Like
1Likes
Like

Posted 15 January 2013 - 02:45 PM

My Python is rusty, but you should be able to do a string check to see if the input is "quit" first, and if not, do the int() cast and proceed as usual. That'll eliminate the exception handling piece.

Of course, the other side of that is that having a program that can robustly deal with unexpected inputs is a very Good Thing™, so it's really up to you if you value shorter code or more robust code at this point :-)
Wielder of the Sacred Wands

### #3ColinDuquesnoy  Members

1162
Like
1Likes
Like

Posted 15 January 2013 - 04:04 PM

As ApochPiQ said, you can use string check methods to check your inputs, but the pythonic way is to use try/except blocks as you did.

Here is how you could do with if/else statements.
validating = True
while validating is True:
# checks for quit
if guess.isalpha() and guess.upper() == "QUIT":
print ("Goodbye!")
validating = False
playing = False
# checks for valid number
elif guess.isdigit():
validating = False
guess = int(guess)
else:
print("I don't think that was a number")



### #4SiCrane  Moderators

11524
Like
1Likes
Like

Posted 15 January 2013 - 08:12 PM

One thing I would recommend avoiding is repeating the same computation over and over, which is what you are doing when you use int(guesss) repeatedly. Calculate it once and store the result somewhere.

### #5DavitosanX  Members

328
Like
0Likes
Like

Posted 15 January 2013 - 08:40 PM

Thanks a lot for your replies. I didn't know there were isalpha() and isdigit(). I tried looking up the subject searching "validate type in python", maybe it was a bad choice of words.

One thing I would recommend avoiding is repeating the same computation over and over, which is what you are doing when you use int(guesss) repeatedly. Calculate it once and store the result somewhere.

I'll try to remember this. Good advice.

All in all, I guess I wasn't too far off anyway, which is good news. Thanks again!

Starting out in game programming? Me too! Check out my blog, written by a newbie, for the newbies. http://myowngamejourney.blogspot.mx/

Also, if you don't mind a 5 seconds ad, here's a great free ebook for beginners on the subject of pygame: http://bit.ly/19Bem2q

Old topic!

Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.