如何实现猜三位数的演绎逻辑游戏？求Python代码优化建议

你写的Bagels游戏逻辑框架没问题，但有几个可以优化的细节，下面是调整后的代码和具体改进建议：

优化后的完整代码 #

import random

# 生成不含重复数字的三位秘密数字（若允许重复可去掉去重逻辑）
def generate_secret_number():
    while True:
        num = random.randint(100, 999)
        digits = list(str(num))
        if len(set(digits)) == 3:  # 确保数字不重复
            return digits

secret_digits = generate_secret_number()
remaining_tries = 10

print("欢迎玩Bagels游戏！你有10次机会猜测一个三位数，规则如下：")
print("- Fermi：数字和位置都正确")
print("- Pico：数字正确但位置错误")
print("- Bagels：没有正确数字")

while remaining_tries > 0:
    guess = input(f"还有{remaining_tries}次机会，请输入三位数：")
    
    # 输入验证：必须是三位数字
    if not guess.isdigit() or len(guess) != 3:
        print("请输入有效的三位数！")
        continue
    
    guess_digits = list(guess)
    remaining_tries -= 1
    
    # 判断是否猜对
    if guess_digits == secret_digits:
        print("恭喜你！猜对了！")
        break
    
    # 统计提示信息
    hints = []
    for i in range(3):
        if guess_digits[i] == secret_digits[i]:
            hints.append("Fermi")
        elif guess_digits[i] in secret_digits:
            hints.append("Pico")
    
    # 输出提示
    if hints:
        # 按Fermi在前、Pico在后排序，逻辑更清晰
        hints.sort(reverse=True)
        print(" ".join(hints))
    else:
        print("Bagels")

# 循环结束后揭晓答案（如果没猜对）
if remaining_tries == 0:
    print(f"游戏结束，秘密数字是{''.join(secret_digits)}")

具体优化建议 #

• 简化随机数生成：

  1. 你重复导入了random和randint，只保留import random即可，用random.randint(100, 999)能直接生成三位数，无需额外写函数。
  2. 原版代码直接打印了秘密数字，完全失去游戏可玩性，建议删除print(random_number)。
  3. 可选补充：传统Bagels游戏的秘密数字通常不含重复数字，上面的代码加入了去重逻辑，若不需要可去掉while循环，直接返回list(str(random.randint(100, 999)))。

• 删除冗余的字典结构：
  你用字典存储数字和索引，但对比位置和数字只需要列表就能完成，字典完全是多余的，直接用列表操作更简洁高效。

• 修复输入验证逻辑：
  原版代码的input_num < 999 and input_num > 100会排除100和999，正确的三位数范围应该是100 <= input_num <= 999。另外，用字符串判断isdigit()和长度是否为3，比转成整数再判断更直观，还能避免用户输入非数字的错误。

• 修正提示逻辑：

  1. 原版代码中只要有Fermi就会同时打印Pico，这是错误的——比如猜测数字和秘密数字只有一个位置正确，其他数字都不对，不应该输出Pico。正确逻辑是逐个位置检查：位置和数字都对则加Fermi，数字对但位置错则加Pico。
  2. 提示按Fermi在前、Pico在后的顺序输出，能让玩家更容易理解线索。

• 避免无效输入消耗次数：
  原版的for i in range(10)循环中，输入无效也会消耗一次猜测机会，改用while循环结合剩余次数变量，无效输入时不减少次数，更符合游戏规则。

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