Java 8 Stream：不使用foreach向集合添加元素的编码规范问询

首先直接给结论：你当前用peek()往外部集合others里添加元素的做法，完全不符合Stream API的编码规范，绝对要避免。

为什么不能这么用peek()？ #

Java官方文档对peek()的定位写得非常明确：它是用来支持调试，查看元素在流水线中流经某点的情况，本质是一个无副作用的"观察"操作，而不是用来执行修改外部状态的业务逻辑。

你这么写会带来几个严重问题：

• 惰性执行的坑：Stream是惰性求值的，只有当终端操作（比如你的collect）执行时，中间操作才会触发。如果后续代码调整，比如去掉了collect，那这段peek里的添加逻辑直接就不执行了，排查起来非常头疼；
• 线程安全风险：如果哪天把Stream改成并行流（parallelStream()），LinkedList本身不是线程安全的，others.add(t)会直接导致并发修改异常或者数据错乱；
• 可读性极差：其他维护代码的人看到peek，第一反应是这是调试用的临时代码，根本不会想到它在偷偷修改外部集合，很容易引入bug。

正确的做法是什么？ #

方案1：最清晰易懂的常规写法

先正常收集得到tests列表，再把元素同步到others里——这是最符合编码规范，也最容易维护的方式：

final int range = 9;
List<String> help = new ArrayList<>();
for(int i = 5; i < range; i++) {
    help.add(String.valueOf(i + (i*2) + (i*(i+2))));
}
List<Test> others = new LinkedList<>();

// 先正常处理并收集tests
List<Test> tests = help.stream()
    .map(s -> new Test(s, 10))
    .peek(t -> System.out.println(t.getText())) // 这里用peek做调试观察是完全合规的
    .collect(Collectors.toList());

// 再同步到others集合
others.addAll(tests);

方案2：Java 12+可用的teeing收集器（适合需要一次流水线处理多个集合的场景）

如果你的项目用的是Java 12及以上版本，可以用Collectors.teeing()，在一次Stream流处理中同时收集到两个集合，避免两次遍历：

final int range = 9;
List<String> help = new ArrayList<>();
for(int i = 5; i < range; i++) {
    help.add(String.valueOf(i + (i*2) + (i*(i+2))));
}
List<Test> others = new LinkedList<>();

List<Test> tests = help.stream()
    .map(s -> new Test(s, 10))
    .peek(t -> System.out.println(t.getText()))
    .collect(Collectors.teeing(
        Collectors.toList(), // 第一个收集器：生成tests列表
        Collectors.toCollection(LinkedList::new), // 第二个收集器：生成临时的others列表
        (testsList, tempOthers) -> {
            others.addAll(tempOthers); // 把临时列表的元素加到你的原others集合
            return testsList;
        }
    ));

最后再强调下Stream的编码规范 #

• Stream API的核心是无副作用的函数式编程，所有中间操作（比如map、filter）都应该尽量避免修改外部状态；
• peek()的唯一合理用途就是调试：比如打印元素、查看元素的属性值，绝对不要用它来做业务逻辑的修改、收集操作；
• 如果需要将元素同步到多个集合，优先选择"先收集一个集合，再同步"的方式，逻辑清晰且不易出错。

内容的提问来源于stack exchange，提问作者TSH