用于单击 Web 元素的 For 循环

Question

I'm trying to loop the for loop 7 times where the webelement will be click 7 times only. Even if there are other commentsbutton element in the web page, the loop will ignore them and stop clicking.

Currently, the script will still click on the webelement non-stop.

 WebElement commentsbutton = (WebElement) comment.findElement(By.xpath("//button[contains(@class,'-cx-PRIVATE-PostInfo__commentsLoadMoreButton -cx-PRIVATE-PostInfo__commentsLoadMoreButtonEnabled')]")); //view all/ load more comments

//click more comments
for(int i = 0; i < 7; i++) {
    //for (WebElement comments_element : commentsbutton) {
    commentsbutton.click(); //click on button if found
    Thread.sleep(3000); //pause for 5 seconds   
    System.out.println(commentsbutton);
    //}
}

Answer 1

Try this:

WebDriverWait wait = new WebDriverWait(driver, 10);

for(int i=0; i<7; i++) {
    WebElement element = wait.until(
       ExpectedConditions.visibilityOfElementLocated(
           By.xpath("//button[@class='-cx-PRIVATE-PostInfo__commentsLoadMoreButtonEnabled']")));

     driver.findElement(By.xpath("//button[@class='-cx-PRIVATE-PostInfo__commentsLoadMoreButtonEnabled']")).click();
}

Explanation: As per the comments below, you mentioned that you actually wanted to click on the load more comments button on a Instagram post for exactly 7 times. So, if you observe the HTML of the page, especially that of the button, is:

<button class="-cx-PRIVATE-PostInfo__commentsLoadMoreButton -cx-PRIVATE-PostInfo__commentsLoadMoreButtonEnabled">

But once you click the button, the site start fetching more comments to display. In that period, the class name value -cx-PRIVATE-PostInfo__commentsLoadMoreButtonEnabled is removed the HTML becomes this:

<button class="-cx-PRIVATE-PostInfo__commentsLoadMoreButton" disabled="true">

Once it fetches the data and displays it, the class name value which was removed is added back and the button gets enabled.

Hence, if you want to click on the button multiple times to load all or most of the comments, you are gonna have to use a combination of waits and loops. You were using Thread.sleep() to give a pause and then click on the button. But then using a wait is a better option, since it polls the DOM to find the element for that period of time and once it meets it expected condition (in this case visibility of the element), it then moves on to execute the next line of code.