为字符串变量添加内部标签报错问题排查与修复

Let's break down why you're hitting these errors and how to fix them.

The Root Cause #

In Java, labels (like your start:) can only be attached to statements—think loops, conditional blocks, or single executable lines. They can't be placed directly before a variable declaration. When you write start: String zipCodes1 = "...";, the compiler gets confused: it doesn't recognize the variable declaration as a valid statement for the label, so it fails to resolve zipCodes1 later in your code. This breaks the variable's scope and declaration flow entirely.

Fixes You Can Implement #

Here are two solid ways to fix this issue:

1. Move Variable Declarations Outside the Label

Simply pull your variable definitions out from under the label so they're properly declared before the label is used. This keeps the label attached to executable code where it belongs:

// Declare all variables first, outside the label's scope
String zipCodes1 = "90019,90036,90016,..."; // Your full zip code string
String[] zipArray1 = zipCodes1.trim().split(",");
Random zipIndex1 = new Random();

start:
// Now the label applies to the executable logic block
int randomIndex1 = zipIndex1.nextInt(395);
System.out.println("Zip bad! New zip = "+zipArray1[randomIndex1]+"");
driver.findElement(By.xpath("//android.widget.EditText[@index='0']")).sendKeys(zipArray1[randomIndex1]);
Thread.sleep(750);
driver.pressKey(new KeyEvent(AndroidKey.BACK));
Thread.sleep(1000);
driver.findElement(saveLoc).click();
Thread.sleep(5750);
if (driver.findElements(By.xpath("//*[@text='Check your zip and try again']")).isEmpty()) {
    System.out.println("Zip code change successful");
} else {
    continue start;
}

2. Replace Labels with a while Loop (Recommended)

Labels paired with continue can make code harder to read and maintain. Since your goal is to retry the zip code logic until it succeeds, a while loop is a much more idiomatic and clean approach:

String zipCodes1 = "90019,90036,90016,...";
String[] zipArray1 = zipCodes1.trim().split(",");
Random zipIndex1 = new Random();
boolean zipChangeSucceeded = false;

while (!zipChangeSucceeded) {
    int randomIndex1 = zipIndex1.nextInt(395);
    System.out.println("Trying zip code: "+zipArray1[randomIndex1]);
    driver.findElement(By.xpath("//android.widget.EditText[@index='0']")).sendKeys(zipArray1[randomIndex1]);
    Thread.sleep(750);
    driver.pressKey(new KeyEvent(AndroidKey.BACK));
    Thread.sleep(1000);
    driver.findElement(saveLoc).click();
    Thread.sleep(5750);
    
    if (driver.findElements(By.xpath("//*[@text='Check your zip and try again']")).isEmpty()) {
        System.out.println("Zip code change successful!");
        zipChangeSucceeded = true;
    } else {
        System.out.println("Zip code failed, retrying...");
    }
}

This approach eliminates the label entirely, makes your intent clearer, and avoids any scope-related compiler errors.

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