地图vs。peek-Intellij建议[重复]

Question

collection.stream().map(x -> {
    if(condition) {
        x.setY(true);
    }
    return x;
}).collect(Collectors.toList()));

我有一个集合，当满足某个条件时，我想在x变量中设置一些东西。我像上面代码中显示的那样做。它完成了它的工作，并且工作了。然而，IntelliJ建议我将. map与. peek交换。所以代码看起来像：

collection.stream().peek(x -> {
    if(condition) {
        x.setY(true);
    }
}).collect(Collectors.toList()));

虽然短了一行，但从我在peek文档中读到的内容来看：

API注意：此方法主要用于支持调试，在调试中，您希望在元素流过管道中的某个点时看到它们：

那么，IntelliJ的建议有误导性吗？

Answer 1

不，这不是误导。JavaDoc声明：

偷看

Stream peek（消费者行动）

返回一个由该流的元素组成的流，在从结果流中消耗元素时，对每个元素执行提供的操作。

JavaDoc非常清楚，peek能够修改流。

Answer 2

例：

如果在java 8中使用peek（）和Count（），peek（）会起作用，但是如果在java 9中使用，除非你有Filter（），否则不会起作用，因为java 9中的Count（）不会遍历所有元素。

你应该更喜欢forEach而不是peek。

List<Integer> l = new ArrayList<>(Arrays.asList(1,2,3));
long c = l.stream().peek(System.out::println).count();
System.out.println(c);

试试上面java 8和9中的代码，看看区别。我的观点是，你应该遵循API留档所说的，并相应地使用它。