Python石头剪刀布游戏tie辅助函数失效及代码优化求助

平局函数无法触发的高频根因 #

• 出招映射规则不统一：input_detection 模块的用户输入转码规则，和tie函数的判定规则不对齐，比如用户输入"A"被转成对应数值1代表石头，但tie函数直接拿原始输入字符"A"和电脑生成的整型数值1做相等判断，永远无法匹配
• 数据类型不匹配：电脑通过random.randint(1,3)生成的是整型数值，用户侧的出招值被处理成字符串类型（比如把"A"转成了"1"字符串），两边类型不同的情况下==判断永远为假
• 参数传递错误：调用tie函数时传入的不是经过统一格式处理的出招值，比如传了带前后空格的原始输入、未做大小写归一的小写"a"值，和电脑侧的标准化整型值不匹配
• 分支逻辑顺序错误：main函数里把胜负判定的分支放在最前面执行，直接返回了结果，根本没有走到tie函数的调用逻辑
• 随机数重复生成：电脑出招值在单次对局流程里被多次调用random.randint生成，导致判定平局时的电脑出招和生成时的参考值不一致

规范实现参考代码 #

gameFunctions2.py 辅助模块 #

# 全局唯一映射规则，避免多文件规则漂移
MOVE_MAP = {"A":1, "B":2, "C":3}  # A石头 B布 C剪刀
MOVE_NAME = {1:"石头", 2:"布", 3:"剪刀"}

def input_detection(raw_input):
    # 统一做格式归一：去空格、转大写
    processed_input = raw_input.strip().upper()
    if processed_input not in MOVE_MAP:
        return None  # 非法输入返回空
    return MOVE_MAP[processed_input]  # 统一转成和电脑出招一致的整型值

def tie_check(user_move, computer_move):
    # 仅做同类型值比对，不做额外格式转换
    return user_move == computer_move

main.py 主函数 #

import random
from gameFunctions2 import input_detection, tie_check, MOVE_NAME

def main():
    user_raw = input("请输入A(石头)/B(布)/C(剪刀)出招：")
    user_move = input_detection(user_raw)
    if not user_move:
        print("输入非法，请输入A/B/C对应出招")
        return
    # 单局内仅生成一次电脑出招
    computer_move = random.randint(1,3)
    # 优先判断平局，减少分支跳转
    if tie_check(user_move, computer_move):
        print(f"本局平局！双方均出{MOVE_NAME[computer_move]}")
        return
    # 后续胜负判定逻辑

代码优化建议 #

• 所有映射规则统一维护：把出招对应关系、出招名称这类公共配置全部放在gameFunctions2模块中，不在main主函数里单独定义映射规则，从根源避免多文件规则不一致的问题
• 输入校验逻辑内聚：所有输入格式清洗、合法性判断全部放在input_detection函数内完成，不在各个判定分支里零散做格式处理，减少重复代码
• 函数单一职责：tie_check函数仅负责两个同格式出招值的相等判断，不内置输入转换、随机数生成逻辑，降低模块耦合度
• 减少无意义的随机数调用：单局对局的电脑出招值只生成一次，存在变量里传递给后续判定函数，避免多次调用随机数导致的判定结果错乱
• 分支优先级优化：把触发概率较高的平局判断放在最靠前的逻辑位置，减少不必要的分支判断跳转

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