偶数だけが並ぶ配列について

Question

前提・実現したいこと

配列xを用意し、その配列ｘの中の偶数だけを配列ｙ（そのとき配列ｙの要素数も配列ｘの偶数の要素数分にしたい）に入れたいです。
該当のソースコードを実行した際、以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

{2,4,

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 5
at Main.main(Main.java:8)

該当のソースコード

public class Main { 
  public static void main(String[] args){
    int[] x = new int[]{1,2,3,4,5,6,7,8,9,10};
    int[] y = new int[x.length / 2];　//配列ｘの要素数の半分の要素数をもつ配列y（配列ｘの偶数の数分）
      System.out.print("{");
        for(int i = 0; i < x.length; i++){　//配列ｘの要素数分for文を回す
          if(x[i] % 2 == 0){　//もしx[i]が偶数なら
            y[i] = x[i];　//y[i]にx[i]の値を入れる
              if(i < x.length - 1){
                System.out.print(y[i] + ",");
              }else{
                System.out.print(y[i]);
              }
          }
        }
      System.out.print("}");
  }
}

2と4は出力されているのに、残りの6,8,10が出力されないのはどうしてなのでしょうか？
どこがおかしいかアドバイス等いただけますと幸いです。

Answer 1

yの要素数が5しかないためです。

Comments

[nununu_nu]

ｙの要素数が５でも、その中に2,4,6,8,10の計5つの要素を入れてあげたいので問題ないと勝手に解釈してしまっていたのですが…最初にｙの要素数を入れる時の「x.length/2」がいけないということですか？

[swordone]

とりあえずi=4までやって、yの中身を出力してみてください。
あなたが思っているのとは違う結果になるはずですから。

[nununu_nu]

[0, 2, 0, 4, 0]←こうなりました
ということは、もし要素数10だとしても[0,2,0,4,0,6,0,8,0,10]と出力されてしまう、ということですよね
これを[2,4,6,8,10]にするにはどうすれば良いのでしょうか…？
→ｙ[i/2]にx[i]の値を代入してあげることで解決しましたが、問題ないのでしょうか？無理やりな感じがしますが…

[退会済みユーザー]

なぜ無理やりな感じがしたのでしょうか。
もし、奇数のときは変な挙動をするのではないかと考えているなら
x[i] % 2 == 0をx[i] % 2 != 0に変えてみてください。
奇数が出力されるはずです。

[kazuma-s]

配列 x が { 2, 2, 3, 6, 0, 6, 7, 9 } のとき、配列 y は { 2, 2, 6, 0, 6 } にならないといけないんですよね。x の要素数は 8。y の要素数は 5。

[退会済みユーザー]

なるほど。
xの要素がランダムだと例外が出る可能性があるんですね。
ただ、それを許容するとなるとそもそもyの初期化部分を変更しないといけなさそうですが。

[nununu_nu]

今回は配列xに1〜10までの10個の数が順番に入っていて、その半分が偶数になるのが分かってたので要素数等を半分にすればいいのがわかるのですが、これがもし配列xにバラバラの数が入ってるとなると解決できなくなりますよね…その場合についてはまた考えてみます、ご丁寧にありがとうございました！

Answer 2

次の手順でやればうまく行くのは分かりますか？

1. x の中の偶数の個数を求める。
2. その個数の要素を持つ配列 y を確保する。
3. x の中の偶数を y にコピーする。
4. y の値を表示する。

public class Main { 
	public static void main(String[] args){
		int[] x = {1,2,3,4,5,6,7,8,9,10};

		int n = 0;
		for (int e : x)
			if (e % 2 == 0) n++;

		int[] y = new int[n];

		n = 0;
		for (int e : x)
			if (e % 2 == 0) y[n++] = e;

		System.out.print("{");
		if (n > 0) {
			System.out.print(y[0]);
			for (int i = 1; i < n; i++) System.out.print("," + y[i]);
		}
		System.out.println("}");
	}
}

for (int e : x) if (e % 2 == 0) n++;
は
for (int i = 0; i < x.length; i++) if (x[i] % 2 == 0) n++;
と書くこともできます。

e % 2 == 0 は (e & 1) == 0 と書くこともできます。

以上のことを理解できてもできなくてもコメントをお願いします。

Comments

[nununu_nu]

分かりやすかったです！ありがとうございます！

Answer 3

swordone さんの答えの通りですが、y のインデックスを j など別の変数で管理して、代入した際にインクリメントするのも一つの方法かと。