Python代码ValueError处理与if/elif兼容问题及逻辑正确性排查

一、代码现存问题拆解 #

1. 错误处理与业务逻辑完全脱节 #

原代码里的错误处理模块是单独放在后面的，前面的if/elif判断用的是未经过错误校验的初始输入，后面的错误处理只是重新获取了变量，但根本没把这些变量传入判断逻辑执行。相当于错误处理和判断是两段独立运行的代码，自然无法联动生效。

2. 输入环节低级错误 #

fail_credit的输入提示写错了，写成了please enter your credits at pass:，导致该变量实际获取的是pass学分的输入值，完全不符合需求。

3. if/elif逻辑漏洞 #

• 完全没用到defer_credit变量，正常学分判定肯定要结合pass、defer、fail三个维度
• 未校验三个学分的总和是否为120（这类场景通常要求总学分固定为120）
• 逻辑覆盖不全，仅基于pass或fail的单一值判断，忽略了多维度组合的情况

二、修正后的代码（整合错误处理+完善逻辑） #

# 封装带错误处理的学分输入函数
def get_credit(prompt):
    while True:
        try:
            credit = int(input(prompt))
            # 可选：校验学分是否为0-120之间的整十数（符合常见学分规则）
            if credit < 0 or credit > 120 or credit % 10 != 0:
                print("学分必须是0到120之间的整十数，请重新输入")
                continue
            return credit
        except ValueError:
            print("输入不是有效整数，请重新输入")

# 获取三个学分
pass_credit = get_credit("请输入pass学分: ")
defer_credit = get_credit("请输入defer学分: ")
fail_credit = get_credit("请输入fail学分: ")

# 校验总学分是否合规
if pass_credit + defer_credit + fail_credit != 120:
    print("三个学分总和必须为120，请重新运行程序输入")
else:
    # 执行符合常见规则的判定逻辑
    if pass_credit == 120:
        print("progress")
    elif pass_credit == 100:
        print("do not progress-module retriever")
    elif fail_credit > 80:
        print("exclude")
    else:
        print("do not progress-module retriever")

三、逻辑验证说明 #

结合常见的学分判定表格规则，修正后的逻辑：

• Progress: Pass=120（无需考虑defer和fail）
• Module Retriever: Pass=100，或Pass<100且Fail≤80
• Exclude: Fail>80（无需考虑pass和defer）
  同时新增总学分校验，确保输入的学分符合基本规则。

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