爲什麼我的字符串索引超出範圍？

我正在爲我的計算機科學課做一個hang子手遊戲，似乎無法弄清楚如何解決我的問題與字符串。我遇到了一個越界索引錯誤。它說爲什麼我的字符串索引超出範圍？

java.lang.StringIndexOutOfBoundsException：字符串索引超出範圍：7

在這裏發生：

if(theGuess.equals(wordToGuess.substring(i,i+1)))

下面是程序代碼，如果它是任何幫助。

import javax.swing.JOptionPane; 
public class Hangman extends BasicGame 
{ 
    private final String WORDCHOICES= "apple"+"great"+"zebra"+"mouse"+"chick"+"class"+"abhor"+"abide" 
     +"fuzzy"+"brute"+"blunt"+"comic"+"cater"+"stone"+"chaos"+"dufus"+"earth"+"decal"+"happy"+"heist" 
     +"idler"+"lions"+"hates"+"idols"+"lasso"+"lives"+"lisps"+"major"+"mound"+"mango"+"meter"+"mercy" 
     +"marry"+"pilot"+"plots"+"pants"+"overt"+"quack"+"paver"+"polls"+"scorn"+"sapid"+"sails"+"rowdy" 
     +"seeks"+"leech"+"seats"+"spade"+"shoes"+"slurp"; 
    private String wordToGuess; 
    private java.util.Random randy; 

    private int wordNum; 
    private int numCorrect=0; 
    private String[] correctLetters= new String[]{"","","","",""}; 
    HangDraw artist= new HangDraw(); 
    public Hangman() 
    { 
     super(); 
     randy= new java.util.Random(); 
     for(int i = 0; i<5;i++) 
      correctLetters[i]=null; 
     wordNum=0; 
     numCorrect=0; 
     artist.setUp(); 
    } 
    public void guess() 
    { 
     wordNum= 5*randy.nextInt(50); 
     numCorrect=0; 
     int wrong=0; 
     String userGuess=""; 
     int partsDrawn=0; 
     wordToGuess=WORDCHOICES.substring(wordNum,wordNum+5)+" "; 
     while(numCorrect<5&& partsDrawn<5) 
     { 
      userGuess= JOptionPane.showInputDialog("Guess a letter, so far you have: "+ correctLetters[0]+ 
         correctLetters[1]+correctLetters[2]+correctLetters[3]+correctLetters[4]); 

      if(checkLetter(userGuess)) 
      { 
       JOptionPane.showMessageDialog(null, "Correct Guess"); 
       //print the letter 
      } 
      else 
      { 
       //draw the part of the body 
       JOptionPane.showMessageDialog(null,"incorrect"); 
       partsDrawn++; 
       artist.drawParts(partsDrawn); 
      } 
     } 
     if(partsDrawn==5) 
     { 
      JOptionPane.showMessageDialog(null, "failed to guess, the word is: "+wordToGuess); 
     } 
     else 
     { 
      JOptionPane.showMessageDialog(null, "correct, the word was: "+ wordToGuess); 
     } 
    } 

    private boolean checkLetter(String theGuess) 
    { 
     boolean matches=false; 
     for(int i=0;i<wordToGuess.length();i++) 
     { 
      if(theGuess.equals(wordToGuess.substring(i,i+1))) 
      { 
       correctLetters[i]=theGuess; 
       matches=true; 
       numCorrect++; 
      } 
     } 
     return matches; 
    } 

}

謝謝您提供

來源

2013-10-29 abysmaldan

您是否知道'WORDCHOICES'是以下字符串：'applegreatzebramousechickclassabhorabidefuzzybrutebluntcomiccaterstonechaosdufusearthdecalhappyheistidlerlionshatesidolslassoliveslispsmajormoundmangometermercymarrypilotplotspantsovertquackpaverpollsscornsapidsailsrowdyseeksleechseatsspadeshoesslurp'？爲什麼不使用數組？那麼你的話不會被限制在5個字符。 – Cruncher

我不應該使用數組，我的老師想要250個字符的字符串中有50個5個字母的單詞，謝謝你的建議，儘管 – abysmaldan

似乎每天都有人在這裏有一個由老師組成的不同的瘋狂要求。如果你想教字符串操作，那麼不要使用一個具有更好的數組/集合解決方案的例子。顯然這不是你的錯，有時候我真的不明白。 – Cruncher

因爲你明顯超出範圍。你不能從字符串的末尾開始子串。

for(int i=0;i<wordToGuess.length();i++) 
{ 
    if(theGuess.equals(wordToGuess.substring(i,i+1))) 
    { 
     correctLetters[i]=theGuess; 
     matches=true; 
     numCorrect++; 
    } 
}

找出

最快的方法是調試應用程序

來源

2013-10-29 17:18:38 TecHunter

substring的endIndex是唯一的，因此它可以等於wordToGuess.length（），您的解決方案將跳過最後一個字符。 – Daniel

好的，謝謝，我仍然需要檢查最後一個字符，所以我在之前的章節中將字符串連接到「wordToGuess」的末尾並修復它 – abysmaldan

@abysmaldan當你所需要做的是連接空間是一個可怕的解決方案修復你的循環參數。 – jlars62

任何幫助的問題是，你是出界外的前面for循環：for(int i=0;i<=wordToGuess.length();i++)。這將循環超過字符串中可用字符的末尾。注意到您使用的是使用端點標記的子字符串函數，後一個端點必須位於字符串的範圍內，因此應將此循環指定爲for(int i=0;i<wordToGuess.length();i++)。這將確保最後一次迭代不會調用IndexOutOfBoundsError。

來源

2013-10-29 17:17:45 abiessu

您的代碼將跳過最後一個字符。 /最後編輯修正了它 – Daniel

更正了，謝謝，錯過了端點上Java文檔中的'獨佔'標記。 – abiessu

根據Java docs，字符串＃子引發IndexOutOfBoundsException

如果將beginIndex爲負，或endIndex大於此String對象的長度大，或beginIndex大於endIndex。

在你的循環的最後一次迭代，i將等於字符串的length，並i+1比字符串的長度，因此異常大。

所以，你需要改變：

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

到

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

來源

2013-10-29 17:18:19 jlars62

從i<=wordToGuess.length()在for循環改變你的狀況，i<wordToGuess.length()，因爲現在你的循環來的那一刻，當i變最後一個元素，所以i+1自然指向「超出界限」。

來源

2013-10-29 17:18:32

補充以下變化

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

來源

2013-10-29 17:18:55 Shamse

這爲什麼解決這個問題，OP的錯誤是什麼？ – Mark

你的循環是for(int i=0;i<=wordToGuess.length();i++)

讓我們假設一個簡單的例子，wordToGuess="ABC"

循環生成i = 0到i的值= 3。

對於i = 0，選擇第一個字符，對於i = 1第二個，對於i = 2第三個，i = 3沒有意義。

因此使用for(int i=0;i<wordToGuess.length();i++)

來源

2013-10-29 17:19:05 Daniel

核心Java是在告訴你什麼是錯的真棒。你說if(theGuess.equals(wordToGuess.substring(i,i+1)))是拋出IndexOutOfBoundsException？這意味着wordToGuess長度小於一個字符（即空字符串）。基本上你試圖獲得長度爲1 [i，i + 1）的子字符串。

因爲你是學生，我認爲自己完成這些工作很重要，我只是想給你一個提示：看看for循環的條件。另外，用調試器遍歷代碼，或者至少放入一些System.out.println（wordToGuess）語句來查看這些值是什麼。

來源

2013-10-29 17:22:22 MadConan

爲什麼我的字符串索引超出範圍？

回答

相關問題