Java自定义排序比较双日期属性对象时抛出比较方法契约违反异常

你当前的比较逻辑从根本上破坏了比较器要求的传递性规则，具体矛盾如下：

• 比较键的选择逻辑存在非对称缺陷：原代码的空值判断条件为first.getPrimaryDate() == null || second.getPrimaryDate() == null时，会直接取两个对象的secondaryDate做比较，完全忽略非空那个对象的primaryDate作为排序基准的优先级。
• 直接触发契约违反的典型场景（完全匹配你提到的业务约束）：
  1. 构造三个合法业务对象：
     • 对象X：primaryDate=2024-02-01，secondaryDate=2024-01-01（满足primary不早于secondary的约束）
     • 对象Y：primaryDate=null，secondaryDate=2024-01-01（secondary非空）
     • 对象Z：primaryDate=2024-03-01，secondaryDate=2024-01-01（满足primary不早于secondary的约束）
  2. 按原逻辑做两两比较：
     • X vs Y：因Y的primaryDate为null，直接比较secondaryDate，二者secondary值相同，返回相等（0）
     • Y vs Z：因Y的primaryDate为null，直接比较secondaryDate，二者secondary值相同，返回相等（0）
     • X vs Z：二者的primaryDate都非空，比较primaryDate，X的2024-02 < Z的2024-03，返回X < Z
  3. 上述结果完全违背传递性要求：比较器规则要求如果X=Y、Y=Z，则必须满足X=Z，但实际逻辑中X<Z，排序算法在检测到这类逻辑矛盾时，就会抛出Comparison Method Violates Its General Contract!异常。

核心思路是为每个对象独立提取统一的有效排序键，不再在比较过程中交叉引用其他对象的字段做判断，完全匹配“优先使用primaryDate、为空则兜底用secondaryDate”的业务规则：

1. 新增有效日期提取方法，为每个对象单独计算排序基准：

   private LocalDate resolveSortDate(MyClass item) {
       return item.getPrimaryDate() != null 
           ? item.getPrimaryDate() 
           : item.getSecondaryDate();
   }
   

2. 基于Java Comparator的链式API重构比较器逻辑，结合日期降序、Code值升序的次级排序要求：

   Comparator<MyClass> comparator = Comparator
       .comparing(this::resolveSortDate)
       .reversed()
       .thenComparing(MyClass::getCodeAsLong);
   

3. 用重构后的比较器执行排序即可：

   return listOfClass.stream()
       .sorted(comparator)
       .collect(Collectors.toList());
   

• 原代码中reversed()的链式位置本身没有语法错误，但结合自定义lambda的逻辑容易出现比较方向反转的偏差，使用Comparator.comparing()的链式写法可以完全避免手动编写比较分支带来的空值判断错误。
• 可直接借助Comparator.nullsFirst/nullsLast的能力兼容空值场景，不需要手动写条件判断分支，进一步降低出错概率。

内容的提问来源于stack exchange，提问作者Eduardo H.