Just a simple guessing game

There are few improvements you can do:

1. Remove the first sec_no = random.randint(0, 10). You are already initializing it inside of the main while loop.
2. Create a variable that contains minp.lower(). Instead of calculating it over and over - keep it in a variable and avoid code duplication.
3. Get rid of ncout or att. They are practically saving the same thing, keep only att and change the while and if condition to att > 0.
4. Add some comments - File comments and function comments.
5. General python tip: do not write code in main, Use functions. Function for each case, function for main loop, function to check if string is int and the __main__ should look like this:

if __name__ == '__main__': main_function() 

Except for that - good logic, nice code :)

You could implement a hint to the answer so you can broaden the range of numbers. This is just a quick example.

import random num = random.randrange(1, 100) lives = 5 print(''' Welcome to the guessing game. Guess a number between 1-100.\nYou have {} lives '''.format(lives)) while True: try: if lives == 0: print('You died!\nThe number you were looking for is {}'.format(num)) break guess = int(input('Guess: \n')) if guess == num: print('You win!') break if guess < num: lives -= 1 print('Number is higher...\nlives: {}\n'.format(lives)) if guess > num: lives -= 1 print('Number is lower...\nlives: {}\n'.format(lives)) except ValueError: print('Numbers only!') 

The result:

Welcome to the guessing game. Guess a number between 1-100.You have 5 lives Guess: 50 Number is lower... lives: 4 Guess: 30 Number is lower... lives: 3 Guess: 25 Number is lower... lives: 2 Guess: 10 Number is higher... lives: 1 Guess: 16 Number is lower... lives: 0 You died! The number you were looking for is 13