Return to Revisions

Pretty clean looking code, but let's look at a few ways it could be better or more pythonic:

• Use elif statements:

    if personInput == randnumb:
        # ...
    elif personInput > randnumb:  # You already know they're != at this point
        # ...
    elif ...
  

• Use snake case for your variable names. E.g., use person_input instead of personInput (which is camel case), and rand_num instead of randnumb. In that second case, it's also more common to shorten "number" to "num" instead of "numb".

• Use format strings instead of print's argument concatenation, e.g.:

    print("Guess lower ({} guesses left)".format(max_guesses - guesses))
  

• Check for and handle errors:

    try:
        person_input = int(input("What is your guess? "))
    except ValueError:
        print("Not sure what you meant by that, please input an integer")
        continue
  

• I prefer to be more safe than sorry when handling my own incrementor, such as guesses, just in case I do something funky that lets it skip over max_guesses:

    # Since you "break" on correct answer, we don't need to check that again
    if guesses >= max_guesses:
        # ...