元の配列の値は変更せず、int型の配列xを受け取り、値を逆に入れ替えた新しい配列を作成し、その参照を返すメソッドの作成したい。

Question

ソースを書いてみると長くなってしまったので、もっと簡潔で正しいソースはどのように記述するといいでしょうか。
見辛いソースでお手数をおかけしますが、回答をお願いいたします。

元の配列の値は変更せず、int型の配列xを受け取り、値を逆に入れ替えた新しい配列を作成し、その参
照を返すメソッドの作成したい。(動作を確認するための適切なmain()メソッドも作成し、動作を確認が必要)

package program01;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class Program01 {
	public static void main(String[] args) {
		try {
			BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
			System.out.print("配列のサイズを入力してください:");
			int array_num = Integer.parseInt(reader.readLine());
			System.out.println("値を入力してください:"); 
			System.out.print("x[" + 0 + "]" + "=");
			int array_value = Integer.parseInt(reader.readLine());
			int[] x = new int[array_num];
			x[0] = array_value; 
			for (int i = 1; i < x.length; i++) {
				System.out.print("x[" + i + "]" + "=");
				int array_value2 = Integer.parseInt(reader.readLine());
				x[i] = array_value2;
			}
			int[] y = x;
			System.out.println("リバースしてコピーしました。"); 
			reverseArrayCopy(y);
			System.out.println("元の配列をもう一度表示します。");
			System.out.println("(変更されていませんね。)");
			for (int i = 0; i < x.length; i++) {
				System.out.println("x[" + i + "]" + "=" + x[i]);
			}
		} catch (Exception e) {
			// tryの中でエラーがあればこの中に入る
			System.out.println("入力エラーです");
		} finally {
			// エラーでも正常でも実行します
			System.out.println("プログラム終了");

		}
	}

	public static int[] reverseArrayCopy(int[] y) {
		for (int i = (y.length) - 1; i >= 0; i--) {
			int j = (y.length - i) - 1;
			System.out.println("y[" + j + "]" + "=" + y[i]);
		}
		return y;
	}
}

Answer 1

ソースを書いてみると長くなってしまったので、もっと簡潔で正しいソース

• ソースが長い原因はなんだろう
• ソースが正しくない点はなんだろう

この２点について「自分で確認する」「自分で原因を考える」ことをまず行い、それに対して意見を求めるような質問にした方がよいと思います。

プログラムを書いたばかりで自分で吟味せずに単に添削を求めてみてもプログラミングの知識や力はつかないと思います。また質問者さんが何を考えたかを聞けないと「何をアドバイスすると約にたちそうか」が絞れないものです。

Answer 2

以下、サンプル、動作未確認。元のプログラムでやろうとしていることの推測。

    public static int[] reverseArrayCopy(int[] y) {
        for (int i = 0, rev_i = y.length - 1; i < rev_ i; i++, rev_i--) {
            int temp = y[i];
            y[i] = y[rev_i];
            y[rev_i] = temp;
        }
        return y;
    }

以下、サンプル、動作未確認、お題を読んででやろうとしていることの推測

    public static int[] reverseArrayCopy(int[] y) {
        int size = y.length;
        int[] x = new int[size];

        for(int i = 0; i < size; i++){
            x[i] = y[size - 1 - i];
        }
        return x;
    }