Selenium中弹窗多次出现问题：循环点击直至其消失的实现

你现有代码的核心问题是：仅用if判断一次弹窗是否存在，再通过foreach遍历元素集合，这没法实现循环直到弹窗完全消失的逻辑；而且硬编码Thread.Sleep稳定性极差，容易因页面加载延迟导致操作失败。

下面是修改后的代码，用循环持续检查弹窗状态，搭配更可靠的等待逻辑：

// 用while循环持续检测弹窗，直到它不再显示
while (basePage.IsElementPresent(zipModal))
{
    // 显式等待弹窗容器可交互（替代Thread.Sleep，适配页面加载节奏）
    var wait = new WebDriverWait(Driver, TimeSpan.FromSeconds(10));
    wait.Until(driver => 
        driver.FindElement(zipModal).Displayed && driver.FindElement(zipModal).Enabled);

    if (!basePage.IsElementPresentXPath(cityModal))
    {
        // 等待目标点击元素就绪后再操作
        wait.Until(driver => driver.FindElement(By.CssSelector("[class*='row zip-row highlighted']")).Enabled);
        basePage.Click("[class*='row zip-row highlighted']");
    }
    else
    {
        wait.Until(driver => driver.FindElement(By.CssSelector("[class*='row zip-caption']")).Enabled);
        basePage.Click("[class*='row zip-caption']");
        basePage.ClickCity(cityModal);
    }

    // 短暂等待弹窗关闭，避免循环检查过于频繁
    Thread.Sleep(500);
}

关键调整说明： #

• 替换if为while循环：只要弹窗还存在，就重复执行关闭逻辑
• 用WebDriverWait显式等待：确保元素可见且可交互后再操作，比固定时长的Thread.Sleep更稳定
• 增加操作后短等待：避免循环检测频率过高，减少不必要的资源消耗

额外提醒： #

• 确认basePage.IsElementPresent方法是判断元素可见且存在，而非仅存在于DOM中（如果方法只检查DOM存在，建议修改为判断Displayed属性）
• 如果zipModal是元素集合，要调整IsElementPresent的判断逻辑，确保是检查弹窗容器的显示状态

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