Logの計算時に落ちてしまう

Question

以下のコードで実行するとx = 4のときsumLog[ x ] = の式のあとにアプリが終了してしまいます。
尚、sum[ x ]はどの場合も先ほどの式の時には４になっています。

AREA = 5,ALLNODES = 20

        int x = 0,y = 0;
        int[] sum;
        double sumLog[];

        sum = new int[AREA];
        sumLog = new double[AREA + 1];

        while ( x != AREA + 1)//x座標
        {
            while( y != AREA + 1)//y座標
            {
                sum[ x ] += array[ x ][ y ];
                y++;
            }

            if ( x > AREA )
            {
                Toast ts = Toast.makeText(this,"確認",Toast.LENGTH_SHORT);
                ts.show();
            }
            sumLog[ x ] = Math.log10(sum[ x ]) / Math.log10(ALLNODES);
            x++;
            y = 0;
        }

なぜ落ちてしまうのでしょうか、、、
どなたかご教授ください。

Comments

[swordone]

落ちたときのエラーログを載せて下さい。加えてarrayとはなんですか？

Answer 1

配列sumの要素数はAREA(=5)であるので、使用できる添え字は0から4まで。
一方ループカウンタxは0から5(ループ終了条件がx != AREA + 1(= 6)であるため)の値をとりえる。

つまり配列sumの要素数を超えて使用してしまうことになります。

Answer 2

int sum[]はnew int[AREA]で初期化されているので要素は5です。
ところがif(x>AREA)で判定した後にsum[x]としていることから要素数を超えた領域にアクセスしているせいで落ちていると推測されます。

Comments

[shinig001]

まさしくその通りでした！
無事解決しました。

ありがとうございました。

Answer 3

落ちている原因は他の方の回答のとおりだと思いますが、while文の書き方としてwhile ( x != AREA + 1)という書き方は危険です。xが(AREA+1)以外の時はwhileの条件が成立してしまうためです。
色々と処理を追加していく中で、ループの中でx += 2;とやってしまうとループを抜けられなくなる可能性があります。

while(x < AREA+1)

と不等号を使ったほうが危険度は減ります。

Comments

[shinig001]

アドバイスありがとうございます

今後、失敗を減らすためにも参考にさせていただきます。