ArrayIndexOutOfBounds例外

質問

ArrayIndexOutOfBoundsExceptionのバグが発生しています。

下記コードで、どうして上記のバグが発生するのか不明で困っておりますので、お分かりの方がいらっしゃったら、ぜひお教えください。

Dataクラスというクラス内に次のようにコードを書いています。

//全部で何問か。

static int x;

//x行10列の配列の生成。

static String[][] questionArray = new String[x][10];

//要素数xの重複がない０から１６６までの乱数の生成。

static int[] getrandomNo(){

int[] no = new int [x];

for(int i=0; i<x; i++){

int j;

do{
j=0;

no[i]=1+rand.nextInt(166);

for(; j<i; j++)

if(no[j] == no[i]) break;

} while (j<i);

}

return no;

}

//167行10列のString型二次元配列ｔから、ｘ行分抜き出しランダムに構成。

static String[][] getquestionArray(){

int []ra = getrandomNo();

for(int i=0 ;i<x ; i++)

for(int j=0; j<10;j++)

questionArray[i][j] = t[ra[i]][j];　

return questionArray;

}
//上記の二次元配列を変数qAに代入。

static String[][] qA = getquestionArray();

これを、別のTestクラスからアクセスします。

//klabel1にqAの5行5列目の文字列を表示。

　　klabel1.setText(Data.qA[4][4]);

ここで上記のArrayIndexOutOfBoundsExceptionが出るようなのです。

どこで問題が発生しているのかわからず、困っています。

お分かりの方、ぜひご教授ください。

回答No.7

Tacosan · ベストアンサー

質問文中のプログラム (の断片) と全然違うじゃん. これじゃぁ, 書かれても意味ないよ....

t の初期化を
static String[][] t = new String[167][10];
としてみたけど, うちの環境 (Windows 7 Ultimte, 64bit, java のバージョンは 1.6.0_24) では「10→9にすると問題なく動くが」のところを 10 にしたままでもまったく問題ない (というか ArrayIndexOutOfBoundsException は出ない) ですねぇ.

うん, 今 1.7.0_03 にしたけどやっぱり出ない.

「実は t の初期化のときに要素数を間違えている」とかいうオチ (つまりは #6 の指摘の通り) だったりして. t.length とか t[0].length とか t[166].length とかって見た?

お礼

Tacosan様

たびたびのコメントありがとうございます。

解決しました。

これまでお二人にご指摘いただいていた通りの個所が原因でした。

２箇所ありました。

一箇所は、ご指摘の通り、167行10列のつもりの配列が、t[98]が９列になっていました。

そして、もう一箇所は、xに自分のつくったtestクラスから数値を代入しているつもりで代入できていませんでした。NO.3へのお礼で「Testクラスからではなく、直接代入してもダメでした。」と記入していますが、これは、上記のt[98]が９列になっていたことが原因でした。t[98]を10列になおし、xへの代入方法を変えると、ArrayIndexOutOfBoundsExceptionは出ずに、動くようになりました。

Tacosan様とaskaaska様、お二人のどちらもベストアンサーとさせていただきたいのですが、システム上、一人しかできませんので、修正する具体的手がかりをいただいたTacasan様をベストアンサーとさせていただきます。

askaaska様にも本当に的確なアドバイスをいただき心より感謝申し上げます。

回答No.6

askaaska

実際に発生しているのは
t[ra[i]][j];　
の場所だったりしない？

お礼

askaaska様

ご指摘ありがとうございます。

私もそこのところ怪しいと思いつつ、検証の仕方がわからず、確認できていないという状況です。検証できしだい、またご報告したいと思うのですが、取り急ぎ御礼申しあげます。

そこのところを考えてみようという覚悟をいただきました。ありがとうございます。

回答No.5

Tacosan

ごめん, どう確かめたのかが全くわからない.

可能なら, コマンドプロンプト上で実行できる「完全なプログラム」を出してもらえますか?

お礼

Tacosan様

重ねてお世話になります。以下のコードで、コマンドプロンプトで確かめました。

import java.util.Random;
class Array2{
static String[][] t = {　省略。167行10列のString型2次元配列。　};
static Random rand = new Random();
static int x =167 ;
//要素数xで、０から１６６までの整数を重複なくランダムに並べた配列。
static int[] getrandomNo(){
int[] no = new int [x];
for(int i=0; i<x; i++){
int j;
do{
j=0;
no[i]=rand.nextInt(167);//最初の質問では1～167になるので修正。
for(; j<i; j++)
if(no[j] == no[i]) break;
} while (j<i);
}
return no;
}
//x行10列のStrin型2次元配列の生成。
static String[][] questionArray = new String[x][10];
//上記x行10列のString型２次元配列に、他の167行10列のString型配列から、ランダムにx行
//抽出し、さらに順序もランダムに変えて代入。
static String[][] getquestionArray(){
int []ra = getrandomNo();
for(int i=0 ;i<x ; i++)
for(int j=0; j<10; j++) //このままだとArrayIndexOutOfBoundsException。10→9にすると問題なく動くが、10列目が
//nullになる。ただ、実際に作成している環境では10→9に修正してもArrayIndexOutOfBoundsException。
questionArray[i][j] = t[ra[i]][j];
return questionArray;
}
//変数qAで上記配列を参照させる。
static String[][] qA = getquestionArray();
//メイン
public static void main(String[] args){
//配列の大きさを表示。
System.out.println(qA.length); //xをとりあえず上で167で設定しているので167を表示。
//String型2次元配列の一部を表示。
System.out.println(qA[0][0]);　//適切に表示。
System.out.println(qA[0][8]);//適切に表示。
System.out.println(qA[0][9]);//null
System.out.println(qA[166][0]);//適切に表示。
System.out.println(qA[166][8]);//適切に表示。
System.out.println(qA[166][9]);//null
}
}

以上です。お時間使っていただき、本当にありがとうございます。

回答No.4

Tacosan

配列の大きさは確認した?

お礼

Tacosan様

重ねてのコメントありがとうございます。

確認していませんでしたので、コマンドプロンプトで実験してみました。

qA.lengthで、ArrayIndexOutOfBoundsException:9と出たので、for文中をj<9としてみたところ、

大きさを出してきました（xと同じ値になります）。

ここで、なぜj<9とすると出ないのか、自分の頭の中はけっこう混乱しているのですが、

さらに、元のプログラムでj<9としてみてもArrayIndexOutOfBoundsExceptionはやはり出てしまいさらにわからなくなっています。

また、何かご指摘いただければ幸いです。

回答No.3

askaaska

xに5より小さい値が入ったら
ArrayIndexOutOfBoundsException
になると思うけど

お礼

askaaska様

重ねてコメントをいただきありがとうございます。

ご指摘の通り、

「xに5より小さい値が入ったらArrayIndexOutOfBoundsExceptionになる」のですが、

それより大きくても、やはりなるのです。

Testクラスからではなく、直接代入してもダメでした。

重ねてのご指摘、心より感謝です。

また、お気づきの点がございましたら、教えてください。

回答No.2

askaaska

xが与えられていないからでしょ？

お礼

コメントありがとうございます。

申し訳ありません。私の質問記入の手落ちです。

変数xに対しては、Testクラスから１～１６７の数値を任意に代入して確認しています。

おそらく、そこ以外でコード記入の間違いをしていると思うのです。

何かお気づきの点あれば、他にもぜひお教えください。

回答No.1

Tacosan

どのようなときに ArrayIndexOutOfBoundsException が出るかわかっていますか?

お礼

Tacosan様

記入した欄が「補足」のほうでしたので、「お礼」が適切かと思いこちらに記入します。

コメントありがとうございました。

何か他にもお気づきの点あれば、どうぞよろしくお願いいたします。

補足

配列として確保された領域を超えた場所にアクセスした場合に出る現象だと思っています。

どこで出ているのかわかりません。

なお、最初のxについては、上記のTestクラスから値を１～１６７の範囲で任意に代入しています。

よろしくお願いします。