Java 8中使用Math.round排序员工经验浮点数出错问题

这个问题我之前踩过坑！核心问题出在你用Math.round()来生成Comparator.compare()的返回值，这完全违背了这个方法的设计逻辑。

为什么当前代码会出错？ #

Comparator.compare()的返回值规则是：

• 返回负数：表示第一个参数应该排在第二个参数前面
• 返回0：表示两个参数排序优先级相同
• 返回正数：表示第一个参数应该排在第二个参数后面

而你用Math.round(t.getExperience() - t1.getExperience())时，会把本应区分的细微差值错误处理：
比如比较5.5和5.0，差值是0.5，Math.round()返回1——这会让compare认为5.5应该排在5.0的后面，直接导致了你看到的5.0,5.5这种错误排序。
再比如如果两个值的差值是0.4，Math.round()会返回0，错误地判定这两个值相等，破坏排序逻辑。

正确的解决方案 #

最简洁可靠的方式是使用Double.compare()方法，它专门为浮点数比较设计，能正确处理精度问题和排序逻辑：

Java 7及以下写法：

Collections.sort(employeeList, new Comparator<Emp>() {
    @Override
    public int compare(Emp t, Emp t1) {
        // 降序排序：将t1的经验值和t的经验值比较，实现从大到小排列
        return Double.compare(t1.getExperience(), t.getExperience());
    }
});

Java 8+ 简化Lambda写法：

employeeList.sort((emp1, emp2) -> Double.compare(emp2.getExperience(), emp1.getExperience()));

如果你想手动实现差值判断（不推荐，但可以理解逻辑） #

如果不想用Double.compare()，也可以手动判断差值的符号，但要注意浮点数精度问题：

Collections.sort(employeeList, new Comparator<Emp>() {
    @Override
    public int compare(Emp t, Emp t1) {
        double experienceDiff = t1.getExperience() - t.getExperience();
        if (experienceDiff > 0) {
            return 1; // t1经验更大，排在前面
        } else if (experienceDiff < 0) {
            return -1; // t经验更大，排在前面
        } else {
            return 0; // 经验值相等
        }
    }
});

总结 #

永远不要用Math.round()或者直接将浮点数差值强转为int来作为compare的返回值——这会丢失关键的排序信息，导致混乱的结果。使用Double.compare()是处理浮点数排序的标准做法，既安全又简洁。

内容的提问来源于stack exchange，提问作者Shailendra Madda