查找网页元素失败时重试FindElement方法最多3次的实现问题

现有代码核心问题 #

• 计数变量maxTries是方法内部局部变量，每次递归调用时都会重新初始化为0，永远达不到重试上限，触发无限递归死循环
• 条件判断语法错误：C#中不存在!>运算符，你期望的「小于3次继续重试」的逻辑应该使用maxTries < 3实现
• 递归调用时没有传递当前重试次数，计数逻辑完全失效

修复代码 #

推荐使用迭代写法实现，逻辑更直观，避免递归潜在的栈溢出风险：

public static void FindElement(IWebDriver webDriver, string selector)
{
    // 总共尝试3次
    const int maxAttempts = 3;
    for (int i = 0; i < maxAttempts; i++)
    {
        try
        {
            webDriver.FindElement(By.XPath(selector));
            Console.WriteLine("Element found.");
            return;
        }
        catch (NoSuchElementException e)
        {
            // 最后一次尝试失败直接输出异常信息
            if (i == maxAttempts - 1)
            {
                Console.WriteLine(e.ToString());
                return;
            }
            Console.WriteLine("Element not found. Retrying.");
            // 可选优化：添加短等待，适配异步渲染场景，避免请求过于频繁
            Thread.Sleep(500);
        }
    }
}

如果需要保留递归实现，可修改为带计数参数的递归逻辑：

// 对外暴露的公共调用方法
public static void FindElement(IWebDriver webDriver, string selector)
{
    FindElementWithRetry(webDriver, selector, 0);
}

// 内部递归重试方法
private static void FindElementWithRetry(IWebDriver webDriver, string selector, int currentTryCount)
{
    const int maxAttempts = 3;
    try
    {
        webDriver.FindElement(By.XPath(selector));
        Console.WriteLine("Element found.");
    }
    catch (NoSuchElementException e)
    {
        currentTryCount++;
        if (currentTryCount < maxAttempts)
        {
            Console.WriteLine("Element not found. Retrying.");
            Thread.Sleep(500);
            FindElementWithRetry(webDriver, selector, currentTryCount);
        }
        else
        {
            Console.WriteLine(e.ToString());
        }
    }
}

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