Collections.sortによる点数のソート

Question

###目的
Comparableインタフェースを用いて，Collections.sortメソッドでソート。
また、これをComparatorインタフェースで実装。

###問題点
int型で起動できないためInteger型にして比べましたが、データの順序が逆になるだけで点数順で上手くソート出来ません。
また、Collectios.sortでのソートが出来ないので、Comparatorインタフェースでのソートも出来ていません。

###ソースコード

import java.util.ArrayList;
import java.util.Collections;

public class SortScoreComparable {
	public static void main(String[] args) {
	    ArrayList<Score> score = new ArrayList<Score>();

	    score.add(new Score("taro", 12, 97));
	    score.add(new Score("jiro", 42, 54));
	    score.add(new Score("sabu", 42, 47));
	    score.add(new Score("siro", 57, 97));
	    score.add(new Score("goro", 87, 40));
	    score.add(new Score("roku", 99, 99));
	    score.add(new Score("nana", 14, 23));
	    score.add(new Score("hati", 42, 54));

	    System.out.println("ソート前");
	    for (Score s : score) {
	      System.out.println(s.getName() + " : Math: " + s.getMath() + " : English : " + s.getEnglish());
	    }

	    Collections.sort(score);

	    System.out.println("ソート後");
	    for (Score s : score) {
	      System.out.println(s.getName() + " : Math: " + s.getMath() + " : English : " + s.getEnglish());
	    }
	  }
	}
	class Score implements Comparable<Score> {
	  private String name;
	  private int math;
	  private int english;

	  public Score(String name, int math, int english) {
	    this.name = name;
	    this.math = math;
	    this.english = english;
	  }
	  public String getName() {
	    return name;
	  }
	  public int getMath() {
	    return math;
	  }
	  public int getEnglish() {
	    return english;
	  }
	  
	  public int compareTo(Score another) {
		Integer Math = new Integer(math);
		Integer i = new Integer(another.getMath());
		if (Math.compareTo(i) < 0) {
			//自分が相手よりも低い
			return -1;
		} else if (Math.compareTo(i) > 0){
			//自分が相手よりも高い
			return 1;
		} else {
			//それ以外＝同一順位
			return 0;
		}
	   }
	}

###備考
順序付けは，数学の成績が良い順とし，数学の成績が同点の場合は，英語の成績が良い順とし，それでも順序が付かない場合は同位とする。
多いかとは思いますが、どなたかお願いします。

Comments

[kei344]

質問タグが「JavaScript」になっていますので、「JAVA」に付け替えてください。

Answer 1

Javaの標準APIは、
しっかりと整備された日本語のドキュメントがあるので、（すごいありがたい）
不便だなと思うことがあればちゃんと調べてみること。（だいたいある）
あと、できれば命名規約についても調べて？Math変数はさすがにちょっと・・・

public int compareTo(Score another) {
    int result = Integer.compare(this.math, another.math);
    return (result == 0 ? Integer.compare(this.english, another.english) : result);
}

Collections#sortで引数がList１つの場合は、
Comparable#compareToが正しく実装されている限り昇順ソートなので、
降順にしたければソート後Collections#reverseを使って結果をひっくり返す

余談：
「どっちも同点なら名前五十音順」とか追加したい場合は、
reverseしたときにそれごとひっくり返ってしまうので、
その時はComparator使ってごにょごにょする

Comments

[nabe_u]

回答ありがとうございます。
このコードを追加したところ、数学のソートはすることが出来ました。
ただ、数学が同点だったときの英語のソートが出来ませんでした。
提出の期限なので数学のソートのみで提出しようと思います。
もっと勉強します。

Answer 2

import static java.util.Comparator.*;

import java.util.ArrayList;

public class SortScoreComparable {
	public static void main(String[] args) {
		ArrayList<Score> score = new ArrayList<Score>();

        score.add(new Score("taro", 12, 97));
        score.add(new Score("jiro", 42, 54));
        score.add(new Score("sabu", 42, 47));
        score.add(new Score("siro", 57, 97));
        score.add(new Score("goro", 87, 40));
        score.add(new Score("roku", 99, 99));
        score.add(new Score("nana", 14, 23));
        score.add(new Score("hati", 42, 54));

		System.out.println("ソート前");
		score.forEach(s -> System.out.printf("%s : Math : %d : English : %d\n", s.getName(), s.getMath(), s.getEnglish()));

		score.sort(comparing(Score::getMath).thenComparing(Score::getEnglish).reversed());

		System.out.println("ソート後");
		score.forEach(s -> System.out.printf("%s : Math : %d : English : %d\n", s.getName(), s.getMath(), s.getEnglish()));
	}
}

class Score {
	private String name;
	private int math;
	private int english;

	public Score(String name, int math, int english) {
		this.name = name;
		this.math = math;
		this.english = english;
	}

	public String getName() {
		return name;
	}

	public int getMath() {
		return math;
	}

	public int getEnglish() {
		return english;
	}
}