文字列配列は常にスローArrayIndexOutOfBoundsException

Question

私は、ユーザー入力を介して値を文字列配列に入力しようとしています。後でこのクラスでもっとやっていきたいですが、配列がいっぱいになっていることを確認して印刷したいだけです。私は特定の値を満たすためにwhileループを使用していますが、ユーザーが0を入力して停止するのを待っていますが、配列のサイズやループの値に関係なく、ArrayIndexOutOfBoundsExceptionが発生します。 0を入力してもそれを停止しません。以下の完全なコード。

また、例外はそれに関連付けられている私の配列のサイズを持っている（この場合、25）

ヘルプは非常に感謝、感謝することでしょう！

import java.util.Scanner; 

public class Dincision { 
static Scanner scanner = new Scanner(System.in); 

public static String entered; 
public static String[]foods; 

public static void main (String[]args){ 
    getChoices(); 
    int count=0; 
    for (count=0; count<=24; count++){ 
     System.out.println(foods[count]); 
    } 
} 

static public void getChoices() { 
    int i=0; 
    foods= new String[25]; 
    String input; 
    System.out.println("Enter an eating option."); 
    input=scanner.next(); 
    while (input != "0"){ 
     foods[i]=input; //error here// 
     i++; 
    } 
    System.out.println("That's all!"); 

} 
} 

Answer 1

while文のループを配列のサイズに制限する必要があります。あなたは存在しない食べ物を手に入れようとします。

Answer 2

ここに2つの問題があります。まず、それはnot the correct way to compare Stringsです。代わりにwhile(!input.equals("0")を使用してください。

第2に、ループ内に新しいデータを取得していません。配列の境界を越えるまで、同じ入力値を何度も何度も繰り返し使用し続けます。ループを次のように変更します。

input=scanner.next(); 
while(!input.equals("0")){ 
    foods[i]=input; 
    i++; 
    input=scanner.next(); 
} 

ループ内で新しい入力を取得します。

また、安全のために、whileループに境界チェックを追加する必要があります。でそれを置くためにこれ以上のスペースがありません場合は、ループを停止

while(!input.equals("0") && i < foods.length){ 

Answer 3

を、私はコメントすることはできませんので：。入力がNULLの場合、それは動作しますので、私は

!"0".equals(input) 

を好みます。