SonarQUBE 和 compareTo 日期

Question

我得到了这个声纳错误：

改为使用原始值。规则：既不是“Math.abs”也不是否定 应该用于可能是“MIN_VALUE”的数字

在这种比较日期的方法中：

public int compareDates(MyDto a, MyDto b) {
        SimpleDateFormat sdf = new SimpleDateFormat("dd.MM.yyyy hh:mm");
        try {
            Date dateA = sdf.parse(a.getStartDate() + " " + a.getStartHour());
            Date dateB = sdf.parse(b.getStartDate() + " " + b.getStartHour());
            return - dateA.compareTo(dateB); //Sonar BUG
        } catch (ParseException e) {
            logger.error("Unable to parse date: " + e.getMessage());
            return 0;
        }   
    }

我正在使用这种方法按日期对 MyDto 的集合进行排序：

Collections.sort(myDtoList, (a,b) -> compareDates(a,b));

现在我真的不知道该怎么修改。比较返回的整数可以是负数，也可以不是，我该如何解决才能让 SQ 开心？

Answer 1

IMO 这几乎是误报。

dateA.compareTo(dateB) 的结果可能可能是Integer.MIN_VALUE，在这种情况下-dateA.compareTo(dateB) 将是also evaluate to Integer.MIN_VALUE，因此不会产生您想要的结果。

实际上，该调用极有可能返回-1、0 或1（即not specified 只返回这些值，但Date.compareTo() 的所有实现我见过这样做的）。

但是有一个非常简单的方法可以完全避免否定（以及这个特定问题）：

如果你想要相反的方向，只需返回dateB.compareTo(dateA)！只需交换您用作参数的内容与否定结果的效果相同，而没有声纳提到的风险。