Java8零安全比较

Question

在比较两种产品时有问题。我希望比较它们中每一个的vintage (这是可选的)属性。但是，每当该属性为null时，就会抛出NPE。我想用Comparator.nullsLast(..)我可以处理空值..。但似乎我对这是如何工作的有误解，或者代码有问题。我需要更改什么才能使工作成为零友好型？

@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()认为空值大于/小于nonNull对象

编辑

这是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。

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

Answer 2

如果传入的密钥提取器函数是比较，或者提取的属性是null，则该方法的重载将引发异常。既然您提到了vintage属性有时可以是null，那么这就是NullPointerException的原因。

要克服这一问题，另一种办法是使用这个比较器：

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

密钥提取器，即IShopProduct::getVintage是用于提取排序密钥的函数。

密钥比较器，即Comparator.nullsLast(Comparator.naturalOrder())，用于比较排序键。

这里的Comparator.naturalOrder()只返回一个比较器，该比较器按自然顺序比较Comparable对象。