罗马数字计算器代码出现UnboundLocalError问题求助

错误原因 #

UnboundLocalError: local variable 'valid_num2' referenced before assignment 说白了就是你代码里的valid_num2变量，在某些代码分支里根本没被赋值就被引用了。比如你的验证逻辑可能只在输入的罗马数字合法时才给valid_num2赋值，一旦输入不合法，这个变量就没定义，后续用到它的时候自然报错。

修复方案 #

1. 提前初始化变量 #

在开始验证逻辑前，先给valid_num1和valid_num2设置默认值（比如False），确保不管后续验证结果如何，变量都有明确的定义：

# 先初始化验证标记
valid_num1 = False
valid_num2 = False

# 执行num1的验证逻辑
if 检查num1合法的条件:
    valid_num1 = True
    # 其他合法后的处理
else:
    print("第一个罗马数字不合法")

# 执行num2的验证逻辑
if 检查num2合法的条件:
    valid_num2 = True
    # 其他合法后的处理
else:
    print("第二个罗马数字不合法")

# 后续使用valid_num1和valid_num2就不会报错
if valid_num1 and valid_num2:
    # 执行转换与计算逻辑

2. 合并验证与转换逻辑（更省心） #

既然你提到移除验证代码后，转换函数能处理所有输入，不如直接用转换函数完成验证——让转换函数遇到无效罗马数字时返回None，通过判断返回值来验证合法性，避免单独写验证逻辑时遗漏分支：

def roman_to_int(s):
    roman_map = {'I':1, 'V':5, 'X':10, 'L':50, 'C':100, 'D':500, 'M':1000}
    total = 0
    prev_val = 0
    # 倒序遍历罗马数字字符
    for char in reversed(s):
        # 字符不在映射表中即为无效
        if char not in roman_map:
            return None
        current_val = roman_map[char]
        if current_val < prev_val:
            total -= current_val
        else:
            total += current_val
        prev_val = current_val
    return total

# 实际使用逻辑
num1 = input("输入第一个罗马数字: ")
num2 = input("输入第二个罗马数字: ")

int1 = roman_to_int(num1)
int2 = roman_to_int(num2)

if int1 is None:
    print(f"{num1} 不是合法的罗马数字")
elif int2 is None:
    print(f"{num2} 不是合法的罗马数字")
else:
    print(f"{num1} 转换为整数：{int1}")
    print(f"{num2} 转换为整数：{int2}")
    # 这里添加你的计算逻辑

关键提醒 #

局部变量必须在所有可能被引用的代码路径中都有定义，不能只在部分条件分支里赋值。合并验证与转换逻辑不仅能减少出错概率，还能让代码更简洁。

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