使用Comparator按Float参数排序JSON触发契约违反错误求助

哥们，我太懂你遇到的这个坑了！这个报错本质是你的Comparator违反了Java比较器必须遵守的通用契约——也就是自反性、对称性和传递性，尤其是处理Float类型的时候，很容易因为忽略NaN、null或者正负零这些特殊情况踩雷。

先说说你大概率踩的坑 #

你原代码里的比较逻辑估计是直接用了>/<运算符，或者没处理Float的特殊值：

• NaN的问题：Float.NaN和任何值（包括自己）用>/<比较都会返回false，这会导致a.compare(b)和b.compare(a)的结果不符合对称性，甚至传递性直接失效。
• null值处理缺失：如果你的myContact对象或者它的percent属性可能为null，没做判断的话，要么抛空指针，要么破坏比较契约。
• 正负零的坑：Float里的+0.0和-0.0数值相等，但如果逻辑处理不当也可能引发契约违反问题。

修复后的Comparator代码示例 #

直接给你改好的代码，注释里写清了每个关键处理点：

public class MyChangeComparator implements Comparator<myContact> {
    private final int type;

    // 构造方法初始化排序类型（比如1=升序，2=降序）
    public MyChangeComparator(int type) {
        this.type = type;
    }

    @Override
    public int compare(myContact a, myContact b) {
        // 先处理myContact对象本身为null的情况（按你的业务需求决定null的位置）
        if (a == null && b == null) return 0;
        if (a == null) return type == 1 ? -1 : 1; // null放前面（升序）或后面（降序）
        if (b == null) return type == 1 ? 1 : -1;

        // 获取要比较的Float属性（替换成你实际的方法名）
        Float percentA = a.getPercent_...();
        Float percentB = b.getPercent_...();

        // 处理属性为null的情况
        if (percentA == null && percentB == null) return 0;
        if (percentA == null) return type == 1 ? -1 : 1;
        if (percentB == null) return type == 1 ? 1 : -1;

        // 处理NaN：统一把NaN放到排序末尾（可根据需求调整）
        boolean isNaN_A = Float.isNaN(percentA);
        boolean isNaN_B = Float.isNaN(percentB);
        if (isNaN_A && isNaN_B) return 0;
        if (isNaN_A) return type == 1 ? 1 : -1;
        if (isNaN_B) return type == 1 ? -1 : 1;

        // 用Float.compare()处理正负零和正常数值比较，它已经遵守了比较契约
        int compareResult = Float.compare(percentA, percentB);

        // 根据type决定升序还是降序
        return type == 1 ? compareResult : -compareResult;
    }
}

敲黑板划重点 #

1. 一定要用Float.compare()：别自己写>/<的判断，这个方法已经帮你处理了NaN、正负零的特殊情况，完全符合比较契约。
2. 明确null的排序规则：不管是对象还是属性为null，都要提前定义好它们的排序位置，不能放任不管。
3. 测试特殊场景：一定要拿包含NaN、null、正负零的数据测试，确保排序逻辑稳定。

内容的提问来源于stack exchange，提问作者Hadi Ahmadi