java インターフェース 比較

Question

public class Student implements Comparable { //implements -> Comparableインターフェース継承
	private int id; // 学籍番号
	private String name; // 氏名
	private int grade; // 学年

	public Student(int id, int grade, String name) {
		this.id = id;
		this.grade = grade;
		this.name = name;
		}

	public String toString() {
		return id + "," + grade + "," + name;
	}

	@Override
	public int compareTo(Object o) { //object抽象的、曖昧
		Student st = (Student) o;

		// TODO:
		// thisとstを比較し...
		
		// 1.thisの方が学年が下ならば、負の整数(例えば-1)を返す。
		if(this.grade < st.grade ) return -1;
		// 2.thisの方が学年が上ならば、正の整数(例えば1)を返す。
		if(this.grade < st.grade ) return 1;
		// 3.学年が同じでthisの方が学籍番号の数値が小さければ、負の整数を返す。
		if(this.grade == st.grade && this.id < st.id) return -2; 
		// 4.学年が同じでthisの方が学籍番号の数値が大きければ、正の整数を返す。
		if(this.grade == st.grade && this.id > st.id) return 2;
		// 5.それ以外なら 0 を返す。
		return 0;
	}

}

import java.util.Arrays;
public class StudentList {
	 public static void main(String[] args) {
	        // 学生データを準備
	        Student[] list = new Student[] {
	                // 2年生
	                new Student(10190003, 2, "Alice"),
	                new Student(10190004, 2, "Bob"),
	                // 1年生
	                new Student(10200001, 1, "Carol"),
	                new Student(10200002, 1, "Dave"),
	                // 5年生
	                new Student(21200005, 5, "Eve"),
	                new Student(21200006, 5, "Frank")
	        };
	 
	        // list配列の添え字0以上6未満の範囲を並び替える
	        Arrays.sort(list, 0, 6);
	 
	        // 並び替えた結果を順に出力する
	        for (int i = 0; i < list.length; i++) {
	            System.out.println(list[i]);
	        }
	    }
}

student配列を学年の低い学生から学年の高い学生に、同じ学年であれば学籍番号の数値の小さい学生から大きい学生に、並ぶようにしたいです。

エラー文
Exception in thread "main" java.lang.ClassCastException: java.lang.Integer cannot be cast to comptest.Student
at comptest.Student.compareTo(Student.java:29)
at comptest.Student.<init>(Student.java:19)
at comptest.StudentList.main(StudentList.java:15)

実行結果をこうしたい

10200001,1,Carol
10200002,1,Dave
10190003,2,Alice
10190004,2,Bob
21200005,5,Eve
21200006,5,Frank

public String toString()を使うと思うのですが、どこで使うのかもわかりません。
どういうプロセスで書けばよいのでしょうか。

Comparableはあとで追加します。
Comparable対応にする際に必要となるcompareToメソッドを記述しています

Comments

[ozwk]

Studentクラスのコンストラクタ中にあるcompareTo()2つは一体なにを意図して書いたんですか?

[ozwk]

コード書き換えたっぽいですけどまだ問題起きてます?

[junnnnchan]

正常に動くのですが、エラーが消えません。。
エラー文
Testsuite: comptest.AllTests
Tests run: 4, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.027 sec

Testcase: testStudentList took 0.001 sec
Testcase: testSort took 0.008 sec
Testcase: testCompareTo took 0.002 sec
FAILED
1491391,3,aykbを引数に1924546,6,artbmwのcompareToを呼んだとき、compareToの結果が正でない
junit.framework.AssertionFailedError: 1491391,3,aykbを引数に1924546,6,artbmwのcompareToを呼んだとき、compareToの結果が正でない
at comptest.StudentTest.testCompareTo(StudentTest.java:247)

Testcase: testStudent took 0.001 sec

[junnnnchan]

符号間違えてました。。

Answer 1

そもそも、ジェネリクスの入らないComparableを使うのが適当ではありません。

まずは、implements Comparable<Student>としてみましょう。すると、不適当な型で比較しようとしている箇所がコンパイルエラーになってくれます。

Answer 2

とりあえず

public class Student implements Comparable { //implements -> Comparableインターフェース継承
    private int id; // 学籍番号
    private String name; // 氏名
    private int grade; // 学年

    public Student(int id, int grade, String name) {
        this.id = id;
        this.grade = grade;
        this.name = name;
-       compareTo(id); // <--------------- いらない
-       compareTo(grade);    //  <--------------- いらない
    }

    public String toString() {
        return id + "," + grade + "," + name;
    }

    @Override
    public int compareTo(Object o) { //object抽象的、曖昧
        Student st = (Student) o;

        // TODO:
        // thisとstを比較し...
-       this.compareTo(st); //  <--------------- いらない

        // 1.thisの方が学年が下ならば、負の整数(例えば-1)を返す。
        if(this.grade < st.grade ) return -1;
        // 2.thisの方が学年が上ならば、正の整数(例えば1)を返す。
        if(this.grade < st.grade ) return 1;
        // 3.学年が同じでthisの方が学籍番号の数値が小さければ、負の整数を返す。
        if(this.grade == st.grade && this.id < st.id) return -2; 
        // 4.学年が同じでthisの方が学籍番号の数値が大きければ、正の整数を返す。
        if(this.grade == st.grade && this.id > st.id) return 2;
        // 5.それ以外なら 0 を返す。
        return 0;
    }

}

Comments

[gentaro]

「いらない」というコメント書くぐらいなら
```diff
```
で差分提示した方が良いような。