如何封装try-except循环函数简化重复代码？调试指定示例

嘿，我来帮你搞定这个重试函数的问题！你原来的代码逻辑有几个明显的问题，导致它根本没法实现你想要的“失败就重试”的效果，咱们一步步来修复：

原代码的核心问题 #

1. 你直接把 driver.find_element_by_xpath(...).click() 作为参数传给函数——这个操作会立即执行，而不是等到函数内部去执行，所以如果查找元素失败抛出异常，根本不会被函数里的try...except捕获到。
2. 函数里的try块只是直接break，完全没有执行你想要重试的操作，等于这个循环和异常捕获根本没起到作用。

修复后的代码 #

首先，我们需要让函数接受一个可调用的操作（比如lambda函数或者自定义函数），这样才能在循环里重复执行它。同时要记得导入time模块哦：

import time

def retry_on_failure(operation, delay=0.25):
    while True:
        try:
            # 执行传入的操作，成功就返回结果并退出循环
            result = operation()
            return result
        except Exception:
            # 捕获到异常就等待指定时间，然后继续重试
            time.sleep(delay)

然后调用的时候，把你要重试的操作包装成lambda，这样它才会在函数内部被执行：

retry_on_failure(lambda: driver.find_element_by_xpath('//*[@id="j_list_card"]/ul/li/h3/a').click())

为什么这样就能工作？ #

• lambda表达式把你的元素查找和点击操作包装成了一个可调用对象，只有当函数里调用operation()的时候，才会实际执行这个操作。
• 如果执行过程中抛出异常（比如元素还没加载出来的NoSuchElementException），会被except块捕获，然后等待0.25秒后再次循环执行操作，直到成功为止。

可选优化：添加最大重试次数 #

无限循环重试可能不是个好主意，咱们可以给函数加个最大重试次数的限制，避免死循环：

import time

def retry_on_failure(operation, delay=0.25, max_retries=10):
    retries = 0
    while retries < max_retries:
        try:
            result = operation()
            return result
        except Exception as e:
            retries += 1
            time.sleep(delay)
            print(f"操作失败，第{retries}次重试: {str(e)}")
    # 超过次数后抛出异常，方便你排查问题
    raise RuntimeError(f"已重试{max_retries}次，操作仍然失败")

这样如果重试10次还不成功，函数就会抛出异常，你可以根据具体情况调整max_retries的值。

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