对于循环错误，我该如何解决？

Question

for(int x = 0;x<14;x++){
    day[x]= theSheet.changeLetters(day[x]);
}

public String changeLetters(String entering){
    if(entering.equalsIgnoreCase("a")){
        entering = "10";
    } else {
        if(entering.equalsIgnoreCase("b")){
            entering = "11";
        } else {
            if(entering.equalsIgnoreCase("c")){
                entering = "12";
            } else {
                if(entering.equalsIgnoreCase("d")){
                    entering = "13";
                } else {
                    if(entering.equalsIgnoreCase("e")){
                        entering = "14";
                    } else {
                        if(entering.equalsIgnoreCase("f")){
                            entering = "15";
                        } else {
                            if(entering.equalsIgnoreCase("g")){
                                entering = "16";
                            } else {
                                if(entering.equalsIgnoreCase("h")){
                                    entering = "17";
                                }
                            }
                        }
                    }
                }
            }
        }
    }
    return entering;
}

说错误在这里 if(entering.equalsIgnoreCase("a")) 并且在我用来运行该方法的 for 循环中。我正在尝试将放入字符串中的字母更改为数字。

谁能向我解释错误可能在哪里？我很难发现问题。它让我可以很好地输入字母，但是一旦进入此 for 循环并运行此方法，它就会出现异常。

Answer 1

正如@jlordo 已经说过的那样，避免嵌套如此深度的 if 语句。

作为替代方案，您可以将 switch 语句与 enum 结合使用 - 尽管以下方法包含更多代码，但它更通用且更适合扩展（例如，使用字母表中的所有字母及其他字母） ) 并且相对于您想要表达的内容而言，它会使代码更具可读性和更合适。

Letter letter;

if (letter.equals("a")) letter = Letter.a;
if (letter.equals("A")) letter = Letter.A;
// and so on ...

switch (letter) {
    case a : { 
        // do some code here ...
        break;
    }
    case A : { 
        // do some code here ...
        break;
    }
    // and so on ...
}

public enum Letter {
    a (1),
    A (2),
    b (3),
    B (4),
    c (5),
    C (6);
    // and so on

    private final int index;

    Letter(int i) {

        index = i;
    }

    public int getIndex () {

        return index;
    }
}

请注意，如果您使用的是 Java 7，即使没有枚举也可以使用 switch 语句，因为它也接受字符串，如下所示：

switch (entering) {

    case "a" : {

        // ...
    }

    // ...
}

Answer 2

根据

是的，代码在 for 循环中初始化，然后使用 for(int x =1;x

你有一个逻辑错误。您正在覆盖以前的条目，而不是将数组填充到最多 14 个项目。所以8 之后的项目被留下null，因此NullPointerException。

试试这个：

String[] day = new String[14];

for( int i = 0; i < 14; i+=2 ) {
    day[i] = JOptionPane.showInputDialog("Enter hour pairs for day "+(i/2+1) +".\n Enter the first digit: ");
    day[i+1] = JOptionPane.showInputDialog("Enter the second digit: ");
}

---

作为奖励，您可以通过以下方式简化 if-else 代码：

public String changeLetters( String entering ) {
    return String.valueOf(entering.toUpperCase().charAt(0) - 55);
}

Answer 3

你为什么不使用

if (condition) {
    // ...
} else if (condition2) {
   // ...
} else if (condition3) {
   // ...
}
// and so on

使您的代码更具可读性。你的嵌套条件是一团糟。如果你修复它们，你也可以修复错误（如果它在你展示给我们的代码部分中）。

同时添加

System.out.println("Entering = '" + entering "'");

在你的方法开始时，看看是否真的收到了你期望它收到的东西。