.map 与 .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

例子：

如果您在 java 8 中使用peek()with count()，peek()会起作用，但是如果您在 java 9 中使用它，除非您有filter()，否则它不会，因为count()在 java 9 中不会遍历所有元素。

你应该更喜欢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 文档的说明并相应地使用它。

Answer 2

不没有误导。该JavaDoc的状态：

窥视

Stream peek（消费者行为）

返回一个由该流的元素组成的流，另外 在每个元素上执行提供的操作，因为元素从结果流中被消耗。

JavaDoc 很清楚，peek 有修改流的能力。