如何从流可以为空的对象返回可为空的列表

Question

我有一个方法，它接受一个内部有ArrayList的对象：

private List<String> getWatchlistDetail(MatchData k) {
        if(k == null)
            return new ArrayList<>();
        return
                k.getWatchListDetail()
                        .stream()
                        .collect(Collectors.toList());
    }

MatchData k可以是null，有时k.getWatchListDetail（）也可以是null。

我需要检查两种情况。首先，如果它能抛出NPE。

上面的实现可以做到这一点，但我尝试使用可选。ofNullable（）或带有链接的流，所以我可以在一行链接中完成。

Answer 1

MatchData k是null，有时k.getWatchListDetail（）也是null，所以我需要检查两者

...所以我可以用一条链子来做。

当然，您可以在一条stream语句中同时检查这两条语句。

private List<String> getWatchlistDetail(MatchData k) {
    
    return Stream.ofNullable(k)
        .map(MatchData::getWatchListDetail)
        .filter(Objects::nonNull)
        .flatMap(List::stream)
        .collect(Collectors.toList());
}

但是，请记住：

• 这不是补救措施，而是试图隐藏设计缺陷

现在将其与下面的方法进行比较，该方法既不隐藏空检查，也不滥用流，并且是可选的：

private List<String> getWatchlistDetail(MatchData k) {
    
    return k != null && k.getWatchListDetail() != null ?
           k.getWatchListDetail() : Collections.emptyList();
}

Java引入函数式编程是为了降低代码的复杂性，而不是相反，仅将其用于隐藏空检查不会让您感到任何困难。

就其本身而言，空检查不是问题，而只是问题的一个指示器。为了解决这个问题，需要改进应用程序的设计。

使用可选。不可用的

关于是否可以使用可选来避免条件逻辑的问题，请看@StuartMarks（JDK开发人员）的回答：

可选的主要用途如下：

可选旨在为库方法返回类型提供一种有限的机制，其中显然需要表示“无结果”，并且为此使用null极有可能导致错误。

典型的代码味道是，它不是使用方法链接来处理从某个方法返回的可选项，而是从可为null的内容创建可选项，以链接方法并避免条件。