Selenium多复选框点击问题：仅点击首个元素，无法选中Campus复选框

核心错误原因 #

你的代码里if (text.contains("Campus"));这一行末尾多了一个分号，导致if语句变成空执行体。后面的s.click(); break;会无条件执行，所以循环第一次就会点击第一个元素并跳出循环，完全没做文本匹配判断。

修正后的代码 #

WebDriver driver = new ChromeDriver();
driver.get("https://www.snapdeal.com/products/mens-footwear-sports-shoes");
driver.manage().window().maximize();
driver.findElement(By.xpath("//button[@data-filtername=\"Brand\"]")).click();
Thread.sleep(2000);
List<WebElement> list = driver.findElements(By.xpath("//div[@class=\"brand-filter-columns\"]//div//label/a"));
System.out.println(list.size());
for (WebElement s : list) {
    String text = s.getText();
    System.out.println(text);

    if (text.contains("Campus")) {
        s.click();
        break;
    }
}

额外优化建议 #

1. 替换固定等待为显式等待：Thread.sleep()是固定等待，页面加载慢时会失效、加载快时会浪费时间，建议用显式等待保证元素加载完成后再操作：

WebDriverWait wait = new WebDriverWait(driver, Duration.ofSeconds(10));
// 等待品牌按钮可点击并点击
wait.until(ExpectedConditions.elementToBeClickable(By.xpath("//button[@data-filtername=\"Brand\"]"))).click();
// 等待品牌筛选列表加载完成
wait.until(ExpectedConditions.visibilityOfElementLocated(By.xpath("//div[@class=\"brand-filter-columns\"]")));

2. 直接定位目标复选框：不用遍历所有元素，直接通过文本定位到Campus对应的复选框，更高效且避免遍历冗余：

// 直接定位Campus对应的复选框（实际可点击的通常是input标签，而非a标签）
WebElement campusCheckbox = wait.until(ExpectedConditions.elementToBeClickable(
    By.xpath("//label[contains(text(), 'Campus')]/preceding-sibling::input[@type='checkbox']")
));
campusCheckbox.click();

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