初期化がうまくいかない

Question

以下のコードで初期化を試みましたが、Object内のどの要素を取り出そうとしてもエラーで終了してしまいます。

何か良い解決方法があればご教授ください。

m = 0;n = 0;
        Object = new int[49][49];
        while (m != DEF + 1 && n != DEF + 1)
        {
            if(n == m)
            {
                Object[m][n] = 0;
                Toast ts = Toast.makeText(this,"n == m",Toast.LENGTH_SHORT);
               // ts.show();
                n++;
            }
           else if(n != DEF)
           {
               Object[n][m] = 1;
               Toast ts = Toast.makeText(this,"n != DEF",Toast.LENGTH_SHORT);
               //ts.show();
               n++;
           }
            else if(n == DEF)
            {
                n = 0;
                Toast ts = Toast.makeText(this,"n == DEF",Toast.LENGTH_SHORT);
                ts.show();
                m++;
            }
        }

Comments

[swordone]

どういうエラーが出ているかを記載して下さい。あと、Objectを変数名で使うのはやめて下さい。

[shinig001]

問題が発生したため終了します。とのメッセージがでますが...。エラー内容の確認方法があれば教えていただければ幸いです。あと、変数名に関しては以後気を付けます。ご指摘ありがとうございました。

[swordone]

デバッグで動かしているのならLogcatという画面に例外ログが出ているはずです。

[j7408022]

DEFってなに。

[shinig001]

DEFは５が代入されています。
説明不足でした。申し訳ありません。

[j7408022]

Objectの定義文は？
今回はいらないけどm,nなどの変数はすべて定義を記載するべき。

Objectの定義を間違えてる気がする。。。

Answer 1

DEFの値がわかりませんが、もし49だとすれば、m=49になった際にObject[49][n]を参照することになり、これは存在しないので(配列のインデックスは0~48)、ArrayIndexOutOfBoundsExceptionになります。
また、表示しきれないToastが大量にたまるのも問題になりそうな予感がします。

Toastを無視すれば、for文で簡単に書けるのではないですか？
ややこしいので対象の配列をarrayとします。

for (int m = 0; m < DEF; m++) {
    for (int n = 0; n < DEF; n++) {
        array[m][n] = (m == n) ? 0 : 1;
    }
}

また、各列インデックスが同じものだけ違う値を入れるだけの操作で、入れるごとに条件判断するのも煩雑なので、一旦1で埋めてから条件に合う場所だけ0にするという方法も考えられます。

for (int m = 0; m < DEF; m++) {
    java.util.Arrays.fill(array[m], 1);
    array[m][m] = 0;
}