hangman doesn't end after guessing the word

[vinci]

Hi,

I should say from the beginning that I'm very new in Python. I've been following a video tutorial where hangman is being tested. I've got the following code which seems to work for the guy who teaches, but not to me. And I simply don't understand the logic of it to be honest and can't understand how it can actually work.
This is the code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60

import random     # make a list of words words = [     'apple',     'banana',     'orange',     'coconut',     'strawberry',     'lime',     'grapefruit',     'lemon',     'kumquat',     'blueberry',     'melon' ] while True:     start = input("Press enter/return to start, or enter Q to quit")     if start.lower() == 'q':         break     # pick a random word     # choice functions picks a random item out of an iterable     secret_word = random.choice(words)     print("secret_word is {}".format(secret_word))     bad_guesses = []     good_guesses = []       while len(bad_guesses) < 7 and len(good_guesses) != len(list(secret_word)):     # draw spaces     # draw guessed letters and strikes         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('')           # take a guess         guess = input("Guess a letter: ").lower()           if len(guess) != 1:             print("You can only guess a single 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 letters!")             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 didn't guess it! My secret word was {}".format(secret_word))

What I don't understand is why how this line "if len(good_guesses) == len(list(secret_word)" can actually work. The only situation where I'd assume it can work is when you've only guessed words whose letters don't repeat. So for a word like 'blueberry' it won't work.
Is this a mistake that the guy has made?
How could I improve the code, so that the game ends also when I have to guess words whose letters repeat.

Thanks!

[Cryptus]

Which part of the code doesnt work? It seems to run normal for me.

[vinci]

The code works. The problem is that it doesn't work as it's supposed to work when the word contains letters which repeat themselves, as I've already stated.
I've just found the solution, though. Instead of using list the conditional, I should use set, and then a word like 'blueberry' works fine:

1	if len(good_guesses) == len(set(secret_word)):

P.S. to be more explicit, if a word like 'blueberry' is chosen, then the program will never exit, you keep guessing and you're going to be kept in a loop.

[ichabod801]

You're right, the game does not end if there is a double letter in the word. And you can't guess the same letter twice. It shouldn't be testing against len(list(secret_word)), but rather against len(set(secret_word)). Using set(secret_word) will give you all the unique letters in the string.