Selenium显式等待未生效抛出NoSuchElementException异常问题咨询

你的显式等待没有生效、直接抛出NoSuchElementException的核心原因是传参时提前执行了元素查找动作：
你在调用waitForElementToLoad方法时，第二个参数写的是driver.findElement(By.cssSelector("portal-application[title='AW Acc']"))，Java执行方法调用时会优先计算所有传入参数的值，再执行方法内部逻辑——也就是说你还没进入等待方法的重试判断逻辑，就已经先执行了findElement去搜元素，这时候登录提交后页面还没跳转到目标页面、元素还没渲染出来，自然直接抛出找不到元素的异常，后面的等待逻辑根本没有运行的机会。
Thread.sleep()能正常运行是因为它会先强制阻塞线程固定时长，等页面完全加载出目标元素后再往下执行findElement，自然不会报错，但这种固定等待不管元素加载快慢都要等满设置的时长，会大幅拉长测试执行时间。

调整自定义等待方法的逻辑，把元素查找的动作放到显式等待的重试循环内部执行，不要提前查找元素：

• 修改自定义等待方法，将入参从提前查找好的WebElement改为By类型的定位符，等待过程中会循环执行元素查找+可见性判断，找不到元素时自动重试，直到元素可见或者达到超时时间。
  改后的等待方法代码如下（适配Selenium 4+版本，高版本WebDriverWait超时参数需要传入Duration类型）：

  import java.time.Duration;
  import org.openqa.selenium.By;
  import org.openqa.selenium.WebDriver;
  import org.openqa.selenium.support.ui.ExpectedConditions;
  import org.openqa.selenium.support.ui.WebDriverWait;
  
  public static void waitForElementToLoad(WebDriver driver, By elementLocator, int seconds) {
      WebDriverWait wait = new WebDriverWait(driver, Duration.ofSeconds(seconds));
      wait.until(ExpectedConditions.visibilityOfElementLocated(elementLocator));
  }
  

• 调整业务代码中的调用方式，去掉提前执行的findElement，直接传入定位符即可：

  // 原错误调用写法
  // waitForElementToLoad(driver, driver.findElement(By.cssSelector("portal-application[title='AW Acc']")), 100);
  // 改后正确调用
  waitForElementToLoad(driver, By.cssSelector("portal-application[title='AW Acc']"), 10);
  

• 一般业务场景下显式等待超时设置10-30秒足够，不需要设置100秒这么长，否则页面真的出现加载异常时会等待过久，浪费执行资源。
• 脚本里不要同时混用隐式等待（implicitlyWait）和显式等待，否则会出现等待时长叠加的不可预期问题，统一用显式等待处理所有元素加载等待场景即可。

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