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]

朋友们,无论我使用两个比较器得到什么,在添加对象时是否可以实现相同的效果?我不能先添加元素然后使用新的比较器来实现所需的顺序。
我正在操纵数千个值,因此还需要考虑性能。

4

4 回答 4

5

TreeSet添加元素进行排序和唯一检查时,它将使用比较器,

现在的问题是,如果您将比较器用于卷号,您将按照卷号和唯一卷号对其进行排序。你不能在树集中同时拥有两者。

我建议你去。

  1. TreeSet在这里你专注于重复删除
  2. 然后,一旦您拥有独特的数据,ArrayList就可以按照您想要的任何顺序对其进行排序
于 2010-12-16T12:37:05.417 回答
4

订购

@ralph关于使用带有指定比较器的答案TreeSet是一个很好的答案,使用它。

设计

您应该将“学生数据库”的概念包装在一个公开和记录正确行为的类中,而不仅仅是使用原始集合。如果按特定顺序获取学生列表是设计要求,请公开方法(也许返回Iterable<Student>那个说明。在幕后,您可以根据使用模式做各种事情:

  • 按感兴趣的领域维护一个或多个学生Set和/或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()在您的Student班级上,仅比较学生姓名。然后你会在你的TreeSet. 显然,如果你这样做,你需要studentSet.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);
}

然后,您的树集中充满了姓名唯一的学生,如果不是,您的添加操作将报告失败。(抛出异常只是一种可能的途径,并且仅在添加具有重复姓名的学生实际上是异常情况时才适用,但您没有说。)

于 2010-12-16T14:39:08.933 回答
1

您可以使用不同的比较器初始化新的 TreeSet。- 所以你所要做的就是编写一个新的比较器(实现 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,那么您有两种方法:

  • 以某种方式实现比较器,如果 studens 的名称相等,它返回 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 回答