Java8 Null 安全比较

Question

比较两种产品时遇到问题。我想比较它们每个的年份(可选)属性。但是只要此属性为 null，就会抛出 NPE。我想用 Comparator.nullsLast(..) 我可以处理空值......但似乎我对它的工作原理有误解，或者代码有问题。我需要更改什么才能使这项工作对 null 友好？

@Override
public int compare(IProduct product1, IProduct product2) throws ProductComparisonException {

    Comparator<IShopProduct> comparator =
        Comparator.nullsLast(Comparator.comparing(IShopProduct::getVintage));

    return comparator.compare((IShopProduct)product1.getProvidedProductData(),
                              (IShopProduct)product2.getProvidedProductData());
}

提前致谢

Answer 1

应该是

Comparator<IShopProduct> comparator = 
            Comparator.comparing( IShopProduct::getVintage, 
                             Comparator.nullLast(naturalOrder()));

Comparator.nullFirst()/nullLast() 考虑 null 值大于/小于非 Null 对象

编辑

这是 Comparator.comparing() 的实现:

public static <T, U extends Comparable<? super U>> Comparator<T> comparing(
        Function<? super T, ? extends U> keyExtractor)
{
    Objects.requireNonNull(keyExtractor);
    return (Comparator<T> & Serializable)
        (c1, c2) -> keyExtractor.apply(c1).compareTo(keyExtractor.apply(c2));
}

如您所见，它会调用 keyExtractor.apply(c1).compareTo()，因此如果 keyExtractor.apply(c1) 为 null，它将抛出 NPE

我建议使用以下函数的代码:

public static <T, U> Comparator<T> comparing(
        Function<? super T, ? extends U> keyExtractor,
        Comparator<? super U> keyComparator)
{
    Objects.requireNonNull(keyExtractor);
    Objects.requireNonNull(keyComparator);
    return (Comparator<T> & Serializable)
        (c1, c2) -> keyComparator.compare(keyExtractor.apply(c1),
                                          keyExtractor.apply(c2));
}

基本上它会提取值，然后将比较值传递给Comparator。

这些值将被传递给解析为 value1.compareTo(value2) 的 naturalOrder() 比较器。通常它会抛出 NPE，但我们用 Comparator.nullLast 包装了它，它有特殊的 null 处理程序。