ループ脱出しようとしたらindexoutofboundsexceptionが出る

Question

ループでｎの数を増やしていってallRowListの中身がなくなったら
ループを脱出しようと考えていましたが、
indexoutofboundsexception
というエラーが出ました。
存在しないインデックスを指定するとでるエラーらしいです。
代わりにallRowListの中身がなくなったらループを抜けられるようにするにはどうしたらいいでしょうか。

package save.function;

public class JudgeName {
    HoldResult holdResult;

    public JudgeName( HoldResult holdResult) {
        this.holdResult = holdResult;
    }

    public void judgeName() {
        int n = 0;
        while ( true) {
            try {
                if (holdResult.allRowList.get( n) != null) {

                    if ( holdResult.allRowList.get( n).get( 0).equals( holdResult.getPlayerName())) {
                        holdResult.setExistsTheName( true);
                        holdResult.setTotalWin( Integer.parseInt( holdResult.allRowList.get( n).get( 1)) + holdResult.getTotalWin());
                        holdResult.setTotalLose( Integer.parseInt( holdResult.allRowList.get( n).get( 2)) + holdResult.getTotalLose());
                    }
                    else {
                        holdResult.getRemovedList().add( holdResult.allRowList.get( n).get( 0) + ",");
                        holdResult.getRemovedList().add( holdResult.allRowList.get( n).get( 1) + ",");
                        holdResult.getRemovedList().add( holdResult.allRowList.get( n).get( 2) + "\n");
                    }

                }else {
                    break;
                }

            }
            catch ( Exception e) {
                e.printStackTrace();
            }

            n++;
        }
    }
}

Answer 1

while(true)を使ってループさせるのではなく、for(int n=0; n<allRowList.size(); n++)でループさせれば、allRowListの長さを超えずにループができますよ。

もし、if (holdResult.allRowList.get( n) != null)でループ回数を制限できていると思っていたら、それはあまりよくないです。このコードの場合、存在しない要素に対してif (holdResult.allRowList.get( n) != null)の処理を行ってしまっているために、OutOfIndex例外が出てしまっているのだと思いますよ。

package save.function;

public class JudgeName {
    HoldResult holdResult;

    public JudgeName( HoldResult holdResult) {
        this.holdResult = holdResult;
    }

    public void judgeName() {
        for(int n=0; n<holdResult.allRowList.size(); n++) {
            try {
                if (holdResult.allRowList.get(n) != null) { //下の行で.get(0)、.get(1)、get(2)した時にヌルポがでないようnullチェック

                    if ( holdResult.allRowList.get( n).get( 0).equals( holdResult.getPlayerName())) {
                        holdResult.setExistsTheName( true);
                        holdResult.setTotalWin( Integer.parseInt( holdResult.allRowList.get( n).get( 1)) + holdResult.getTotalWin());
                        holdResult.setTotalLose( Integer.parseInt( holdResult.allRowList.get( n).get( 2)) + holdResult.getTotalLose());
                    }
                    else {
                        holdResult.getRemovedList().add( holdResult.allRowList.get( n).get( 0) + ",");
                        holdResult.getRemovedList().add( holdResult.allRowList.get( n).get( 1) + ",");
                        holdResult.getRemovedList().add( holdResult.allRowList.get( n).get( 2) + "\n");
                    }
                }else {
                    continue; //.get(n)がnullの時にループを終わりたかったらbreakなんですが、.get(n+1)の処理を継続してよければ、ここはcontinueです。
                }

            }
            catch ( Exception e) {
                e.printStackTrace();
            }
        }
    }
}

Comments

[退会済みユーザー]

i の未定義エラーになりそうなのと n=0固定なのか

[yukapome789]

コメントありがとうございます！全部nに更新したのに反映されてなかった・・・。元々のコードがループ内の変数にnつかってたので、iに修正するの面倒でnにしました。

[退会済みユーザー]

解決しましたありがとうございます！

Answer 2

存在しないインデックスを指定するとでるエラーらしいです。

なら、それが存在するかどうかをチェックしてループを抜けましょう

Comments

[LouiS0616]

誤爆でしょうか？

[y_waiwai]

いや、なぜか直前に別のところで書いた回答が書き込まれたようで。
teratailの誤動作？

[LouiS0616]

なるほど。このサイト、しばしば謎動作がありますよね...

[y_waiwai]

この誤動作はちょっと怖いですねえ。。

[退会済みユーザー]

編集履歴がおもしろいことになってるのかこれは（編集無し）

Answer 3

とりあえずリストの各要素をインデックス付きで巡回したいなら、for文を使えば良いかと。

for(int n = 0; n < holdResult.allRowList.size(); ++n) {
    ...
}