使用 Comparator.nullsLast 时出现 NullPointerException

Question

我有以下代码，其中包含 2 个类 MyRange 和 MyCustomValue -class MyRange {    private Long id;    private Double minValue;    private Double maxValue;    // getters and setters    // equals, hashCode and toString    @Override    public boolean equals(Object obj) {        if (this == obj)            return true;        if (obj == null || getClass() != obj.getClass())            return false;        MyRange other = (MyRange) obj;        return Objects.equals(this.id, other.id) &&                Objects.equals(this.minValue, other.minValue) &&               Objects.equals(this.maxValue, other.maxValue);    }}class MyCustomValue {    private String value;    private MyRange myrange;    //getters and setters    // equals, hashCode and toString}如果它value是空的，MyCustomValue我希望它在最后。所以我写了如下的比较器public static final Comparator<MyCustomValue> externalMVComparator = (emv1, emv2) -> {    if(emv1.getValue() != null && emv2.getValue() == null) {        return -1;    } else if (emv1.getValue() == null && emv2.getValue() != null) {        return 1;    } else {        return myrangeMinValueComparator.compare(emv1, emv2);    }}private static final Comparator<MyRange> minValueComparator =  Comparator.nullsLast(Comparator.comparingDouble(value -> value.getMinValue()));private static final Comparator<MyCustomValue> myrangeMinValueComparator = Comparator.nullsLast(Comparator.comparing(MyCustomValue::getMyrange, minValueComparator));上述比较器工作正常。所以我决定改变externalMVComparator如下（即，使用thenComparing更多的可读性）private static final Comparator<MyCustomValue> valueComparator = Comparator.nullsLast(Comparator.comparing(MyCustomValue::getValue));public static final Comparator<MyCustomValue> externalMVComparator2 = Comparator.nullsLast(valueComparator.thenComparing(myrangeMinValueComparator));但是对列表进行排序，externalMVComparator2结果为NullPointerException. 我的代码中有什么错误？

Answer 1

问题出在以下行（Comparator.为清楚起见，我删除了）：

Comparator<MyCustomValue> valueComparator = nullsLast(comparing(MyCustomValue::getValue));

Comparator您所做的将处理类型的null值MyCustomValue。它不会处理由 .null返回的 s getValue。您必须使用 2 参数版本Comparator.comparing并null为值提供 -safe 比较器：

valueComparator = comparing(MyCustomValue::getValue, nullsLast(naturalOrder()));

以上将处理您实际想要排序的常见情况value。当我查看您的代码时，我认为您的意思是只使用valuefornull检查，否则不想按它排序。如果是这种情况，您可以将nullsLast( (x,y) -> 0) 其用作 null 安全的第二个参数，因为comparing它会将所有字符串视为相等。您也可以使用valueComparator = comparing(mcv -> mcv.getValue() == null)因为true按false自然顺序排列，但这可能不太清楚。

如果您还想处理nulls of MyCustomValue，则必须再次将其包装nullsLast起来。