compareTo() 方法的实现问题

Question

基本上，我有2个班级。 其中一个有一个私有成员 ArrayList（来自另一个类的对象），列表中的每个对象都有一个私有字段点。 我有一种方法可以遍历列表并获得所有点的总和。 所以我只想通过它们的总和来比较 list1 > list2 。 但我没能做到——我的 compareTo() 总是返回 0。

这是一个简短的代码示例。

public class StudentsGroup implements IFile, Comparable {
    private List<Student> studentsList = new ArrayList<Student>();


    public int compareTo(Object o) {
        if(StudentsGroup.getTotalPoints(studentsList) < ((StudentsGroup)o).getTotalPoints(studentsList))
            return 1;
        else if(StudentsGroup.getTotalPoints(studentsList) > ((StudentsGroup)o).getTotalPoints(studentsList))
            return -1;
        else 
            return 0;
    }

    public static int getTotalPoints(List<Student> studentsList1) {
        int totalPoints = 0;
        for(Student o : studentsList1) {
            totalPoints += o.getStudentPoints();
        }
        return totalPoints;
    }
}

方法

Answer 1

 if(
  StudentsGroup.getTotalPoints(studentsList) <   
  ((StudentsGroup)o).getTotalPoints(studentsList))

您将相同的studentsList传递给计算的双方。

“其他组” o根本没有使用。

它可能看起来像使用了o ，但getTotalPoints是一个static方法，您在哪个实例上调用它并不重要。 编译器也会就此向您发出警告。 不要忽略编译器警告。

立即修复是将代码更改为

if( getTotalPoints(studentsList) < getTotalPoints((StudentsGroup)o).studentsList)

但是您可能应该将getTotalPoints方法从public static更改为public （非静态）。 而不是将列表作为参数传递，它可以只this.studentsList内部使用this.studentsList 。

if (this.getTotalPoints() < ((StudentsGroup)o).getTotalPoints())

Answer 2

在这种情况下，我会检查值是否相同（或都为 0）

public class StudentsGroup implements IFile, Comparable<StudentsGroup> {
    private List<Student> studentsList = new ArrayList<Student>();


    public int compareTo(StudentsGroup sg) {
        return Integer.compare(getTotalPoints(), sg.getTotalPoints());
    }

    public int getTotalPoints() {
        return Math.toIntExact(studentsList.stream()
                                           .mapToInt(Student::getStudentPoints).sum());
    }
}

通过简化代码，您不太可能将静态方法与实例方法(StudentsGroup)o).getTotalPoints(studentsList)只需调用StudentsGroup.getTotalPoints(studentsList)因为您没有实例方法。