Python中OOP方法嵌套递归调用的风险及应对方案

Python的调用栈有默认深度限制（大概1000层），你当前的写法每次遇到异常都会嵌套调用start，之前的方法上下文不会被释放，栈会不断累积。比如流程是：start() → navigation_controller() → check_for_unexpected_conditions() → start() → ... 每触发一次重启，栈就多一层，最终会触发RecursionError导致程序崩溃。

解决方法：用循环代替递归式重启 #

核心思路是把整个流程放在一个顶层无限循环里，遇到异常时不嵌套调用重启方法，而是重置状态、清理资源后回到循环起点，让流程线性执行，避免栈累积。

修改后的代码示例 #

class Session:
    def __init__(self, driver_factory):
        self.driver_factory = driver_factory  # 传入driver创建方法，方便重启时重建
        self.state = "ACTION1"
        self.driver = None

    def run(self):
        # 顶层无限循环，负责整个流程的持续运行与重启
        while True:
            try:
                # 初始化/重建会话
                self.driver = self.driver_factory()
                self._setup_initial_page()
                self._process_state_machine()
            except Exception as e:
                # 打印异常日志（可选）
                print(f"检测到异常，触发重启: {str(e)}")
                # 清理当前会话资源
                if self.driver:
                    self.driver.quit()
                # 重置状态到初始值
                self.state = "ACTION1"

    def _setup_initial_page(self):
        self.driver.get("目标网址")

    def _process_state_machine(self):
        # 状态机循环，处理各步骤逻辑
        while True:
            # 检查异常情况，遇到问题直接抛出，让上层循环处理重启
            self._check_unexpected_conditions()

            # 处理各状态逻辑
            if self.state == "ACTION1":
                elements = self.driver.find_elements(...)
                if len(elements) > 0:
                    self.state = "ACTION2"
                    self._advance_to_action2()
            elif self.state == "ACTION2":
                # 执行ACTION2的操作
                ...
                # 比如完成后切换到下一个状态或结束当前会话
                self.state = "ACTION3"
            # 其他状态处理...

    def _check_unexpected_conditions(self):
        # 检测到异常时，直接抛出异常，不调用start
        if self._is_page_error():
            raise RuntimeError("页面加载失败")
        if self._is_navigation_stuck():
            raise RuntimeError("页面跳转异常")

    def _is_page_error(self):
        # 实现页面错误检测逻辑
        return "500 Error" in self.driver.page_source

    def _is_navigation_stuck(self):
        # 实现跳转异常检测逻辑
        return self.driver.current_url == self.last_url  # 需要记录last_url

    def _advance_to_action2(self):
        # 执行ACTION1到ACTION2的跳转操作
        ...

关键改进点 #

• 顶层循环接管重启：所有重启逻辑都在run()的while循环里处理，避免嵌套调用，栈深度始终保持在较低水平。
• 异常抛出代替方法调用：_check_unexpected_conditions()遇到问题时抛出异常，而不是直接调用重启方法，让上层循环统一处理重启逻辑。
• 显式资源清理：每次重启时关闭旧的driver实例，避免浏览器进程泄漏，同时重置状态到初始值。
• 状态机分离：把状态流转逻辑放在单独的_process_state_machine()循环里，流程更清晰，也避免了嵌套调用。

这样修改后，无论触发多少次重启，调用栈都不会累积，彻底解决栈溢出的问题，同时代码结构更清晰，维护性更强。

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