質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.59%

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

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 4,024

score 15

目的

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;
        }
       }
    }

備考

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • kei344

    2017/09/27 15:30

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

    キャンセル

回答 2

checkベストアンサー

0

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で引数がList1つの場合は、
Comparable#compareToが正しく実装されている限り昇順ソートなので、
降順にしたければソート後Collections#reverseを使って結果をひっくり返す

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/28 10:09

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

    キャンセル

0

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;
    }
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.59%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る