テキストファイルからArrayListに内容を取り込み、その情報をソートして表示させたいです

Question

前提

Javaの質問です。

Javaでプログラミングを勉強し始めて3週間と少しのド初心者です。クラスやオブジェクト指向の概念など基本的なことは一通り学んだつもりですが少しでも応用的な内容になるとかなり苦戦してしまいます。
恐縮ですがご助力いただけますと幸いです。

名簿としてID番号、名前、生年月日、性別を記載したテキストファイル(プロジェクト直下に作成)をArrayListに取り込み、ソートしてコンソールに一覧表示するプログラムを書いています(未完)。
ArrayListへのテキストファイルの取り込みと、ソートしないファイルそのままの状態での一覧表示までは試行錯誤の末なんとかできるようになりました。
その次の段階でID、名前、生年月日などでソートし名簿を昇順や降順で表示をさせたいのですが、コンパイルエラーの出ないプログラムがどうしてもうまく書けません。

実現したいこと

・テキストファイルをArrayListへ取り込み、その内容をID、名前、生年月日などでソートしコンソールへ一覧表示させたいです。
・どのようにコードを書き換えれば正常に動作するようになるか知りたいです。

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

Exception in thread "main" java.lang.Error: Unresolved compilation problem: 
	型 MemberComparator は継承された抽象メソッド Comparator.compare(Object, Object) を実装する必要があります

該当のソースコード

import java.io.BufferedReader;
import java.io.File;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.IOException;
import java.util.ArrayList;
import java.util.Collections;

public class SortTest {

	public ArrayList<SortBean> sortbeanArray = new ArrayList<SortBean>();

	public static void main(String[] args) {
		SortTest st = new SortTest();
		st.read();
		st.show();
	}

	public void read() {//ファイルを読み込み、ArrayListの中に入れる

		try {
			File file = new File("meibo.txt");

			if (file.exists()) {
				if (file.isFile() && file.canRead()) {
					BufferedReader br = new BufferedReader(new FileReader(file));

					String str;
					sortbeanArray.clear();
					while ((str = br.readLine()) != null) {
						String[] arr = str.split(",");//カンマで区切って読み込む
						SortBean bean = new SortBean();
						bean.setID(arr[0]);
						bean.setName(arr[1]);
						bean.setBirthDay(arr[2]);
						bean.setGender(arr[3]);
						sortbeanArray.add(bean);
					}
					br.close();
					System.out.println("読み込み完了");//成功
				} else {
					System.out.println("読み込み不可");//ファイルはあるが読み込めない
				}
			} else {
				System.out.println("ファイルが見つからない");//ファイルがない
			}

		} catch (FileNotFoundException e) {
			System.out.println(e);
		} catch (IOException e) {
			System.out.println(e);
		}
	}

	public void show() {//ArrayListの中に入れたものをソートした上で一覧表示したい

		System.out.println("ソートせずに一覧表示");
		for (int i = 0; i < sortbeanArray.size(); i++) {

			System.out.println(sortbeanArray.get(i).getID()
					+ "," + sortbeanArray.get(i).getName()
					+ "," + sortbeanArray.get(i).getBirthDay()
					+ "," + sortbeanArray.get(i).getGender());
		}
		//ここまで、ソートしないままの一覧表示は可能だがここから先がうまくできない
		System.out.println("IDでソートし一覧表示");

		for (int i = 0; i < sortbeanArray.size(); i++) {

			Collections.sort(sortbeanArray, new MemberComparator());

			System.out.println(sortbeanArray.get(i).getID()
					+ "," + sortbeanArray.get(i).getName()
					+ "," + sortbeanArray.get(i).getBirthDay()
					+ "," + sortbeanArray.get(i).getGender());
		}

	}
}
//--------------------ここから別ファイルにクラスを分けています-------------------------
import java.util.Comparator;

public class MemberComparator implements Comparator {

	public int compare(SortBean sb1, SortBean sb2) {

		if (sb1.getID() > sb2.getID()) {

			return -1;
		} else if (sb1.getID() > sb2.getID()) {
			return 1;
		} else {
			return 0;
		}
	}

}
//----------------------ここから別ファイルにクラスを分けています-----------------------
public class SortBean {

	private String id;
	private String name;
	private String birthday;
	private String gender;

	public SortBean() {
	}

	public String getID() {
		return id;
	}

	public void setID(String id) {
		this.id = id;
	}

	public String getName() {
		return name;
	}

	public void setName(String name) {
		this.name = name;
	}

	public String getBirthDay() {
		return birthday;
	}

	public void setBirthDay(String birthday) {
		this.birthday = birthday;
	}

	public String getGender() {
		return gender;
	}

	public void setGender(String gender) {
		this.gender = gender;
	}

}
//-------サンプルとしてこの下にテキストファイル"meibo.txt"の内容を入れています(ID、名前、生年月日、性別)---------
//これらの内容をソートして表示させたい
/*

57,田中,1999/11/11,Male
25,山田,1970/03/12,Female
33,伊藤,1989/05/01,Female
49,斎藤,2000/01/12,Male
50,杉山,2010/05/05,Male
63,河井,1996/01/30,Female
5,松田,1989/03/21,Male

*/

試したこと

自分で調べCollectionsとComparatorを使用しソートを行う方法にたどり着き(投稿したコードにそのうちの一つを自分が試しているものに書いています)その方法を使いましたがダメでした。
ほかにも書き方の異なる類似の方法を試しましたが同様にうまくできませんでした。

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

Java17を使用しています。

Comments

[jimbe]

コードのマークダウンは、ファイル毎に分けてください。

[YT0014]

Comparatorの正しい仕様を確認されることをお勧めします。
https://docs.oracle.com/javase/jp/8/docs/api/java/util/Comparator.html

Answer 1

型 MemberComparator は継承された抽象メソッド Comparator.compare(Object, Object) を実装する必要があります

型パラメータTが必要です。
Comparator (Java Platform SE 8 )

if (sb1.getID() > sb2.getID()) {

文字列を不等号で比較することはできません。
数値として比較したいのでしょうから、ID自体を数値にするか比較時に数値するなりしてください。

Java17を使用しています。

せっかくJava17なんでしたら、もうちょっとモダンな書き方でいきましょう^^
歴史的経緯をスルーして使いこなす Java ファイル入出力処理 - Qiita

StreamAPIはちょっと難しいと思ったら、Files.readAllLinesなんかはお手軽です（大きいファイルには向きません）
Files (Java Platform SE 8 )

import java.io.IOException;
import java.nio.file.*;
import java.util.Comparator;
import java.util.List;
import java.util.stream.Collectors;

public class SortTest {
    public static void main(String[] args) throws IOException {
        var st = new SortTest();

        var path = Paths.get("meibo.txt");
//        st.dummy(path);

        st.read(path);
        st.show();
    }

    List<SortBean> sortbeans;

    void read(Path path) {
        if (!Files.exists(path)) {
            System.out.println("ファイルが見つからない");
        } else if (!Files.isReadable(path)) {
            System.out.println("読み込み不可");
        } else {
            try (var stream = Files.lines(path)) {
                sortbeans = stream
                    .map(line -> {
                        var array = line.split(",");
                        var bean = new SortBean();
                        bean.setID(array[0]);
                        bean.setName(array[1]);
                        bean.setBirthDay(array[2]);
                        bean.setGender(array[3]);
                        return bean;
                    })
                    .collect(Collectors.toList());

                // せっかくstream.toList()が追加されたのだから使いたいところだが変更不可
                // sortで落ちる（UnsupportedOperationException）
                // [JDK 16 : stream.toList() に見るAPI設計の難しさ - A Memorandum](https://blog1.mammb.com/entry/2021/06/07/000000)
//                    .toList();

            } catch (IOException e) {
                e.printStackTrace();
            }
        }
    }

    void show() {
        System.out.println("ソートせずに一覧表示");
        sortbeans.forEach(System.out::println);

        System.out.println("\nIDでソートし一覧表示");
        sortbeans.sort(new MemberComparator());
        // stream.toList() で変更不可のリストでも、sortedで作り直すことはもちろん可能
//        sortbeans = sortbeans.stream().sorted(new MemberComparator()).toList();
        sortbeans.forEach(System.out::println);

        System.out.println("\n名前でソートし一覧表示");
        // ↑の省略形 匿名クラス
        sortbeans.sort(new Comparator<SortBean>() {
            @Override public int compare(SortBean o1, SortBean o2) {
                return o1.getName().compareTo(o2.getName());
            }
        });
        sortbeans.forEach(System.out::println);

        System.out.println("\n誕生日でソートし一覧表示");
        // ↑の省略形 ラムダ式
        sortbeans.sort((o1, o2) -> o1.getBirthDay().compareTo(o2.getBirthDay()));
        sortbeans.forEach(System.out::println);

        System.out.println("\n性別でソートし一覧表示");
        // ↑の省略形 Comparator.comparing
        sortbeans.sort(Comparator.comparing(SortBean::getGender));
        sortbeans.forEach(System.out::println);
    }

    void dummy(Path path) throws IOException {
        var csv = """
            57,田中,1999/11/11,Male
            25,山田,1970/03/12,Female
            33,伊藤,1989/05/01,Female
            49,斎藤,2000/01/12,Male
            50,杉山,2010/05/05,Male
            63,河井,1996/01/30,Female
            5,松田,1989/03/21,Male""";
        Files.writeString(path, csv);
    }
}

class MemberComparator implements Comparator<SortBean> {
    public int compare(SortBean sb1, SortBean sb2) {
        return Integer.valueOf(sb1.getID()).compareTo(Integer.valueOf(sb2.getID()));
    }
}

class SortBean {
    private String id;
    private String name;
    private String birthday;
    private String gender;

    public String getID() { return id; }

    public void setID(String id) { this.id = id; }

    public String getName() { return name; }

    public void setName(String name) { this.name = name; }

    public String getBirthDay() { return birthday; }

    public void setBirthDay(String birthday) { this.birthday = birthday; }

    public String getGender() { return gender; }

    public void setGender(String gender) { this.gender = gender; }

    @Override public String toString() {
        return String.join(",", id, name, birthday, gender);
    }
}

Comments

[chahahaan]

回答ありがとうございます。
丁寧にコードまで書いていただいて本当にありがたいです。
参考URLまで張っていただき恐縮です。
文字列は不等号で比較できない点、言われてみれば確かにそうですが完全に見落としていました。
自分ではまだ知らない記述の仕方がたくさんあるのでひとつずつ調べるなりデバッグで参考にさせていただきます。

[TN8001]

ソートの部分は↓で解決できると思いますがどうでしょうか？

> 型パラメータTが必要です。
> 文字列を不等号で比較することはできません。

> 自分ではまだ知らない記述の仕方がたくさんあるのでひとつずつ調べるなりデバッグで参考にさせていただきます。

調べてみて解決できたのなら「解決済」にしてください。
[ヘルプ｜質問を解決済みにしたい](https://teratail.com/help#resolve-question)

回答コードについての疑問点や不明点は、お気軽にコメントしてください。

回答コードがモダンすぎるということであれば、Java8以前程度のコード（ｇｇってよく出てくるような）に書き直しても構いませんが...

Answer 2

for (int i = 0; i < sortbeanArray.size(); i++) {
   Collections.sort(sortbeanArray, new MemberComparator());

Collections.sort は、 comparator によって sortBeanArray を並び替えます。
従ってループの中では無く外で行うことです。

Collections.sort(sortbeanArray, new MemberComparator());
for (int i = 0; i < sortbeanArray.size(); i++) {

---

勉強用なら、 TN8001 さんと違った方向のコードにしてみます。

import java.io.*;
import java.time.LocalDate;
import java.time.format.DateTimeFormatter;
import java.util.*;

public class SortTest {
  public static void main(String[] args) {
    try {
      new SortTest()
          .readFrom("meibo.txt") //ファイル名は埋め込むよりパラメータで渡す
          .show("■ ソートせずに一覧表示")
          .sort((a,b) -> a.id < b.id ? -1 : a.id > b.id ? 1 : 0) //ラムダ式 & 三項演算子
          .show("■ IDでソートし一覧表示");
    } catch(FileNotFoundException e) {
      System.out.println("ファイルが見つかりません."+e.getMessage());
    } catch(IOException e) {
      System.out.println("ファイルの読み込みで問題が発生しました. "+e.getMessage());
    }
  }

  private List<Person> persons = new ArrayList<>();

  SortTest readFrom(String filename) throws FileNotFoundException, IOException { //throws で例外時の処理は呼び出し側に任せる
    persons.clear();
    Person.Builder builder = new Person.Builder();
    try(BufferedReader reader = new BufferedReader(new FileReader(filename))) { //try-with-resource ( close が自動的 )
      for(String line; (line=reader.readLine()) != null; ) { //'readLine' の結果を受ける変数名は 'line' のほうが合ってる感じ. そして変数の有効範囲(スコープ)は狭く
        String[] tokens = line.split(",");
        try {
          if(tokens.length == 4) {
            persons.add(builder
                .setId(tokens[0])
                .setName(tokens[1])
                .setBirthday(tokens[2])
                .setGender(tokens[3])
                .build());
          } else {
            throw new IllegalStateException("データが4つでは無い. tokens.length="+tokens.length);
          }
        } catch(Exception e) {
          System.out.println("format error. line="+line);
          e.printStackTrace(); //例外の詳細の表示はコレ
        }
      }
    }
    System.out.println("読み込み完了");
    return this;
  }

  SortTest sort(Comparator<Person> comparator) {
    Collections.sort(persons, comparator);
    return this;
  }

  SortTest show(String header) {
    System.out.println(header);
    for(Person person : persons) System.out.println(""+person);
    return this;
  }

  private static class Person {
    enum Gender { Male, Female } //少ない選択肢なら enum

    //birthday の解析/表示用
    private static final DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy/MM/dd");

    static class Builder { //Person の生成はこのクラスが行う
      private int id;
      private String name;
      private LocalDate birthday;
      private Gender gender;

      Builder setId(String id) {
        this.id = Integer.parseInt(id);
        return this; //自身を返すことで、呼び出し側が '.' で複数のメソッドを繋げて書けるようになる
      }
      Builder setName(String name) {
        if(name == null) throw new NullPointerException();
        this.name = name;
        return this;
      }
      Builder setBirthday(String  birthday) {
        this.birthday = LocalDate.parse(birthday, formatter);
        return this;
      }
      Builder setGender(String gender) {
        this.gender = Gender.valueOf(gender);
        return this;
      }
      Person build() {
        return new Person(id, name, birthday, gender);
      }
    }

    final int id; //final にすると変更出来なくなる
    final String name;
    final LocalDate birthday;
    final Gender gender;

    private Person(int id, String name, LocalDate birthday, Gender gender) {
      this.id = id;
      this.name = name;
      this.birthday = birthday;
      this.gender = gender;
    }

    @Override
    public String toString() { //表示用の文字列を自分で生成する
      return String.join(",",""+id,name,birthday.format(formatter),""+gender);
    }
  }
}

meibo.txt

57,田中,1999/11/11,Male
25,山田,1970/03/12,Female
33,伊藤,1989/05/01,Female
49,斎藤,2000/01/12,Male
50,杉山,2010/05/05,Male
63,河井,1996/01/30,Female
5,松田,1989/03/21,Male

実行結果

読み込み完了
■ ソートせずに一覧表示
57,田中,1999/11/11,Male
25,山田,1970/03/12,Female
33,伊藤,1989/05/01,Female
49,斎藤,2000/01/12,Male
50,杉山,2010/05/05,Male
63,河井,1996/01/30,Female
5,松田,1989/03/21,Male
■ IDでソートし一覧表示
5,松田,1989/03/21,Male
25,山田,1970/03/12,Female
33,伊藤,1989/05/01,Female
49,斎藤,2000/01/12,Male
50,杉山,2010/05/05,Male
57,田中,1999/11/11,Male
63,河井,1996/01/30,Female

Comments

[chahahaan]

コメントありがとうございます。
確かにループの中にCollections.sortを置くのはおかしいですね。
参考にさせていただきます！