Java：for 循环使用字符串 charAt 方法崩溃

Question

我在这段代码中有一个错误。 调试器表明它的原因是这行代码char chr = getSecretWord.charAt(i);

这段代码的作用是寻找userInput和secretWord之间的匹配。 我有一个for loop来一个一个遍历 secretWord 字母的长度，如果有匹配的字母，则返回 true。 如果没有，则返回 false...但是当它假设返回 false 时程序崩溃了...我想这与这一行有关，但不知道究竟是什么getSecretWord.charAt(i);

    private boolean isMatchingSecretWord(String userInput)
{
    String secretWord = "";
    String getSecretWord = getSecretWord();
    for (int i = 0; i <= getSecretWord.length();i++)
        {
        char chr = getSecretWord.charAt(i);
        secretWord = ""+chr;

        if (secretWord.equals(userInput))
        {
            println("is true");
            return true;
        }
    }
    return false;
}

作为旁注，我对这段代码所做的是否正确，将 getSecretWorld() 方法分配给 String 以便我可以使用 Strings 方法length() ？

String getSecretWord = getSecretWord();

for (int i = 0; i <= getSecretWord.length();i++)

调试代码：

Exception in thread "Thread-4" java.lang.StringIndexOutOfBoundsException: String index out of range: 4    
    at java.lang.String.charAt(String.java:686)    
    at Hangman.isMatchingSecretWord(Hangman.java:49)    
    at Hangman.userInput(Hangman.java:34)    
    at Hangman.run(Hangman.java:20)*

Answer 1

for (int i = 0; i <= getSecretWord.length(); i++)

应该：

for (int i = 0; i < getSecretWord.length(); i++)
//               ^^^
//             see here

n字符字符串（或n元素数组）的有效索引为0到n-1含）。

因此，如果您的秘密词是xyyzy ，则有效索引为 0 到 4。 您的原始循环将i设置为 0 到5 进行迭代，因此存在问题。

---

但是当您可以通过一些简单的事情逃脱时，那里似乎有很多不必要的代码。

首先，我会消除混淆的来源 - 函数名称听起来像用户输入和秘密词必须完全匹配，而您的评论则另有说明：

谢谢，这有效。 但是循环的原因是用户输入了一个字母，我想看看那个字母是否在 SecretWord 中。 （这是一个刽子手游戏）。

在这种情况下，您只想查看秘密单词中是否存在单个字符。 我会更改函数名称以适应，即使那样，也可以用更少的代码来完成：

private boolean isInSecretWord (String userInput) {
    String secretWord = getSecretWord();
    return secretWord.contains(userInput);
}

Answer 2

由于您的 for 循环没有正确循环，您遇到了越界错误，我对其进行了修改，以便循环不会越界，并且您的 secretWord 变量没有正确填充，代码现在应该按预期工作:)

private boolean isMatchingSecretWord(String userInput)
{
    String secretWord = "";
    String getSecretWord = getSecretWord();
    for (int i = 0; i < getSecretWord.length();i++)
        {
        char chr = getSecretWord.charAt(i);
        secretWord = secretWord + chr;

        if (secretWord.equals(userInput))
        {
            println("is true");
            return true;
        }
    }
    return false;
}