データから最年長を表示したい

Question

実現したいこと

指定したデータから最年長を出力するプログラムを作成したい

前提

・生年月日を考慮した出力結果にしたい
・CSV形式で出力する

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

エラーメッセージ

Java

class Quiz8 {
    public static void main(String[] args){
	String[][] senbatsu = {
	    {"Name","BloodType","Year","Month","Day","HomeTown"},
	    {"A","O","1992","11","21","Oita"},
	    {"B","B","1991","7","15","Kagoshima"},
	    {"C","AB","1994","3","26","Saitama"},
	    {"D","AB","1991","4","8","Tokyo"},
	    {"E","B","1997","3","8","Aichi"},
	    {"F","B","1993","7","14","Osaka"},
	    {"G","A","1998","3","19","Kagoshima"},
	    {"H","O","1990","8","13","Tokyo"},
	    {"I","A","1994","3","30","Saitama"},
	    {"J","B","1992","12","8","Kyoto"},
	    {"K","A","1991","6","24","Aichi"},
	    {"L","B","1993","9","19","Nara"},
	    {"M","O","1990","1","17","Saitama"},
	    {"N","B","1991","11","29","Aichi"},
	    {"O","B","1993","4","1","Aichi"},
	    {"P","B","1994","11","25","Tokyo"}
	};
	int i,j;
	int maxy = Integer.parseInt(senbatsu[1][2]);//1列目
	int maxm = Integer.parseInt(senbatsu[1][3]);//2列目
	int maxd = Integer.parseInt(senbatsu[1][4]);//3列目
	int maxi = 1;
	int y,m,d;
	for(i=2; i<senbatsu.length; i++){//1行目がタイトル行のため、2行からカウントしていく
	    y = Integer.parseInt(senbatsu[i][2]);//年の行
	    m = Integer.parseInt(senbatsu[i][3]);//月の行
	    d = Integer.parseInt(senbatsu[i][4]);//日の行
    int maxY = Integer.parseInt(senbatsu[maxi][2]);//年を比較した時、大きい値を記録
    int maxM = Integer.parseInt(senbatsu[maxi][3]);//月を比較した時、大きい値を記録
    int maxD = Integer.parseInt(senbatsu[maxi][4]);//日を比較した時、大きい値を記録
    
    if (y<maxY || (y==maxY && m<maxM) || (y==maxY && m==maxM && d<maxD)) {
        maxi = i;
     //指定した年数<年の最大値　かつ　（指定した年＝最大値　かつ　指定した月＜月の最大値）　かつ　（指定した年＝最大値　または　指定した月＝最大値　または指定した月＜最大値））
     //各最大値は、maxiに記録
    }
 }
	System.out.print(senbatsu[maxi][0]);
	for(j=1; j<senbatsu[maxi].length; j++){
	    System.out.print( ", "+senbatsu[maxi][j] );
	}
	System.out.println("");
    }
}

試したこと

みな様から頂いたアドバイスをもとに、プログラムを修正しました。
ありがとうございます！

上記プログラムで、一応、正しく実行できる様子です。
しかし、私自信が、正しく構文を理解できているのか自信がありません。

各行ごとにコメントで行の意味を記載しました。
間違って理解している箇所がありましたら、ご指摘いただけないでしょうか？
よろしくお願いいたします。

補足情報（FW/ツールのバージョンなど）

上記プログラムを実行した場合、
実行結果が↓のになるように作成したいと考えています。

M, O, 1990, 1, 17, Saitama

Comments

[jimbe]

>表計算（集計）機能
とは？

if で使われている変数が何なのか、定義もありません。
if で何と何と比較しようというのでしょうか。
if 比較した結果、何の変数にどんな値を設定するつもりなのでしょう。

[xebme]

y,m,dに分けると判定が難しいですね
int ymd = y*10000 + m*100 + d なら簡単になりませんか

[xebme]

if ブロックの処理わからないのなら
maxy,maxm,maxdはそのときの最小の生年月日　（更新が必要）
最後にmaxiを得たい　（更新が必要）

Answer 1

maxiはその時点の最年長の行を記録しておく変数です。senbatsu配列を1行ずつ見ていって、今見ている行の生年月日とmaxiに記録されている行の生年月日を比較します。このとき今見ている行の生年月日のほうが古い場合は、今見ている行が現時点での最年長ですから、maxi変数を今見ている行番号で更新します。senbatsu配列のすべての行を見終わった際には、maxi変数にはsenbatsu配列の最年長の行が記録されているので、これをコンソールに表示すればよいです。

生年月日の比較方法ですが、java.util.Dateやjava.util.CalendarあるいはDate and Time APIなど、日時を扱う標準ライブラリを使うことが一般的ですが、それらを利用しない方法もあります。以下はその例です。

int maxi = 1;
for (int i = 2; i < senbatsu.length; i++) {
    int y = Integer.parseInt(senbatsu[i][2]);
    int m = Integer.parseInt(senbatsu[i][3]);
    int d = Integer.parseInt(senbatsu[i][4]);
    int maxY = Integer.parseInt(senbatsu[maxi][2]);
    int maxM = Integer.parseInt(senbatsu[maxi][3]);
    int maxD = Integer.parseInt(senbatsu[maxi][4]);
    if (y<maxY || (y==maxY && m<maxM) || (y==maxY && m==maxM && d<maxD)) {
        maxi = i;
    }
}

ちなみに現代的なJavaでは、同じことを以下のように実装することもできます。

Arrays.stream(senbatsu)
      .skip(1)
      .min(Comparator.comparing(row -> LocalDate.of(Integer.parseInt(row[2]), Integer.parseInt(row[3]), Integer.parseInt(row[4]))))
      .ifPresent(row -> System.out.println(String.join(",", row)));

Comments

[JAVA-B1]

アドバイスいただいた内容で、
無事、解決することができました。
ありがとうございます。

Answer 2

考え方だけ書いてみます。

まずは年(maxyとy)を比較した場合、下記3パターンが考えられます。
（1）maxy < y の場合
（2）maxy = y の場合
（3）maxy > y の場合

（1）に合致する場合、今のループで見ているデータが最年長であることはあり得ません。maxy、maxm、maxdはそのままにしておき、次のデータと比較しましょう。
（2）に合致する場合、次は月(maxmとm)を比較する必要があります。
（3）に合致する場合、今のループで見ているデータが暫定で最年長です。maxy、maxm、maxdを更新して、次のデータと比較しましょう。

月と日の比較の仕方についても、同様に考えてみてください。

最年長のデータを探すということは、すべてのデータを見る必要があります。(データ数分forループを回す必要があり、途中でbreakしません)
「暫定で最年長のデータは配列の何番目なのか」の情報も持っていないと最後に出力できないのでご注意ください。

※ちなみに、特に理由がないのであればDate型に変換してcompareTo で比較した方が楽です。

Answer 3

これでいい?

class test {
    public static void main(String[] args){
        String[][] senbatsu = new String[][] {
            new String[] {"Name","BloodType","Year","Month","Day","HomeTown"},
            new String[] {"A",  "O",  "1992", "11", "21", "Oita"     },
            new String[] {"B",  "B",  "1991",  "7", "15", "Kagoshima"},
            new String[] {"C", "AB",  "1994",  "3", "26", "Saitama"  },
            new String[] {"D", "AB",  "1991",  "4",  "8", "Tokyo"    },
            new String[] {"E",  "B",  "1997",  "3",  "8", "Aichi"    },
            new String[] {"F",  "B",  "1993",  "7", "14", "Osaka"    },
            new String[] {"G",  "A",  "1998",  "3", "19", "Kagoshima"},
            new String[] {"H",  "O",  "1990",  "8", "13", "Tokyo"    },
            new String[] {"I",  "A",  "1994",  "3", "30", "Saitama"  },
            new String[] {"J",  "B",  "1992", "12",  "8", "Kyoto"    },
            new String[] {"K",  "A",  "1991",  "6", "24", "Aichi"    },
            new String[] {"L",  "B",  "1993",  "9", "19", "Nara"     },
            new String[] {"M",  "O",  "1990",  "1", "17", "Saitama"  },
            new String[] {"N",  "B",  "1991", "11", "29", "Aichi"    },
            new String[] {"O",  "B",  "1993",  "4",  "1", "Aichi"    },
            new String[] {"P",  "B",  "1994", "11", "25", "Tokyo"    }
        };

        java.util.Calendar cal = java.util.Calendar.getInstance();
        java.util.Date maxDate = null;
        int maxi = 1;
        for ( int i = 1; i < senbatsu.length; ++i ) {
          cal.set(
            Integer.parseInt(senbatsu[i][2]),
            Integer.parseInt(senbatsu[i][3]),
            Integer.parseInt(senbatsu[i][4]),
            0, 0 );
          java.util.Date date = cal.getTime();
          if ( i == 1 || maxDate.before(date) ) {
              maxDate = date;
              maxi = i;
            }
        }

        for( int i = 0; i < senbatsu[maxi].length; ++i ) {
          System.out.print( (i == 0 ? "":", ") + senbatsu[maxi][i] );
        }
        System.out.println();
    }
}