Basic python coding

Question

Trying to do a simple guessing game program in python, but I'm more comfortable in java. When the correct number is entered, it says it is too high and won't exit the while loop. Any suggestions?

import random
comp_num = random.randint(1,101)
print comp_num
players_guess = raw_input("Guess a number between 1 and 100: ")
while players_guess != comp_num:
 if players_guess > comp_num:
 print "Your guess is too high!"
 elif players_guess < comp_num:
 print "Your guess is too low!"
 players_guess = raw_input("Guess another number between 1 and 100: ")
print "CONGRATULATIONS! YOU GUESSED CORRECTLY!"

• Mr. Polywhirl: Use elif for the second if...
• Max Regitnig: still only outputs too high...
• Tony Suffolk 66: it is not a good idea to keep updating your question to reflect the answers (even if the answers are correct). It makes the answers look irrelevant, and removes any chance of other learning from your errors.
• maksimov: Some IDEs like PyCharm come with a handy debugger. Saves time asking questions online.

Answer 1

I would guess it is because you are comparing string and int. Whatever is captured from raw_input is captured as a string and, in Python:

print "1" > 100 # Will print true

For it to work, convert:

players_guess = raw_input("Guess a number between 1 and 100: ")

to

players_guess = int(raw_input("Guess a number between 1 and 100: "))

• Max Regitnig: thanks, guys! but it still won't exit the while loop once I enter the right number..
• maksimov: Or perhaps you forgot to cast your second raw_input to an int... like your updated question indicates. You should not update your question with answers, since you're only confusing things.
• ugo: @MaxRegitnig like maksimov says, you probably forgot to cast your second raw_input at the bottom of the loop.
• Padraic Cunningham: or just make 100 a string
• Jonathon Reinhart: Welcome to the wonderful world of dynamic typing. You'll learn to love it as much as hate it.

Answer 2

You are comparing a string to an int. That's why you get odd results.

Try this:

players_guess = int(raw_input("Guess a number between 1 and 100: "))

Answer 3

import random
comp_num = random.randint(1,101)
print comp_num
players_guess = int(raw_input("Guess a number between 1 and 100: "))
while players_guess != comp_num:
 if players_guess > comp_num:
 print "Your guess is too high!"
 elif players_guess < comp_num:
 print "Your guess is too low!"
 players_guess = int(raw_input("Guess another number between 1 and 100: "))
print "CONGRATULATIONS! YOU GUESSED CORRECTLY!"

you need to force the input to int

• Tony Suffolk 66: you missed the int from the 2nd raw_input call.
• Hackaholic: yea it was a typing mistake, down voting is not a soln
• Tony Suffolk 66: for what it is worth - I didn't downvote, I just stated a reason why someone might have done so - it does seem a little harsh to me.

Answer 4

Try this code:

import random
comp_num = random.randint(1,101)
print comp_num
players_guess = int(raw_input("Guess a number between 1 and 100: "))
while players_guess != comp_num:
 if players_guess > comp_num:
 print "Your guess is too high!"
 elif players_guess < comp_num:
 print "Your guess is too low!"
 players_guess = int(raw_input("Guess another number between 1 and 100: "))
print "CONGRATULATIONS! YOU GUESSED CORRECTLY!"

• dakab: You could improve your answer by adding a short explanation.

Answer 5

in your code change in print formation and use 'int' datatype in numerical argument. i will change the cond so try it once:

 import random
comp_num = random.randint(1,101)
print ('comp_num')
players_guess = int(raw_input("Guess a number between 1 and 100: "))
while players_guess != comp_num:
 if players_guess > comp_num:
 print('Your guess is too high!')
 elif players_guess < comp_num:
 print('Your guess is too low!')
 players_guess = int(raw_input("Guess another number between 1 and 100: "))
print('CONGRATULATIONS! YOU GUESSED CORRECTLY!')