使用默认实现的空安全映射比较器。

113

在Java 8中是否有内置的可能性来创建一个空安全映射比较器,而无需编写自己的Comparator实现?

运行以下代码时,可能会导致NPE,因为Comparator.comparing()keyExtractor参数可能返回null值:

public class ToSort
{

    private String sortBy;

    public ToSort(String sortBy)
    {
        this.sortBy = sortBy;
    }

    public String getSortBy()
    {
        return sortBy;
    }

    public static void main(String[] args)
    {
        // mapping comparator
        Comparator<ToSort> comp = Comparator.comparing(ToSort::getSortBy);                          
        SortedSet<ToSort> set = new TreeSet<>(comp);
        ToSort o1 = new ToSort("1");
        ToSort o2 = new ToSort(null);

        set.add(o1);

        System.out.println(set.contains(o2)); //NPE because o2.getSortBy() == null

    }
}

"main" 线程中出现异常: java.lang.NullPointerException at java.util.Comparator.lambda$comparing$77a9974f$1(Comparator.java:469) at java.util.Comparator$$Lambda$2/1480010240.compare(Unknown Source) at java.util.Comparators$NullComparator.compare(Comparators.java:83) at java.util.TreeMap.getEntryUsingComparator(TreeMap.java:376) at java.util.TreeMap.getEntry(TreeMap.java:345) at java.util.TreeMap.containsKey(TreeMap.java:232) at java.util.TreeSet.contains(TreeSet.java:234) at test.ToSort.main(ToSort.java:48)"

使用

Comparator<ToSort> comp = Comparator.nullsFirst(Comparator.comparing(ToSort::getSortBy));

只有 ToSort 对象为 null 时才能正常处理,因此也无法正常工作。

我知道如何编写自己的 Comparator 实现,但我正在寻找一种更“优雅”的解决方案。

Comparator.comparingNullsFirst(ToSort::getSortBy)
2个回答

208

找到一个可能的解决方案:

Comparator.comparing(ToSort::getSortBy, 
      Comparator.nullsFirst(Comparator.naturalOrder())
)

3
所以这个解决方案之所以有效,是因为比较器嵌套的顺序。在你的问题中,最内层的比较器(也就是第一个执行的)是 ToSort::getSortBy,而 nullsFirst 则包裹在它周围。在这个答案中,nullsFirst 比较器现在首先被执行,提供了在返回 null 的 getSortBy 调用之前的安全性。 - Geert

1

流的片段:

somePersonList
    .stream()
    .sorted(
        Comparator.comparing(SomePerson::getName,
        Comparator.nullsLast(String::compareTo))
    )
    .collect(Collectors.toList())

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接