Beginning guessing game in Python

Good job for a first experience in Python! You can improve your code a little by looking at these recommendations:

• To choose a random integer, you could simply use random.randrange(0,11)
• You should probably not import at the function level either, try writing your import statements at the top of your file.
• The game() function is called recursively; you shouldn't do that. You could have an infinite loop that fires game() back when exited.

• Personally, I wouldn't use lists with "yes" and "no" values, try something like

  if "y" in choice.lower():
      return True 
  else:
      return False
  

• Last but not least, you should check if the number entered is an integer. If it is not, ask the user to enter a valid integer until they do so.

Some general things that I noted about your code.

• As noted by WayToDoor in his preliminary draft, you import inside your function. There are some cases where this is useful behavior as it means you'll only import a module when your function needs it, however it is generally encouraged that you put your imports at the top of the file.
• The Python random module has a function called randint which will generate a random number for you in the specified range.
• You have a lot of nested if statements. Your code might look neater if you rethought what you're doing so that the logic is flat.
• Take a look at this PEP8 style guide checker. PEP8 is the recommended style for Python code to ensure some measure of readability and consistency is present in your code.
• Scottbb describes a way to refactor this code so that more useful information can be gained from the play_game method and the code is grouped more logically.
  • Check out his answer!

Just because I think large bullet-pointed lists are a bit blah, here's some code to think on.

import random


def play_game(name, lower=1, upper=10):
    """
    Play a number guessing game!
    :param name: The name of the player
    :param lower: The lower limit of numbers to be guessed greater than or equal to 0
    :param upper: The upper limit of numbers to be guessed greater than the lower limit.
    :return: A boolean that determines whether the player wants to play again.
    """
    # Generate a random number in the given range, inclusive.
    secret = random.randint(lower, upper)
    number_of_tries = 0

    print("---------------------------------------------------------------------\n"
          "---- Welcome, {}!\n"
          "---- I am thinking of a number between {} and {}.\n"
          "---- Let's see how many times it will take you to guess the number.\n"
          "---------------------------------------------------------------------\n"
          .format(name, lower, upper))

    # Main loop
    guessing_numbers = True
    while guessing_numbers:
        guess = input("What is guess #{}?\n".format(number_of_tries))

        # Make sure that the user enters a valid number.
        while not guess.isdigit():
            print("Sorry, that isn't a valid input.")
            print("Please only enter integers.")
            guess = input("What is guess #{}?\n".format(number_of_tries))

        # Now that we can assume it's a valid number,
        # we can turn the string into an integer.
        guess = int(guess)
        # Increment the number of tries by 1
        number_of_tries += 1

        if guess < secret:  # If the guess is too low
            print("Too Low. Try again!")
        elif guess > secret:  # If the guess is too high
            print("Oops! Too high, better luck next try!")
        else:  # They've guessed the correct number!
            guessing_numbers = False  # Break out of the loop

    if number_of_tries == 1:
        guess_form = "guess"
    else:
        guess_form = "guesses"

    print("-------------------------------------------------\n"
          "---- {}! You got it in {} {}!\n"
          "-------------------------------------------------\n"
          .format(name, number_of_tries, guess_form))

    if number_of_tries < 3:
        print("Bravo!")
    elif number_of_tries < 5:
        print("Hmmmmpf... Better try again")
    elif number_of_tries < 7:
        print("Better find something else to do!")
    else:  # They took more than 6 tries.
        print("You should be embarrassed! My dog could do better.")

    choice = input('Would you like to play again? ("yes" or "no")\n')
    if 'y' in choice.lower():
        return True  # They want to play again.
    else:
        print("Okay.. Goodbye :(")
        return False  # They don't want to play again.


def main():
    # Get the user's name so we can address them.
    name = input("What is your name?\n")
    playing = True
    while playing:
        playing = play_game(name)

# This means you can import the code without running it or
# if you want to you can run the code like an executable.
if __name__ == "__main__":
    main()

I had basically written everything Zenohm wrote (but way too wordier). I would like to suggest the slight change to Zenohm's code.

Remove the end of play_game(), from choice = input(...) on, and move that logic to main():

def play_game():
    ...
    # deleted everything from 'choice = input(...)' on
    return number_of_tries

def main():
    name = input("What is your name? \n")  # Asks the user for their name
    results = []
    while (True):
        results.append(play_game(name))
        choice = input('Would you like to play again? ("yes" or "no")\n')
        if 'y' not in choice.lower():
            break

    # do something with the list of results from gameplays,
    # such as printing the average number of tries
    average = float(sum(results))/len(results)
    print("On average, it took you {} tries to guess.".format(average)

# This means you can import the code without running it or
# if you want to you can run the code like an executable.
if __name__ == "__main__":
    main()

This cleanly separates the business logic of the game from the mechanics of running the game multiple times. Also, it frees up play_game() to return the number the number of guesses the user took, so that main() (or other calling code, in the future) can collect statistics about gameplay.

Besides the great commentary given by WayToDoor (which Zenohm wrote up), I'd just like to remark on your commenting style.

The comments are quite superfluous, stating exactly what the code is doing, and as such do not add any value.