1

私はJavaにかなり慣れていませんが、一般的にOOPではありません。私は、ユーザーがJavaクラス用にランダムに生成されたパターン(実際には数字)の増加するシーケンスに従う古典的なSimonゲームに取り組んでいます。エラーが発生しました:

java.lang.ArrayIndexOutOfBoundsException: 20
at Simon.generateSequence(Simon.java:58)
at Simon.<init>(Simon.java:23)

これを修正する方法がわかりません。配列内よりも多くの要素を呼び出しているようには見えません。コードは次のとおりです。

import acm.program.*;
import acm.graphics.*;
import java.awt.Color;
import java.awt.Font;
import javax.swing.*;
import java.awt.event.*;


public class Simon extends Program implements ActionListener
{
private int array[];
private int currentSeqLength;
private int usersInput;


public Simon()
{
    //Initialize Class Values
    array = new int[20];
    currentSeqLength = 1;
    usersInput = 0;

    generateSequence();

    while(currentSeqLength <= array.length)
    {
        playSequence();

        //Wait For User's Input Here, Assign To Variable
        //usersInput = 

        if (pushButton(usersInput)== true)
        {

            currentSeqLength++;
        }

        else
        {
            gameOverMessage();
            break;

            //Reset Variables:

        }

    }


}

public void generateSequence()
{

    //Fill Array With Random Numbers
    for (int i = 0; i <= array.length; i++ )
    {
        array[i] = (int)(Math.random()*4);

    }


}

public void setLength(int length)
{
    //Set Current Length To Size Of Given Argument
    currentSeqLength = length;

}

int getLength()
{
    return currentSeqLength;

}

int[] playSequence()
{
    //Print Out The Current Sequence

    //New Local Array To Return
    int newArray[]= new int[currentSeqLength];


    //Repeat As Many Times As Value Of currentSeqLength
    for(int i = 0; i <= currentSeqLength ; i++)
    {
        System.out.println(array[i]);

        //Return an array of int's to the player. 
        newArray[i] = array[i];

    }

    return newArray;
}

boolean pushButton(int usersInput)
{
    //Given A Button Press (0-3), Return Whether That Was The 
        //Correct Button To Play At The Moment

    if (usersInput == array[currentSeqLength])
    {
        return true;
    }

    else
    {
        return false;
    }


}

boolean isTurnOver()
{
    //If Current Sequence Length Matches Or Exceeds Value Of 
        //Array Element In Location Of Current Sequence Length

    if (currentSeqLength >= array[currentSeqLength])
    {
        return true;
    }

    else
    {
        return false;
    }

}

//Not Needed?
boolean isGameOver()
{
    if (pushButton(usersInput) == false)
    {
        return true;
    }

    else
    {
        return false;
    }
}

String gameOverMessage()
{
    return "Game Over";
}

/*public void actionPerformed(ActionEvent event)
{
    int input;


}
*/

}

4

1 に答える 1

0
for (int i = 0; i <= array.length; i++ )
    {
        array[i] = (int)(Math.random()*4);

    }

0 から 20 までループしています。20 は配列の長さ、21 は for ループが実行する反復回数です。

<= を < に置き換えます。

于 2012-11-18T20:51:43.763 に答える