コマンドライン引数で二次元配列

Question

argsで「71 85 93 83 63 31 67 77 81 87 89 55」と指定します。

test[][]に二重for文で入れると思うのですがそのやり方ができず、こんなに長いプログラムになってしまいました。

あと、せっかくint b;を使って最高点を指定しているのですがそれも生かせていません。

どうすれば短く、簡単なプログラムにできますか？

public class a {
	public static void main(String[] args) {
		int[][] test = new int[4][3];
		int n = args.length;
		int sum=0,sum2=0,sum3=0,sum4=0;
		double heikin;
		if (n == 12) {
			System.out.println("コマンドライン引数は" +n+ "個です。");
			
			for (int i=0; i<3; i=i+1) {
				test[0][i] = Integer.parseInt(args[0+i]);
				test[1][i] = Integer.parseInt(args[3+i]);
				test[2][i] = Integer.parseInt(args[6+i]);
				test[3][i] = Integer.parseInt(args[9+i]);
			}
			
			System.out.print("英語：");
			for (int i=0; i<3; i=i+1) {
				System.out.print(test[0][i]+ "\t");
			}
			for (int j=0; j<2; j=j+1) {
				for (int k=j+1; k<3;k=k+1) {
					if (test[0][j] < test[0][k]) {
						int b = test[0][j];
						test[0][j] = test[0][k];
						test[0][k] = b;
					}
				}
			}
			for (int i=0; i<3; i=i+1) {
				sum = sum + test[0][i];
			}
			heikin = (double)sum / 3.0;
			System.out.print("最高点：" +test[0][0]+ "、最低点：" +test[0][2]+ "、平均点：" +heikin);
			System.out.println();
			
			
			System.out.print("数学：");
			for (int i=0; i<3; i=i+1) {
				System.out.print(test[1][i]+ "\t");
			}
			for (int j=0; j<2; j=j+1) {
				for (int k=j+1; k<3;k=k+1) {
					if (test[1][j] < test[1][k]) {
						int b = test[1][j];
						test[1][j] = test[1][k];
						test[1][k] = b;
					}
				}
			}
			for (int i=0; i<3; i=i+1) {
				sum2 = sum2 + test[1][i];
			}
			heikin = (double)sum2 / 3.0;
			System.out.print("最高点：" +test[1][0]+ "、最低点：" +test[1][2]+ "、平均点：" +heikin);
			System.out.println();
			
			
			System.out.print("国語：");
			for (int i=0; i<3; i=i+1) {
				System.out.print(test[2][i]+ "\t");
			}
			for (int j=0; j<2; j=j+1) {
				for (int k=j+1; k<3;k=k+1) {
					if (test[2][j] < test[2][k]) {
						int b = test[2][j];
						test[2][j] = test[2][k];
						test[2][k] = b;
					}
				}
			}
			for (int i=0; i<3; i=i+1) {
				sum3 = sum3 + test[2][i];
			}
			heikin = (double)sum3 / 3.0;
			System.out.print("最高点：" +test[2][0]+ "、最低点：" +test[2][2]+ "、平均点：" +heikin);
			System.out.println();
			
			
			System.out.print("理科：");
			for (int i=0; i<3; i=i+1) {
				System.out.print(test[3][i]+ "\t");
			}
			for (int j=0; j<2; j=j+1) {
				for (int k=j+1; k<3;k=k+1) {
					if (test[3][j] < test[3][k]) {
						int b = test[3][j];
						test[3][j] = test[3][k];
						test[3][k] = b;
					}
				}
			}
			for (int i=0; i<3; i=i+1) {
				sum4 = sum4 + test[3][i];
			}
			heikin = (double)sum4 / 3.0;
			System.out.print("最高点：" +test[3][0]+ "、最低点：" +test[3][2]+ "、平均点：" +heikin);
			System.out.println();
			
		} else {
			System.out.println("エラー! コマンドライン引数が" +n+"個です。");
		}
	}

}

実行結果はこんな感じです。

コマンドライン引数は12個です。
英語：71	85	93	最高点：93、最低点：71、平均点：83.0
数学：83	63	31	最高点：83、最低点：31、平均点：59.0
国語：67	77	81	最高点：81、最低点：67、平均点：75.0
理科：87	89	55	最高点：89、最低点：55、平均点：77.0

補足説明
Eclipseを使っています。

Comments

[dodox86]

> test[][]に二重for文で入れると思うのですが

いえ、いきなり「入れると思う」と言われてもこの質問を読んだ人は何が何だか分かりません。ですので正しい回答も分かりません。どういう入力に対してどういう出力（結果）が欲しいのか、そもそもどういう課題なのか詳細に質問文に書きましょう。

> 実行結果はこんな感じです。

質問者さんのプログラムがそうなのですか？　それとも目指す結果がそうなのですか？　「こんな感じ」という雰囲気で良いのでしょうか。

[takasima20]

ぱっと見た感じだと、科目ごとの処理をサブルーチンにすればいいんじゃないの?

Answer 1

「短く、簡単」の塩梅が難しいですが、私だったらこんな感じにしますかね。

• 同じような処理はメソッドに切り出す
• 早期リターンでネストを減らす
• 最高・最低を出すだけならソートは不要
• 拡張for文を使えるところは使う

public class Main {
    public static void main(String[] args) {
//        args = new String[]{ "71", "85", "93", "83", "63", "31", "67", "77", "81", "87", "89", "55" }; // テスト用

        // 早期リターンすれば、以降のネストが減って見やすい
        if (args.length != 12) {
            System.out.println("エラー! コマンドライン引数が" + args.length + "個です。");
            return;
        }

        System.out.println("コマンドライン引数は" + args.length + "個です。");

        int[][] test = new int[4][3];

//        for (int i = 0; i < 3; i++) {
//            test[0][i] = Integer.parseInt(args[0 + i]);
//            test[1][i] = Integer.parseInt(args[3 + i]);
//            test[2][i] = Integer.parseInt(args[6 + i]);
//            test[3][i] = Integer.parseInt(args[9 + i]);
//        }

        // 二重forで入れたいならこうだが、１行しか減ってないし↑でも良い気もする
        for (int j = 0; j < 4; j++) {
            for (int i = 0; i < 3; i++) {
                test[j][i] = Integer.parseInt(args[i + j * 3]);
            }
        }

        // 短くするならこうだが、（定型パターンとはいえ）何をやっているかわかりにくい
//        for (int i = 0; i < 12; i++) {
//            test[i / 3][i % 3] = Integer.parseInt(args[i]);
//        }


        // testは配列の中に配列が入っているので、test[0]とするとint[3]が取り出せる
        print(test[0], "英語");
        print(test[1], "数学");
        print(test[2], "国語");
        print(test[3], "理科");
    }


    // 同じような処理になった場合は、メソッドに切り出せないか検討する
    private static void print(int[] source, String subject) {
        // 最高点・最低点を出すだけならソートする必要もない
        int min = Integer.MAX_VALUE;
        int max = Integer.MIN_VALUE;
        int sum = 0;

        System.out.print(subject + "：");
        // 拡張for文のほうがすっきり見える
        for (int value : source) {
            System.out.print(value + "\t");

            if (value < min) min = value;
            if (value > max) max = value;
            sum += value;
        }
        System.out.println("最高点：" + max + "、最低点：" + min + "、平均点：" + (double) sum / source.length);
    }
}

この後個人毎の集計が入ったりすると、また悩ましいところですが^^;

Answer 2

短くすればいいんですよね。

public class a {
    public static void main(String[] args) {
        int n = args.length;
        if (n != 12)
            System.out.println("エラー! コマンドライン引数が" + n +"個です。");
		else {
			System.out.println("コマンドライン引数は" + n + "個です。");
			String[] subject = { "英語", "数学", "国語", "理科" };
			int k = 0;
			for (int i = 0; i < 4; i++) {
				System.out.print(subject[i] + "：");
				int max = 0, min = 100, sum = 0;
				for (int j = 0; j < 3; j++) {
					int v = Integer.parseInt(args[k++]);
					if (v > max) max = v;
					if (v < min) min = v;
					sum += v;
					System.out.printf("%6d", v);
				}
				System.out.println("    最高点：" + max + "、最低点：" + min
					+ "、平均点：" + sum / 3.0);
			}
		}
	}
}

二次元配列は不要ですが、どうしても二次元配列に値を入れたかったら、
int[][] test = new int[4][3]; を追加し、
for (int j = 0; ... の for文の中で、test[i][j] = v; とすればよいでしょう。