Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

Python Python Basics (2015) Letter Game App Letter Game Introduction

Oleksiy Kloc
Oleksiy Kloc
733 Points

After word is guessed, the program doesn't break

import random

words = [ 'apple', 'bannana', 'oragne', 'squash', 'melon', 'blueberry', 'raspberry', 'strawberry' ]

while True: start = input("Press enter/return to start, or enter Q to quit") if start.lower() == 'q': break

secret_word = random.choice(words)
bad_guesses = []
good_guesses = []

while len(bad_guesses) < 7 and len(good_guesses) != len(list(secret_word)):
    for letter in secret_word:
        if letter in good_guesses:
            print(letter, end='')
        else:
            print('_', end='')
    print('')
    print('Strikes: {}/7'.format(len(bad_guesses)))
    print('')

    guess = input("Guess a letter: ".lower())

    if len(guess) != 1:
        print("You can only guess a signle letter!")
        continue
    elif guess in bad_guesses or guess in good_guesses:
        print("You've already guessed that letter!")
        continue
    elif not guess.isalpha():
        print("You can only guess letter!")
        continue
    if guess in secret_word:
        good_guesses.append(guess)
        if len(good_guesses) == len(list(secret_word)):
            print("You win!The word was {}".format(secret_word))
            break
    else:
        bad_guesses.append(guess)
else:
    print("You didnt guess it!@ My secret word was {}".format(secret_word))

I've cross-referenced it with the code of others and can't figure out why their's breaks after the word is guessed but mine just allows inputs after the word is guessed.

1 Answer

Steven Parker
Steven Parker
231,236 Points

Your win detection test is to compare the number of good guesses with the length of the word. This will work, but only on words that have no repeated letters. When a word has one or more repeated letters, it takes fewer guesses than the total number of letters to win.

To be able to work with words like "apple", you'll need to modifiy how you check for a win. For examples, you might compare the number of good guesses with the number of unique letters, or you could check if every letter of the word has been guessed.