内置比较：比较方法违反其一般合同

Question

我目前正在Java中对集合进行排序。我收到了错误消息“比较方法违反了它的一般契约”。我也理解这个错误消息，但我（主要）使用Long类型的构建比较方法。所以我不知道，在这种情况下，排序方法仍然违反了契约。这是我的代码：

@Override
public int compareTo(DataAge another) {
    if(this == null || another == null)
        return 0;

    Long a = new Long(this.getAge());
    Long b = new Long(another.getAge());
    return a.compareTo(b);
}

这里是错误：

Java exception occurred:
java.lang.IllegalArgumentException: Comparison method violates its general contract!

at java.util.ComparableTimSort.mergeLo(Unknown Source)

at java.util.ComparableTimSort.mergeAt(Unknown Source)

at java.util.ComparableTimSort.mergeCollapse(Unknown Source)

at java.util.ComparableTimSort.sort(Unknown Source)

at java.util.ComparableTimSort.sort(Unknown Source)

at java.util.Arrays.sort(Unknown Source)

at java.util.Collections.sort(Unknown Source)

at dd.GMAAnalyzer.sortData(Analyzer.java:158)

Answer 1

来自< code>Comparable的Javadoc:

请注意，null 不是任何类的实例，e.compareTo（null） 应该抛出 NullPointerException，即使 e.equals（null） 返回 false。

此外，您不需要创建< code>Long实例:

return Long.compare(this.getAge(), another.getAge());

Answer 2

假设允许nulls，那么方法的逻辑是不正确的，因为null等于任何值。这是错误的，因为它将等同于多个彼此不相等的事物，从而打破了及物性。

若要解决此问题，请确定 nulls 是应该在其他数字的前面还是后面排序，并添加一个单独的 null 比较（这不能等于 null，因此您不需要比较它）。

if(another == null)
    return 1; // If you want nulls in the back, return -1