12

以下は私の学生クラスです

class Student implements Comparable {
   String name;
   int rollNo;

   @Override
   public int compareTo(Object obj) {
        return ((Student)obj).name.compareTo(this.name);
   }
}

最新の変更: それでも正しい結果が得られない

@Override
public int compareTo(Object obj) {
    Student s = (Student) obj;
    if (name.equals(s.name)) { // achieving uniqueness
        return 0;
    } else {
        if (rollNo < s.rollNo) {
            return -1;
        } else if (rollNo > s.rollNo) {
            return 1;
        } else {
            // this makes `name` the second ordering option.
            // names don't equal here
            return name.compareTo(s.name);
        }
    }
}

TreeSet<Student> のオブジェクトを作成すると、一意の名前に基づいて並べ替えられた Student オブジェクトのリストが取得され、名前順に並べ替えられます。

しかし、私の TreeSet<Student> には、student-rollNo 順で一意の学生名が必要です。

コンパレータで可能ですか？誰でも私を助けることができます, すべての提案は大歓迎です. ありがとう。

更新: 完全なプログラムは次のとおりです。

public class Student implements Comparable {

    int rollNo;
    String name;

    Student(String n,int rno) {
        rollNo=rno;
        name=n;
    }

    /**
     * @param args
     */
    public static void main(String[] args) {

        TreeSet<Student> ts = new TreeSet<Student>();
        ts.add(new Student("bbb",2));
        ts.add(new Student("aaa",4));
        ts.add(new Student("bbb",2));
        ts.add(new Student("ccc",3));
        ts.add(new Student("aaa",1));
        ts.add(new Student("bbb",2));
        ts.add(new Student("bbb",5));

        System.out.println(ts);

    }

    @Override
    public int compareTo(Object obj) {
        Student s = (Student) obj;
        if (name.equals(s.name)) { // achieving uniqueness
            return 0;
        } else {
            if (rollNo < s.rollNo) {
                return -1;
            } else if (rollNo > s.rollNo) {
                return 1;
            } else {
                // this makes `name` the second ordering option.
                // names don't equal here
                return name.compareTo(s.name);
            }
        }
    }

    @Override
    public String toString() {
        return name + rollNo;
    }
}

更新:2: ご提案いただきありがとうございます。まだいくつか必要です:)



/*
 * Actual scenario is having different properties,
 * So here I am just relating my actual scenario with Student class
 */
class Student implements Comparable {
    // sorting required on rollNo
    int rollNo;
    // Unique name is required
    String name;

    Student(String n, int rno) {
        rollNo = rno;
        name = n;
    }

    /**
     * 
     * @param args
     */
    public static void main(String[] args) {

        TreeSet<Student> tsName = new TreeSet<Student>();
        // here by default, order & uniqueness by name only
        tsName.add(new Student("ccc", 2));
        tsName.add(new Student("aaa", 4));
        tsName.add(new Student("ddd", 1));
        tsName.add(new Student("bbb", 3));
        tsName.add(new Student("ddd", 5));
        // output: aaa:4, bbb:3, ccc:2, ddd:1
        System.out.println(tsName);

        // creating new comparator for student RollNo
        TreeSet<Student> tsRollNo = new TreeSet<Student>(new Comparator<Student>() {
                    public int compare(Student stud1, Student stud2) {
                        return new Integer(stud1.rollNo).compareTo(stud2.rollNo);
                    }
                });
        tsRollNo.addAll(tsName);
        System.out.println(tsRollNo);
        // now got the desire output: ddd:1, ccc:2, bbb:3, aaa:4
    }

    public boolean equals(Object obj) {
        // internally not used to check equality while adding objects
        // in TreeSet
        System.out.println("equals() for " + this + " & " + ((Student) obj));
        return false;// return false/true doesn't make any sense here
    }

    @Override
    public int compareTo(Object obj) {
        Student s = (Student) obj;
        // internally inside TreeSet, compareTo is used to decide
        // whether two objects are equal or not,
        // i.e. compareTo will return 0 for same object(here student name)
        System.out.println("compareTo() for " + this + " & " + ((Student) obj));
        // achieving uniqueness
        return name.compareTo(s.name);
    }

    @Override
    public String toString() {
        return name + ":" + rollNo;
    }
}

出力:

compareTo() for aaa:4 & ccc:2
compareTo() for ddd:1 & ccc:2
compareTo() for bbb:3 & ccc:2
compareTo() for bbb:3 & aaa:4
compareTo() for ddd:5 & ccc:2
compareTo() for ddd:5 & ddd:1
[aaa:4, bbb:3, ccc:2, ddd:1]
[ddd:1, ccc:2, bbb:3, aaa:4]

友人、2 つのコンパレーターを使用して得たものは何でも、オブジェクトを追加しながら同じことを達成することは可能ですか?? 最初に要素を追加してから、新しいコンパレータを使用して目的の順序を達成することはできません。
私は何千もの値を操作しているので、パフォーマンスも考慮する必要があります。

4

4 に答える 4

5

TreeSet並べ替えと一意のチェックのための要素を追加しながら、コンパレータを使用します。

問題は、ロール番号にコンパレータを使用すると、ロール番号と一意のロール番号でもソートされることです。ツリーセットで両方を一緒にすることはできません。

行くことをお勧めします。

  1. TreeSetここでは、重複除去に集中します
  2. 次に、一意のデータを取得したらArrayList、必要な順序で並べ替えます
于 2010-12-16T12:37:05.417 に答える
4

注文

指定されたコンパレータで aを使用することに関する@ralphの回答TreeSetは良いものです。それを使用してください。

デザイン

単に生のコレクションを使用するのではなく、正しい動作を公開および文書化するクラス内に「学生データベース」の概念をラップする必要があります。特定の順序で学生のリストを取得することが設計要件である場合は、メソッドを公開します (おそらくそれを返すIterable<Student>ことです。バックグラウンドでは、使用パターンに応じてさまざまなことを行うことができます。

  • Set1 つまたは複数のを維持しMaps、興味のある分野ごとに学生を分類/索引付けします。
  • Arrays.sort()および指定されたを使用したオンデマンドのインプレース配列ソートComparator。

例....

final class StudentTable {
   private static final Comparator<Student> studentRollNoComparator = ...;
   private final SortedSet<Student> sortedByRollNo = 
      new TreeSet<Student>(studentRollNoComparator);

   public Iterable<Student> studentsOrderedByRollNo()
   {
      return sortedByRollNo;
   } 

   //see below
   public void addStudent(final Student foo) { ... }
}

独自性

生徒の名前だけを比較するには、クラスをオーバーライドする必要がありますequals()。次に、あなたの. 明らかに、これを行う場合、を挿入する前に確認するために防御的にコーディングする必要があるため、重複があるかどうかがわかります。hashCode()StudentTreeSetstudentSet.contains(newStudent)newStudent

final class Student implements Comparable {
   ...

   @Override
   public boolean equals(Object o)
   {
      return o!=null &&  
             o (instanceof Student) &&
             ((Student)o).name.equals(this.name);
   }

   @Override
   public int hashCode()
   {
      return name.hashCode();  // good enough for this purpose
   } 
}

これを配置すると、学生を挿入するコードは次のようになります。

void addNewStudent(final Student toAdd)
{
   if (studentSet.contains(toAdd)) { 
      throw new IllegalStateException("Student with same name as "+toAdd+" already exists.");
   }

   studentSet.add(toAdd);
}

ツリーセットは、名前が一意の学生でいっぱいになり、そうでない場合、追加操作は失敗を報告します。(例外をスローすることは 1 つの潜在的なルートにすぎず、重複した名前を持つ学生を追加することが実際には例外的な条件である場合にのみ適切ですが、あなたは言いませんでした。)

于 2010-12-16T14:39:08.933 に答える
1

別のコンパレータで新しい TreeSet を初期化できます。- したがって、新しい Comparator (java.util.Comparator インターフェイスを実装する) を作成し、このコンパレータを使用して新しい TreeSet を初期化し、すべての学生をセットに追加するだけです。

TreeSet<Student> sortedByRollNo new TreeSet<Student>(new RollNoComparator());
sortedByRollNo.addAll(allStudents);

TreeSet<Student> sortedByY new TreeSet<Student>(new YComparator());
sortedByY.addAll(allStudents);

各ツリー セットは、並べ替え用に独自のコンパレータを持つことができます。コンパレータが指定されていない場合、ツリー セットはセット要素の自然順序付けを使用します。

追加した

uniqe Students という名前だけが必要な場合は、次の 2 つの方法があります。

  • 学生の名前が等しい場合に0を返すようにコンパレータを実装します（ただし、これはハックのようなものだと思います）。
  • 最初に生徒を名前でフィルタリングしてから、rollNo で並べ替えます。

このようなビット：

TreeSet<Student> sortedByRollNo new TreeSet<Student>(new RollNoComparator());
sortedByRollNo.addAll(new TreeSet<Student>(allStudends)); //this uses the native comparator to filter by uniqe name
于 2010-12-15T07:03:12.920 に答える
0

ここで遅くなって申し訳ありませんが、ここにエレガントな解決策があります：

    public class OwnSortedList<T> extends TreeSet<T> {

    private static final long serialVersionUID = 7109828721678745520L;


    public OwnSortedList(Comparator<T> levelScoreComparator) {
        super(levelScoreComparator);
    }


    public boolean add(T e) {
        boolean existsElement = false;

        Iterator<T> it = iterator();
        while(it.hasNext() && !existsElement){
            T nextElement = it.next();
            if(nextElement.equals(e)){
                // Element found
                existsElement = true;
                Comparator<? super T> comparator = comparator();
                int compare = comparator.compare(nextElement, e);
                if(compare > 0){
                    remove(nextElement);
                    super.add(e);
                    //element added so return true
                    return true;
                }

            }
        }

        if(!existsElement){
            super.add(e);
        }

        return false;
    }
}
于 2013-08-02T08:38:52.390 に答える