Python Hangman猜词游戏问题：如何判断所有字母已猜对？

你的代码存在几个关键问题，导致获胜判断逻辑完全失效：

1. 执行顺序错误：当前代码中，玩家猜对字母后会立刻执行continue，直接跳过后续的获胜判断分支，永远触发不了胜利提示。
2. 重复猜测破坏对比逻辑：empty_list会存储所有猜测的字母（包括重复猜对的），排序后长度会比目标单词的字母列表更长，无法匹配成功。
3. 代码笔误：8hm = HangMan()里多了个数字8，属于无效变量名，会直接导致程序报错。

修复后的代码 #

import random

class HangMan():
    def __init__(self):
        self.words_list = 'sunny', 'number', 'darius', 'calendar', 'table'
        self.words_randomizer = random.choice(self.words_list)

    def guess_letter(self):
        # 用集合存储目标单词的唯一字母，方便后续对比
        target_letters = set(self.words_randomizer)
        guessed_correct_letters = set()
        
        while True:
            guess_letter = input('Please enter letter: ').strip().lower()
            # 过滤无效输入
            if len(guess_letter) != 1 or not guess_letter.isalpha():
                print('Please enter a single valid letter!')
                continue
            
            # 提示重复猜测
            if guess_letter in guessed_correct_letters:
                print(f'You already guessed "{guess_letter}", try another one!')
                continue
            
            if guess_letter in target_letters:
                guessed_correct_letters.add(guess_letter)
                print(f'You are correct, there is letter - {guess_letter}!')
                # 猜对后立即检查是否已猜全所有字母
                if guessed_correct_letters == target_letters:
                    print('Congrats ! You won')
                    break
            else:
                print(f'Sorry, "{guess_letter}" is not in the word. Try again!')

# 修正变量名笔误
hm = HangMan()
print(f'Target word (for testing): {hm.words_randomizer}')
hm.guess_letter()

关键修改说明 #

• 用集合替代列表：集合自动去重，直接对比两个集合就能判断是否猜全所有字母，不需要排序，逻辑更简洁可靠。
• 调整判断顺序：在玩家猜对字母后立即检查胜利条件，满足就终止循环。
• 增加输入校验：过滤空输入、多字符输入和非字母输入，提升游戏体验。
• 处理重复猜测：提示玩家已猜过的字母，避免无效操作。

内容的提问来源于stack exchange，提问作者DaBa