Java中的For循环中断不会使循环停止[关闭]

Question

我有以下代码

    File src = new File("C:\\Users\\Excel Files\\XLFile.xlsx");

    FileInputStream fis = new FileInputStream(src);
    XSSFWorkbook wb = new XSSFWorkbook(fis);
    XSSFSheet sh1 = wb.getSheetAt(0);

    // Start
    String chromePath = "C:\\Users\\chromedriver_win32\\chromedriver.exe";
    System.setProperty("webdriver.chrome.driver", chromePath);

    WebDriver driver = new ChromeDriver();
    driver.manage().window().maximize();

    // UAT
    driver.get("http://www.htmlcodetutorial.com/forms/_SELECT_MULTIPLE.html");
    //Building Action Interaction
    Actions act = new Actions(driver);

    if (sh1.getRow(0) !=null && sh1.getRow(0).getCell(1) !=null) { 
    List<WebElement> allopts = driver.findElements(By.xpath("//select[@name='toppings']//option"));

    List<String> xlall= new ArrayList<String>();

    Iterator<Row> ixlrows = sh1.rowIterator();

    while (ixlrows.hasNext())
    {
        Row row= ixlrows.next();

        if (row.getRowNum() > 0 && row.getCell(1) != null)
        {
            xlall.add(row.getCell(1).getStringCellValue());
        }
    }

    for (WebElement we : allopts)
    {
        if (xlall.contains(we.getText()))
        {
            act.keyDown(Keys.CONTROL)
                    .click(we)
                    .keyUp(Keys.CONTROL)
                    .build()
                    .perform();
        }
    }
 }   
    wb.close();
    fis.close();

我正在做的是匹配来自 Excel 的值并将它们与应用程序选项匹配，它将一一选择它们。

这只是一个示例代码，实际代码有 1003 个选项，这个循环必须经过。一旦它从 excel 中找到值，我想退出循环，将它们与应用程序选项匹配并单击它们。

如您所见，我尝试将 break 放在 if 语句中，并在 if 语句之后立即尝试。但是没有任何工作它仍然继续循环遍历所有选项。

我在哪里添加中断来做到这一点？

附：除了在动作完成后循环被打破之外，其他一切都在工作。

P.S 2. 这是使用 Java 的 Selenium WebDriver 代码的一部分。

更新 1：代码更新

更新 2：是否有人认为此代码中的中断与 for 循环本身无关，它将应用程序中的所有选项存储在数组 List allopts = .... 中？所以即使有休息它也会继续？在这种情况下，我应该怎么做才能停止数组以在看到选项并选择它时立即捕获所有选项？

更新 3： 还有另一个 if 语句。代码已更新为父 if 语句。

编辑：最后在@DonLeopardo 的帮助下，它就像魔术一样工作，

这是工作代码：

    if (sh1.getRow(0) !=null && sh1.getRow(0).getCell(1) !=null) { 
    List<WebElement> allopts = driver.findElements(By.xpath("//select[@name='toppings']//option"));

    List<String> xlall= new ArrayList<String>();

    Iterator<Row> ixlrows = sh1.rowIterator();

    while (ixlrows.hasNext())
    {
        Row row= ixlrows.next();

        if (row.getRowNum() > 0 && row.getCell(1) != null)
        {
            xlall.add(row.getCell(1).getStringCellValue());
        }
    }

List<String> copy = new ArrayList<>(xlall) 

WebElement we;
for(int i=0;i<allopts.size();i++)
{
    we=(WebElement)allopts.get(i);
    if (copy.contains(we.getText()))
    {
        act.keyDown(Keys.CONTROL)
                .click(we)
                .keyUp(Keys.CONTROL)
                .build()
                .perform();
        copy.remove(copy.indexOf(we.getText()));
    }
    if(copy.size()<1)
    {
         break;
    }
}

Answer 1

您可以尝试这些选项，但我不知道它们是否会工作，因为初始代码应该可以工作。 选项 1：

boolean flag=true;
int i=0;
while(i<allopts && flag)
{
if(conditons)//Put your conditions
{
//code
flag=false;
}
i++;
}

选项 2：

boolean flag=false;
for (WebElement we:allopts)
{
if (condition)//Put your conditions
{
//code
flag=true;
}
if(flag)
{
break;
}
}

编辑： 选项 3：（这应该可行）

WebElement we;
for(int i=0;i<allopts.size();i++)
{
we=(WebElement)allopts.get(i);
if (condition)//Put your conditions using we like always: xlall.contains(we.getText())
    {
    //code
    break;
    }
}

我读到你不能停止 foreach 或类似的事情。

编辑 2： 我认为现在我正确理解了这个问题。 你应该这样做：

List<String> allStrOpts=new ArrayList<String>();
for (WebElement we : allopts)
{
     allStrOpts.add(we.getText());
}
for(int i=0;i<xlall.size();i++)
{
     if(allStrOpts.contains(xlall.get(i))
     {
           act.keyDown(Keys.CONTROL)
                .click(allopts.get(allStrOpts.indexOf(xlall.get(i))))
                .keyUp(Keys.CONTROL)
                .build()
                .perform();
     }
}

如果这需要很多时间，你可以试试这个：

//List<String> copy=xlall.clone();

//or

List<String> copy = new ArrayList<>(xlall) 

WebElement we;
for(int i=0;i<allopts.size();i++)
{
    we=(WebElement)allopts.get(i);
    if (copy.contains(we.getText()))
    {
        act.keyDown(Keys.CONTROL)
                .click(we)
                .keyUp(Keys.CONTROL)
                .build()
                .perform();
        copy.remove(copy.indexOf(we.getText()));
    }
    if(copy.size()<1)
    {
         break;
    }
}

我不知道哪个性能更好，我认为是第二个，但我不确定。我用来写它们的顺序是它们出现在我脑海中的顺序。

Answer 2

Selenium 有 Select 类，您可以使用它来选择选项

WebElement selectElement = driver.findElement(By.name("toppings")); // find the <select> tag
Select select = new Select(selectElement); // initialize Select instance

for (WebElement we : allopts) {
    if (xlall.contains(we.getText())) {
        select.selectByVisibleText(we.getText());
    }
}

Answer 3

在您提供的代码中，您可以将 break 放在 if 语句中。如果if 的测试评估为true，并且if 块中的任何方法都没有抛出异常，break 将跳出最近的封闭循环（或switch )。当然，异常也会导致执行离开示例代码中的循环。

由于您似乎表明您已经尝试将break 放在那里，我倾向于认为您提供的代码没有充分捕捉到问题。这可能是因为有另一个级别的循环没有被转义，因为您尝试插入的 break 在 switch 语句中，或者因为 break 通过抛出稍后在循环。