主函数调用的函数错误检测：全局变量方案是否合理？有无更佳方案？

首先得说，你用全局变量的方案在这个小型程序里确实能跑通需求，但正如你意识到的，全局变量确实是不太好的编程习惯——它会让函数之间产生隐式耦合，要是以后要加更多子函数、或者改成多线程运行，很容易出现难追踪的bug，代码的可测试性和维护性也会打折扣。所以虽然能工作，但长远来看并不推荐，下面给你几个更优的替代方案：

方案1：子函数返回错误状态，主函数汇总判断 #

这是最直观的改进：让每个子函数返回一个布尔值，表示自身是否触发了错误，主函数调用时收集所有结果，用any()判断是否有至少一次错误。这种方式完全消除了全局变量，函数之间的依赖关系明明白白，可读性和可维护性都强很多。

示例代码：

#!/usr/bin/env python3
import os
import random
from pathlib import Path

def f1():
    for i in range(1, 5):
        if random.randint(0, 9) == 0:
            return True  # 返回True表示触发错误
    return False

def f2():
    for i in range(1, 5):
        if random.randint(0, 9) == 0:
            return True
    return False

def main():
    # 收集所有子函数的错误状态
    errors = [f1(), f2()]
    # 只要有一个子函数出错，就返回True
    return any(errors)

if __name__ == '__main__':
    has_error = main()
    error_file = Path('/var/tmp/error_triggered') / os.path.splitext(os.path.basename(__file__))[0]
    if has_error:
        error_file.touch()
    else:
        error_file.unlink(missing_ok=True)

方案2：使用类封装状态 #

如果后续需要扩展更多和错误状态相关的逻辑，或者函数需要共享更多上下文，用类来封装会更优雅。把错误状态作为类的实例变量，子函数作为类的方法，既避免了全局变量，也能更好地组织代码结构。

示例代码：

#!/usr/bin/env python3
import os
import random
from pathlib import Path

class ErrorMonitor:
    def __init__(self):
        self.has_error = False

    def f1(self):
        for i in range(1, 5):
            if random.randint(0, 9) == 0:
                self.has_error = True

    def f2(self):
        for i in range(1, 5):
            if random.randint(0, 9) == 0:
                self.has_error = True

    def run(self):
        self.f1()
        self.f2()
        return self.has_error

if __name__ == '__main__':
    monitor = ErrorMonitor()
    has_error = monitor.run()
    error_file = Path('/var/tmp/error_triggered') / os.path.splitext(os.path.basename(__file__))[0]
    if has_error:
        error_file.touch()
    else:
        error_file.unlink(missing_ok=True)

方案3：使用可变对象传递状态（轻量过渡方案） #

如果你觉得类的写法有点重，可以用一个可变对象（比如字典、列表）来传递错误状态——因为可变对象在函数内修改会影响外部，不需要声明全局变量。这种方式比全局变量更可控，状态的传递是显式的，不会有隐式耦合的问题。

示例代码：

#!/usr/bin/env python3
import os
import random
from pathlib import Path

def f1(error_state):
    for i in range(1, 5):
        if random.randint(0, 9) == 0:
            error_state['has_error'] = True

def f2(error_state):
    for i in range(1, 5):
        if random.randint(0, 9) == 0:
            error_state['has_error'] = True

def main():
    error_state = {'has_error': False}
    f1(error_state)
    f2(error_state)
    return error_state['has_error']

if __name__ == '__main__':
    has_error = main()
    error_file = Path('/var/tmp/error_triggered') / os.path.splitext(os.path.basename(__file__))[0]
    if has_error:
        error_file.touch()
    else:
        error_file.unlink(missing_ok=True)

总结 #

如果是当前这种简单的小型程序，方案1（返回值汇总）是最推荐的，它简洁清晰，没有额外复杂度；如果后续有更多扩展需求，方案2（类封装）会更合适；方案3适合临时过渡，比全局变量更优雅但不如前两者规范。

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