java.lang.ArrayIndexOutOfBoundsException: length=0; index=0 のエラーについて

Question

public class Test {
private int number;
private int[] x = new int[number];
private int[] y = new int[number];
private int a = 10;
private int b = 30;

public Test(String name) {

 switch(name) {
    case "first":
      this.number = 5;
      for(int i = 0; i < number; i++) {
          this.x[i] = a;
      }
      this.y[0] = b;
      for(int i = 1; i < number; i++) {
      this.y[i] = this[i-1] + b;
      }
    break;
    case "second":
      this.number = 8;
      for(int i = 0; i < number; i++) {
          this.x[i] = a * 2;
      }
      this.y[0] = b * 3;
      for(int i = 1; i < number; i++) {
      this.y[i] = this[i-1] + b * 3;
      }
   break;
 }

}

xとyの初期値を決めようとすると、xのfor文の中で、
ArrayIndexOutOfBoundsException
エラーが出ます。
どう書き直せばいいでしょうか。

Comments

[amadablam]

少なくとも実行時例外の発生するコードはコンパイルの通る単位で、きちんと整形して提示しましょう。

[swordone]

switch(name)のnameとは？

[swordone]

コードは専用のマークダウンを使ってください。

Answer 1

メンバ変数の宣言時にx,yの初期化を行っているためです。
メンバ変数の初期化はコンストラクタに先立って行われ、初期化宣言を行っていないメンバは0やnullなどになります。
つまりxやyの初期化をする時点ではnumberは0のため、x,yは長さ0の配列になります。
長さ0のため、インデックス0にすらアクセスできないので、エラー(例外ArrayIndexOutOfBoundsException発生)になります。

numberの存在意義がわかりませんが、それを維持するなら、コンストラクタでnumberの設定をした後、配列の初期化をする必要があります。一例が次のようになります。

public class Test { 
    private int number; 
    private int[] x;
    private int[] y; 
    private int a = 10; 
    private int b = 30;

    public Test(String name) {

        switch(name) { 
        case "first": 
            this.number = 5; 
            x = new int[number];
            y = new int[number];
            for(int i = 0; i < number; i++) { 
                this.x[i] = a; 
            } 
            this.y[0] = b; 
            for(int i = 1; i < number; i++) { 
                this.y[i] = this[i-1] + b; 
            } 
            break; 
        case "second": 
            this.number = 8;
            x = new int[number];
            y = new int[number]; 
            for(int i = 0; i < number; i++) { 
                this.x[i] = a * 2; 
            } 
            this.y[0] = b * 3; 
            for(int i = 1; i < number; i++) { 
                this.y[i] = this[i-1] + b * 3; 
            } 
            break; 
        } 
    }
}

Answer 2

配列のオブジェクトを作成した時点の変数numberに、値が設定されていないことが問題です。配列はオブジェクト作成時にリストサイズが決まるので現在の実装で行っているthis.numberは反映されません。
実装を見る限りやりたいことは伝わってきますが、まだ理解が足りていない部分が多くあるようにみえます。
Javaの基本データ型と参照型に対する理解を深めたほうがいいと思われます。

実装に関しては以下の部分を変更すればエラーは避けることができると思います

---

private int number = 5;
private int[] x = new int[number];
private int[] y = new int[number];

---

Comments

[退会済みユーザー]

Testクラスで、nameで名前をつけていくつかインスタンスを作って、nameごとにx, yの配列のサイズを変える、ということをやりたかったんですが、できないんですかね。

[kaPong]

可能です。
今回のエラー原因は、メンバ変数x,yの宣言時に値の設定されていないnumberを配列のサイズとして指定しているためです。
なので、単純にコンストラクタ内でメンバ変数x,yを初期化するように書き換えれば問題ないと思います。

以下が一部の例

---

switch(name) {
case "first":
int number = 5;
x = new int[number];
y = new int[number];
for(int i = 0; i < number; i++) {
this.x[i] = a;
}

---

Answer 3

以下のコードで確認すると、int[]x,int[]yの配列の長さは
最初にprivate int number=1;
として入れた長さ1になっていて、
this.number = 5;
としてnumberの値を入れ替えても配列の長さは変わず1のままな事が分かります
この状態で配列の長さは1なのに
for(int i = 0; i < number; i++) {
で入れ替えたnumberの数5回分だけfor分を回転させて
中でx[]の中身をx[4]まで操作しようとするため、存在しないx[1]～x[4]までを参照しようとして
エラーが出るのです

public class R2{ 
private int number=1; 
private int[] x = new int[number]; 
private int[] y = new int[number];
int a=10;
int b=12;


public R2(String name) {

switch(name) { 
case "first": 
this.number = 5; 
for(int i = 0; i < number; i++) { 
System.out.println("2");
this.x[i] = a; 
} 
this.y[0] = b; 
for(int i = 1; i < number; i++) { 
this.y[i] = this.y[i-1] + b; 
}

}

}
public static void main(String[] args){



R2 xc=new R2("first");



}



}

コンストラクタの引数ごとに配列の長さを変えられるようにしたのが以下です

class Test {
	private int number = 1;
	int[] x = new int[number];
	int[] y = new int[number];
	private int a = 10;
	private int b = 30;

	public Test(String name) {

		switch (name) {
		case "first":
			this.number = 5;
			x = new int[number];
			for (int i = 0; i < number; i++) {
				this.x[i] = a;
			}
			y = new int[number];
			this.y[0] = b;
			for (int i = 1; i < number; i++) {
				this.y[i] = this.y[i - 1] + b;
			}
			break;
		case "second":
			this.number = 8;
			x = new int[number];
			for (int i = 0; i < number; i++) {
				this.x[i] = a * 2;
			}
			y = new int[number];
			this.y[0] = b * 3;
			for (int i = 1; i < number; i++) {
				this.y[i] = this.y[i - 1] + b * 3;
			}
			break;
		}
	}
}

class E2 {

	public static void main(String[] args) {

		Test a = new Test("first");
		Test b = new Test("second");

		for (int c : a.x) {
			System.out.print(c + "-");
		}
		System.out.println();

		for (int c : a.y) {
			System.out.print(c + "-");
		}
		System.out.println();

		for (int c : b.x) {
			System.out.print(c + "-");
		}
		System.out.println();

		for (int c : b.y) {
			System.out.print(c + "-");
		}

	}

}