java.lang.IllegalArgumentException: Comparison method violates its general contract

java.lang.IllegalArgumentException: Comparison method violates its general contract

java.lang.IllegalArgumentException: 比较方法违反了其通用约定！

Java中的Comparator接口指定了比较方法应遵循以下规则：

1. 自反性：compare(a, a) 应返回 0。
2. 对称性：如果 compare(a, b) 返回一个正值，那么 compare(b, a) 应返回一个负值，反之亦然。
3. 传递性：如果 compare(a, b) 返回一个正值 且 compare(b, c) 返回一个正值，那么 compare(a, c) 应返回一个正值。
4. 一致性：如果被比较的对象没有发生变化，比较的结果应保持不变。

如果自定义的比较方法违反了上述规则，将会出现"java.lang.IllegalArgumentException: Comparison method violates its general contract!"的异常。

正常情况下，a与b是具体的基本数据类型值比较，逻辑比较清晰，不容出错

例如:

list.sort((s1, s2) -> {
    final int s1Val = Integer.parseInt(s1.getDeviceType().getLevel().getValue());
    final int s2Val = Integer.parseInt(s2.getDeviceType().getLevel().getValue());
    if (s1Val == s2Val) {
        return 0;
    }
    return s1Val < s2Val ? 1 : -1;
});

或

list.sort((s1, s2) -> {
    return Integer.valueOf(s1.getDeviceType().getLevel().getValue()).compareTo(Integer.valueOf(s2.getDeviceType().getLevel().getValue())) * -1;
});

但当不是具体的数值比较时,逻辑判断模糊,就有可能违反Comparator规则

例如：

list.sort((s1, s2) -> {
    if (s1.getDeviceType().equals(s2.getDeviceType())) {
        return 0;
    }
    return Integer.parseInt(s1.getDeviceType().getLevel().getValue()) < Integer.parseInt(s2.getDeviceType().getLevel().getValue()) ? 1 : -1;
});

总结

当不是具体的数值比较时，要罗列多种具体情况下的比较规则，遵循Comparator规则