Python拆分代码为函数结构后运行失效的故障排查咨询

问题原因 #

你的重构代码有3个核心错误，导致无法正常运行：

• 变量作用域错误：replace_letters_with_numbers函数内定义的color是局部变量，仅在当前函数内部生效，is_valid_color函数无法访问这个变量，运行时会直接抛出NameError: name 'color' is not defined异常。
• 函数没有返回值：字符替换函数执行完替换逻辑后，没有将生成的color结果返回，调用函数后拿不到处理后的字符串，处理结果直接被内存回收。
• 参数传递缺失、逻辑耦合：你没有将替换得到的色值作为参数传入校验函数，反而在校验函数内硬引用不存在的color变量；同时把长度校验、字符合法性校验、结果打印的逻辑全部塞在校验函数里，拆分后的函数职责不清晰。

修正方案 #

按照函数单一职责原则调整逻辑，处理函数返回结果，校验函数接收参数做判断，修正后代码如下：

with open("german.txt") as f:
    words = f.read().split()

def replace_letters_with_numbers(word):
    # 完成字符替换后返回处理结果
    return word.lower().replace("o", "0").replace("i", "1").replace("s", "5").replace("t", "7")

def is_valid_color(color_str):
    # 单独做色值合法性校验，返回布尔值结果
    if len(color_str) not in (3, 6):
        return False
    for c in color_str:
        if c not in "abcdef0123456789":
            return False
    return True

# 主逻辑按流程调用函数
for word in words:
    processed_color = replace_letters_with_numbers(word)
    if is_valid_color(processed_color):
        print(word, "#" + processed_color)

注：由于字符替换操作不会改变字符串长度，原代码中用原始单词计算长度的写法不会出结果错误，但直接校验处理后的色值字符串长度逻辑更严谨。

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