Python终端Hangman游戏无效输入时游戏界面不显示问题及代码优化求助

看起来你遇到的核心问题是只有有效输入时才会刷新游戏界面，而无效输入（重复字母、数字、多字符）的分支里没有调用显示单词状态和已猜字母的逻辑，导致玩家看不到当前游戏进度。同时代码里大量重复的打印语句确实会让维护变麻烦，咱们一步步来解决这些问题：

一、核心问题解决：提取重复的游戏状态显示逻辑 #

你之前尝试抽函数但卡变量的问题很好解决——把需要的变量作为参数传给函数就行，不用依赖外部作用域。咱们可以把所有显示游戏状态的代码（类别、难度、hangman图、当前单词、已猜字母）都放到一个单独的函数里，这样不管输入有效还是无效，都能调用它刷新界面。

比如定义这个函数：

def display_game_state(random_word, guessed_letters, wrong_guesses, category, difficulty):
    # 打印类别和难度
    print(f"Category: {category.capitalize()}")
    print(f"Difficulty level: {difficulty.capitalize()} \n")
    # 打印hangman图形
    print_hangman(wrong_guesses)
    # 显示当前单词的猜测状态
    for letter in random_word:
        if letter in guessed_letters:
            print(f" {letter.upper()} ", end="")
        else:
            print(" _ ", end="")
    print("\n")
    # 显示已猜字母
    print("Previously guessed letters:")
    print(f"{list(guessed_letters.upper())}\n ")

然后在你的主循环里，不管输入是否有效，都调用这个函数。这样每次玩家输入后，都会刷新完整的游戏界面，包括无效输入的情况。

二、重构输入验证逻辑，让代码更清晰 #

原来的if-elif-else结构有点混乱，咱们可以先把所有无效输入的情况先判断完，再处理有效输入，这样逻辑更顺：

修改后的主循环大致是这样：

# 定义最大错误次数常量，方便后续调整
MAX_WRONG_GUESSES = 7

while wrong_guesses < MAX_WRONG_GUESSES:
    player_choice = input("Please pick a letter...: \n").strip().upper()
    clear()  # 先清屏，再处理输入
    
    # 先处理所有无效输入的情况
    valid_input = True
    remaining_guesses = MAX_WRONG_GUESSES - wrong_guesses
    if not player_choice.isalpha():
        print(f"{player_choice} is not a valid letter... You have {remaining_guesses} guess(es) remaining...")
        valid_input = False
    elif len(player_choice) != 1:
        print(f"Please input one letter at a time... You have {remaining_guesses} guess(es) remaining...")
        valid_input = False
    elif player_choice in guessed_letters:
        print(f"{player_choice.upper()} has already been guessed... You have {remaining_guesses} guess(es) remaining...")
        valid_input = False
    
    # 处理有效输入的情况
    if valid_input:
        guessed_letters += player_choice
        if player_choice in random_word:
            print(f"Correct, {player_choice.upper()} is in the word! You have {remaining_guesses} guess(es) remaining...")
        else:
            wrong_guesses += 1
            remaining_guesses = MAX_WRONG_GUESSES - wrong_guesses
            print(f"Sorry, {player_choice.upper()} is not in the word... You have {remaining_guesses} guess(es) remaining...")
    
    # 不管输入有效与否，都显示当前游戏状态
    display_game_state(random_word, guessed_letters, wrong_guesses, category, difficulty)
    
    # 检查是否获胜
    if all(letter in guessed_letters for letter in random_word):
        print(colorama.Fore.GREEN + "Congratulations, you won! \n")
        print(f"The word is {random_word.upper()}!\n")
        sub_menu()
        break

# 循环结束说明猜错次数用完，判定失败
print(colorama.Fore.RED + "\nSorry, you lose... Please try again...\n")
print(f"The word was {random_word.upper()}...\n")
sub_menu()

三、其他代码优化小建议 #

• 输入处理加strip()：防止玩家不小心输入空格，避免因为空格导致的“多字符”或“非字母”错误。
• 简化获胜判断：用all(letter in guessed_letters for letter in random_word)替代原来的wrong_letters计数，更直观也不用额外维护变量。
• 提取常量：把最大错误次数7定义成MAX_WRONG_GUESSES常量，以后调整难度时改一处就行。
• 避免重复计算：把剩余猜测次数remaining_guesses提前计算，所有提示里直接用这个变量，减少重复代码。
• 精简冗余注释：只保留解释复杂逻辑或关键步骤的注释，避免对简单代码的无意义说明。

这样修改后，不仅解决了无效输入时不显示游戏界面的问题，还大大减少了重复代码，让整个函数更易读和维护。

备注：内容来源于stack exchange，提问作者James Fitz