自定义模块函数重复调用返回首次结果的问题排查

嘿，这个问题我太熟了！十有八九是你在模块里踩了Python可变默认参数或者模块级全局变量缓存的坑，导致第一次调用的结果被“记住”了，后续调用不管传啥参数都直接返回旧值。我给你拆解下常见的原因和解决办法：

1. 最容易踩的坑：可变默认参数 #

比如你的函数可能写成了这样（典型错误写法）：

# 错误示例：可变默认参数导致结果缓存
def check_target_file(filename, size, results={}):
    # 检查是否已经缓存过结果
    if (filename, size) in results:
        return results[(filename, size)]
    # 实际的文件检查逻辑
    exists = os.path.exists(filename) and os.path.getsize(filename) == size
    # 把结果存到默认参数里
    results[(filename, size)] = exists
    return exists

Python的默认参数只会在函数定义的时候初始化一次，不是每次调用都新建！所以第一次调用后，results这个字典就被修改并保留了，后续调用不管传什么参数，都会用这个已经有值的字典，自然返回第一次的结果。

修复方法：把默认参数改成None，在函数内部初始化 #

# 正确写法：避免可变默认参数
def check_target_file(filename, size, results=None):
    # 每次调用都新建一个空字典（如果没传results的话）
    if results is None:
        results = {}
    if (filename, size) in results:
        return results[(filename, size)]
    exists = os.path.exists(filename) and os.path.getsize(filename) == size
    results[(filename, size)] = exists
    return exists

2. 另一个常见原因：模块级全局变量缓存结果 #

如果你的模块里有全局变量用来存结果，比如：

# 模块里的全局变量：会一直存在于模块生命周期中
cached_result = None

def check_target_file(filename, size):
    global cached_result
    # 错误：直接返回之前的缓存，不管参数变没变
    if cached_result is not None:
        return cached_result
    # 文件检查逻辑
    cached_result = os.path.exists(filename) and os.path.getsize(filename) == size
    return cached_result

这种情况下，第一次调用后cached_result就被赋值了，后续不管传什么参数，函数都会直接返回这个旧值，完全忽略新的参数。

修复方法： #

• 如果不需要跨调用缓存，直接删掉全局变量，用局部变量存结果就行；
• 如果确实需要缓存不同参数的结果，改成用字典缓存（像上面正确写法那样），并且确保每次调用都能正确匹配参数。

3. 额外排查点：文件路径问题 #

有时候也可能是路径的锅：比如第一次调用用了相对路径，第二次调用时当前工作目录变了，导致函数找不到新的目标文件，误返回第一次的结果。可以在函数里把路径转成绝对路径来避免：

import os

def check_target_file(filename, size):
    # 转成绝对路径，确保每次都找对文件
    abs_filename = os.path.abspath(filename)
    exists = os.path.exists(abs_filename) and os.path.getsize(abs_filename) == size
    return exists

总结一下：先查函数的默认参数是不是用了字典、列表这种可变对象，再查有没有模块级的全局变量乱缓存结果，这两个是最常见的元凶！

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