Java Stream API：为何用reduce填充集合不被推荐？

你的代码用reduce操作可变HashSet的做法，违背了Stream API的设计契约，属于不良代码设计，且在并行流场景下会触发严重的底层问题。下面具体拆解：

1. reduce的核心语义是「纯函数式归约」 #

reduce的设计目标是基于输入元素生成新的结果值，它要求累加器函数是无副作用的——即不能修改外部对象，也不能依赖外部状态。你的代码中，累加器直接修改了外部传入的logins集合，这属于明显的副作用操作，完全偏离了reduce的语义。

官方文档明确说明：reduce的累加器应该是一个纯函数，每次调用都基于输入生成新值，而不是修改已有对象。

2. 并行流场景下必出问题 #

你的代码在串行流中可能「看起来正常」，但一旦切换为并行流（stream.parallel()），会立刻暴露两个致命问题：

• 并发修改异常：多个线程会同时操作同一个logins集合，触发ConcurrentModificationException；
• 结果错误：你的combiner逻辑存在bug——s1.addAll(s2)后返回s2，而非合并后的s1，并行时会导致部分线程的结果丢失，最终输出不完整。

3. collect才是可变容器累积的正确选择 #

Collectors工具类和collect方法是专门为可变容器的安全累积设计的：

• Supplier负责为每个线程创建独立的容器实例（并行时避免共享状态）；
• Accumulator安全修改当前线程的容器；
• Combiner正确合并多个线程的容器结果。

你的需求用collect实现既简洁又安全：

Set<String> logins = stream.map(User::getLogin).collect(Collectors.toSet());

关于「不可变对象累加」的疑问 #

不是必须在不可变对象上执行reduce，但如果要用reduce处理可变对象，必须保证累加器每次返回新的容器实例（纯函数式操作），比如：

Set<String> logins = stream.reduce(new HashSet<>(), 
    (acc, u) -> {
        Set<String> newSet = new HashSet<>(acc);
        newSet.add(u.getLogin());
        return newSet;
    },
    (s1, s2) -> {
        Set<String> newSet = new HashSet<>(s1);
        newSet.addAll(s2);
        return newSet;
    });

但这种方式效率极低（每次创建新集合），完全没有必要——collect就是为这种场景优化的，用reduce属于舍近求远。

总结 #

你的代码属于不良设计，违反了Stream API的语义约定，在并行场景下会触发底层并发问题。可变容器的累积操作请优先使用collect，reduce应留给纯函数式的归约场景（比如求和、求最大值等）。

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