你的比较器是不可传递的。

假设A是B的父类，B是C的父类，因为A > B和B > C，那么A > C必须是这样。然而，如果你的比较器在A和C上被调用，它将返回零，这意味着A == C。这违反了契约，因此抛出异常。

库能够检测到这一点并让您知道，而不是表现得不稳定，这是相当不错的。

在compareParents()中满足传递性要求的一种方法是遍历getParent()链，而不是只查看直接祖先。

即使你的compareTo在理论上具有可传递性，有时细微的bug会把事情搞糟……如浮点运算错误。这件事发生在我身上。这是我的代码:

public int compareTo(tfidfContainer compareTfidf) {
    //descending order
    if (this.tfidf > compareTfidf.tfidf)
        return -1;
    else if (this.tfidf < compareTfidf.tfidf)
        return 1;
    else
        return 0;

}   

传递属性显然持有，但由于某种原因，我得到了IllegalArgumentException。事实证明，由于浮点运算中的微小错误，舍入错误导致传递属性在不应该发生的地方中断!所以我重写了代码，考虑到真正微小的差异0，它工作:

public int compareTo(tfidfContainer compareTfidf) {
    //descending order
    if ((this.tfidf - compareTfidf.tfidf) < .000000001)
        return 0;
    if (this.tfidf > compareTfidf.tfidf)
        return -1;
    else if (this.tfidf < compareTfidf.tfidf)
        return 1;
    return 0;
}   

我曾在一段代码中看到过这种情况，其中经常执行空值检查:

if(( A==null ) && ( B==null )
  return +1;//WRONG: two null values should return 0!!!

只是因为这是我得到的当我谷歌这个错误，我的问题是我有

if (value < other.value)
  return -1;
else if (value >= other.value)
  return 1;
else
  return 0;

>= other。Value(显然)实际上应该是Value > other。值，这样你就可以用相等的对象返回0。

以我为例，我做了如下事情:

if (a.someField == null) {
    return 1;
}

if (b.someField == null) {
    return -1;
}

if (a.someField.equals(b.someField)) {
    return a.someOtherField.compareTo(b.someOtherField);
}

return a.someField.compareTo(b.someField);

我忘记检查的是当a.someField和b.someField都为空时。

如果compareParents(s1, s2) == -1，则期望compareParents(s2, s1) == 1。对于你的代码，这并不总是正确的。

具体来说，如果s1. getparent () == s2 && s2. getparent () == s1。 这只是可能出现的问题之一。