3

pojo クラス内の単一の値で多くの並べ替えの例を取得しました。

しかし、以下の例のように複数の値でソートしたい

ソート前

Name          City           Roll
--------------------------------------

    sekhar ---------bbsr------------23
    himanshu -------agra------------23
    nitin ----------delhi------------23
    nitin ----------delhi------------22
    nitin ----------bbsr------------23
    arun -----------patna------------23
    arun -----------kendrapara-------23

ソート後

Name          City           Roll
--------------------------------------

    arun -------kendrapara------------23
    arun -------patna------------23
    himanshu ----agra------------23
    nitin -------bbsr------------23
    nitin -------delhi------------22
    nitin -------delhi------------23
    sekhar -------bbsr------------23

私のコードはここにあります

package example;

import java.util.ArrayList;
import java.util.Collections;
import java.util.Comparator;
import java.util.List;

public class Demo {

    /**
     * @param args
     */
    public static void main(String[] args) {
        // TODO Auto-generated method stub
        Demo demo = new Demo();
        List<Student> list = new ArrayList<Student>(); // your Car list
        Student st = demo.new Student();
        st.setName("sekhar");
        st.setCity("bbsr");
        st.setRoll(23);
        list.add(st);
        st = demo.new Student();
        st.setName("himanshu");
        st.setCity("agra");
        st.setRoll(23);
        list.add(st);
        st = demo.new Student();
        st.setName("nitin");
        st.setCity("delhi");
        st.setRoll(23);
        list.add(st);
        st = demo.new Student();
        st.setName("nitin");
        st.setCity("delhi");
        st.setRoll(22);
        list.add(st);
        st = demo.new Student();
        st.setName("nitin");
        st.setCity("bbsr");
        st.setRoll(23);
        list.add(st);
        st = demo.new Student();
        st.setName("arun");
        st.setCity("patna");
        st.setRoll(23);
        list.add(st);
        st = demo.new Student();
        st.setName("arun");
        st.setCity("kendrapara");
        st.setRoll(23);
        list.add(st);
        System.out.println("before sort");
        System.out.println("Name:" + "-------------------" + "City...........roll");
        for (Student st1 : list) {
            System.out.println(st1.getName() + " --------------" + st1.getCity() + "------------" + st1.getRoll());
        }

        Collections.sort(list, demo.new CarHorsePowerComparator());
        System.out.println("after sort");
        System.out.println("Name:" + "-------------------" + "City...............roll");
        for (Student st1 : list) {
            System.out.println(st1.getName() + " --------------" + st1.getCity() + "------------" + st1.getRoll());
        }
    }

    public class Student {

        String name;
        String city;
        int roll;

        public int getRoll() {
            return roll;
        }

        public void setRoll(int roll) {
            this.roll = roll;
        }

        public String getCity() {
            return city;
        }

        public void setCity(String city) {
            this.city = city;
        }

        public String getName() {
            return name;
        }

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

    class CarHorsePowerComparator implements Comparator<Student> {
        public int compare(Student stu, Student stu1) {
            int sort = (stu.getName() + stu.getCity() + stu.getRoll()).compareTo((stu1.getName() + stu1.getCity() + stu1.getRoll()));
            return sort;
        }
    }
}

しかし、私はカスタムコンパレータを使用しています

class CarHorsePowerComparator implements Comparator<Student> {
            public int compare(Student stu, Student stu1) {
                int sort = (stu.getName() + stu.getCity() + stu.getRoll()).compareTo((stu1.getName() + stu1.getCity() + stu1.getRoll()));
                return sort;
            }

それは私のために働いていますが、それは正しい方法ではありません。このコードを正しい方法で書く方法を教えてください。

int sort = (stu.getName() + stu.getCity() + stu.getRoll()).compareTo((stu1.getName() + stu1.getCity() + stu1.getRoll()));
4

3 に答える 3

3 
(stu.getName() + stu.getCity() + stu.getRoll()).compareTo((stu1.getName() + stu1.getCity() + stu1.getRoll()))

Name、City、Roll 番号を連結し、結果の文字列をソートしていることを意味します。

最初に並べ替えの優先順位を決定し、その順序でフィールドを並べ替える必要があります。期待される結果を見ると、最初に名前で、次に都市で、最後にロールでソートしたいようです。

class CarHorsePowerComparator implements Comparator<Student> {
            public int compare(Student stu, Student stu1) {
                int sort;
                // Sort on Name first 
                sort = stu.getName().compareTo(stu1.getName);
                if(sort == 0){
                  // If name is equal, sort on city
                  sort = stu.getCity().compareTo(stu1.getCity);
                }   
                if(sort == 0){
                  // If name and city is equal, sort on Roll
                  sort = Integer.compare(stu.getRoll(),stu1.getRoll());
                } 
                return sort;
            }
}
于 2013-10-18T09:21:28.197 に答える
0

プラス演算子を使用しているときは、新しい String オブジェクトをたくさん作成します。文字列は、ロジック目的ではないデータを格納するために使用する必要があります。

エラーが発生しやすいため、パラメータ名をstuおよびとして使用しないでください。andまたはand 'right'stu1を使用できます。meotherleft

再利用するので、クラス宣言は必要ありません。匿名でもいいです

private final static Comparator<Student> STUDENT_NAME_CITY_ROLL_COMPARATOR = new Comparator<Student> {

  private final Collator collator = Collator.getInstance(); //Default local is taken. 

  public int compare(Student left, Student right) {

    int result = collator.compare(left.getName(),right.getName()); 

    if(result == 0) {
        result = collator.compare(left.getCity(),right.getCity());
    } 

    if(result == 0) {
        result = Integer.compare(left.getRoll(), right.getRoll()); //JDK 7
       //result = Double.compare(left.getRoll(), right.getRoll());
    }

    return result;
}

注: 一部の要素が null の場合、実装は NPE をスローします。

コレーター

Integer.compare(int,int)

于 2013-10-18T09:43:39.717 に答える