配列が正しくループしていない - Java

Question

私は60個の値を持つ配列を持っています。次のボタンをクリックすると、番号60まで昇順で配列のすべての値が循環します。再び。

私も以前のボタンがあるので、上に行く代わりに値を下げることができます。私が最初の配列値[0]の前のボタンを押すと、私のアプリケーションがクラッシュし、私はなぜそれがわかりません。

public String nextFact() { 
    i++; 
    if(i >= facts.length) { 
     i = 0; 
    } 
    return facts[i]; 
} 

public String previousFact() { 
    i--; 
    if(i < 0) { 
     i = facts.length; 
    } 
    return facts[i]; 
} 

Answer 1

有効な配列インデックスは0からfacts.length - 1までの範囲ので、あなたは、facts.lengthにiを変更するときは、ArrayIndexOutOfBoundsExceptionを得ている：

は、ここに私のコードです。 i〜facts.length - 1に設定します。

if(i < 0) { 
    i = facts.length - 1; 
} 

長さ以上の折り返しコードは正常に動作しているはずです。

Answer 2

配列のサイズはarray.lengthです。したがって、最後のインデックスはarray.length-1になります。前の関数では、array.lengthをiに代入します。これはarrayのmax indexより大きいインデックスであるため、クラッシュします。 indexoutofboundエラーも発生しているはずです。

あなたはこれでその行を置き換える必要があります。

i = facts.length - 1;