Java 8 - 转换列表的最佳方法：map还是foreach？

java java-8 java-stream

2022-08-31 05:43:28

我有一个列表，我想在其中过滤元素并对每个元素应用一种方法，然后将结果添加到另一个列表中。myListToParsemyFinalList

在Java 8中，我注意到我可以用2种不同的方式做到这一点。我想知道他们之间更有效的方式，并理解为什么一种方式比另一种方式更好。

我对任何关于第三种方式的建议持开放态度。

方法 1：

myFinalList = new ArrayList<>();
myListToParse.stream()
        .filter(elt -> elt != null)
        .forEach(elt -> myFinalList.add(doSomething(elt)));

方法 2：

myFinalList = myListToParse.stream()
        .filter(elt -> elt != null)
        .map(elt -> doSomething(elt))
        .collect(Collectors.toList());

答案 1

不要担心任何性能差异，在这种情况下，它们通常会很小。

方法 2 是首选，因为

它不需要改变存在于 lambda 表达式外部的集合。
它更具可读性，因为在收集管道中执行的不同步骤是按顺序编写的：首先是筛选器操作，然后是映射操作，然后是收集结果（有关收集管道的好处的更多信息，请参阅Martin Fowler的优秀文章。
您可以通过替换所使用的值来轻松更改收集值的方式。在某些情况下，您可能需要编写自己的，但好处是您可以轻松重用它。CollectorCollector

答案 2

我同意现有的答案，即第二种形式更好，因为它没有任何副作用，并且更容易并行化（只需使用并行流）。

性能方面，在您开始使用并行流之前，它们似乎是等效的。在这种情况下，地图的性能会更好。请参阅下面的微观基准测试结果：

Benchmark                         Mode  Samples    Score   Error  Units
SO28319064.forEach                avgt      100  187.310 ± 1.768  ms/op
SO28319064.map                    avgt      100  189.180 ± 1.692  ms/op
SO28319064.mapWithParallelStream  avgt      100   55,577 ± 0,782  ms/op

你不能以同样的方式提升第一个示例，因为 forEach 是一个终端方法 - 它返回 void - 所以你被迫使用有状态的 lambda。但是，如果您使用的是并行流，那确实是一个坏主意。

最后请注意，可以使用方法引用和静态导入以更简洁的方式编写第二个代码段：

myFinalList = myListToParse.stream()
    .filter(Objects::nonNull)
    .map(this::doSomething)
    .collect(toList());