Java错误:Java . lang . illegalargumentexception:比较方法违反了它的一般约定

Question

我正在开发一个旧的应用程序，最初是在Java6编写的，几年前升级到Java7。

在这个应用程序中，我使用集合。通过实现比较器接口，使用自定义比较方法对列表进行排序。列表中的对象类型是Company附表，它具有3个属性comanyName、计划和到期日期。

列表可以包含多个具有相同comanyName但具有唯一过期日期的对象。下面的比较函数按公司名称的升序对列表进行排序，并在同一公司名称列表中按过期日期的降序进行排序。下面是方法实现。

    public int compare(CompanySchedule c1, CompanySchedule c2) {
        int returnVal = 0;
        int value = c1.getCompany().getName().compareTo(c2.getCompany().getName());
        if (value == 0){
            if (c1.getUseExpirationDate() == null || c2.getUseExpirationDate() == null){
                returnVal = -1;
            }
            else{
                int chkdate = c1.getUseExpirationDate().compareTo(c2.getUseExpirationDate());
                if (chkdate == 0){
                    returnVal = 0;
                }
                else if (chkdate > 0){
                    returnVal = -1;
                }
                else if (chkdate < 0){
                    returnVal = 1;
                }
            }
        }
        else if (value < 0){
            returnVal = -1;
        }
        else if (value > 0){
            returnVal = 1;
        }


        return returnVal;
    }

我知道，当在上面的比较方法实现中不满足可传递属性时，将抛出错误java.lang.IllegalArgumentException：比较方法违反了它的一般约定。

有人可以帮助确定此方法将违反传递属性的位置。感谢您的帮助。

Answer 1

我知道我已经迟到了，一旦我找到了解决方案，我基本上忘记了我问过这个问题。正如我所提到的，应用程序是用Java 6编写的，然后根据我对Collections内部实现的理解，升级到Java 7。

Sort方法将使用的算法从Merge Sort更改为Tim Sort，Merge Sort忽略比较方法实现的Transitivity属性，Tim sort要求compare方法实现是可传递的，否则会抛出上述异常，因为我们有一个遗留应用程序，我们不想更改代码中的任何内容，所以我们使用了＜code＞jvm＜/code＞argument＜code＞java.util.Arrays。useLegacyMergeSort=true在排序实现中内部合并排序

Answer 2

我认为这里有一个问题：

if (c1.getUseExpirationDate() == null || c2.getUseExpirationDate() == null){
                returnVal = -1;
}

如果a.getUseExpirationDate（）==null，同时b.getUseExpirationDate=（）==null，您将得到

这会破坏一致性。这种方法可能还有更多问题，但我还没有全部检查。

祝你好运。

编辑

这个代码怎么样？

public int compare(CompanySchedule c1, CompanySchedule c2) {
        int returnVal = 0;
        int value = c1.getCompany().getName().compareTo(c2.getCompany().getName());
        if (value == 0) {
            if (c1.getUseExpirationDate() == null && c2.getUseExpirationDate() != null) {
                returnVal = -1;
            } else if (c1.getUseExpirationDate() != null && c2.getUseExpirationDate() == null) {
                returnVal = 1;
            } else if (c1.getUseExpirationDate() == null && c2.getUseExpirationDate() == null) {
                returnVal = 0;
            } else {
                int chkdate = c1.getUseExpirationDate().compareTo(c2.getUseExpirationDate());
                if (chkdate == 0) {
                    returnVal = 0;
                } else if (chkdate > 0) {
                    returnVal = -1;
                } else if (chkdate < 0) {
                    returnVal = 1;
                }
            }
        } else if (value < 0) {
            returnVal = -1;
        } else if (value > 0) {
            returnVal = 1;
        }


        return returnVal;
    }

为了可比性，我尽量不对其进行太多更改，但应该对其进行重构。基本上，它确定空值小于其他值。