stream流浅拷贝

浅拷贝

在stream中未使用map等方式，此时是浅拷贝

java 复制代码

        List<EmapMachineDTO> distinctList = machineVOList.stream()
                .collect(Collectors.toMap(
                        dto -> buildDedupKey(dto.getDepartment(),
                                dto.getPhase(),
                                dto.getMachineId(),
                                dto.getModuleSec()),
                        Function.identity(),
                        (existing, replacement) -> existing
                ))
                .values()
                .stream()
                .collect(Collectors.toList());

后续对原来的 machineVOList 数据进行操作，是会影响到前面的 distinctList 这个数据的

java 复制代码

        for (EmapMachineDTO vo : machineVOList) {
            if (vo.getMachineId() != null) {
                .....  
                vo.setStatus(statusEnum.getCode());
                vo.setColor(statusEnum.getColor());
            }
        }

问题分析

你观察得很准确。原因在于 distinctList 中的对象和 machineVOList 中的对象是同一个引用，修改其中一个就会影响到另一个。

为什么会这样？

看这段去重逻辑：

java 复制代码

List<EmapMachineDTO> distinctList = machineVOList.stream()
        .collect(Collectors.toMap(
                dto -> buildDedupKey(...),
                Function.identity(),
                (existing, replacement) -> existing
        ))
        .values()
        .stream()
        .collect(Collectors.toList());

Function.identity() 直接使用的就是 machineVOList 里的对象本身，没有做任何拷贝。所以最终 distinctList 只是原列表中某些元素的引用集合。

后续的循环：

java 复制代码

for (EmapMachineDTO vo : machineVOList) {
    vo.setStatus(code);
    vo.setColor(color);
}

遍历的是同一个对象集合，当然就会把状态和颜色写到 distinctList 里包含的那些对象上（因为它们是同一个实例）。

这是问题吗？

从当前逻辑看，很可能正是你想要的效果 ------你最终输出的是 distinctList（resp.setEmapMachineList(distinctList)），所以让去重后的数据也带上状态是正确的。但如果你的设计期望是两个列表互相独立，那就会出现副作用。

何避免（如果需要独立）？

方案1：拷贝后去重

在去重前就先做一份浅拷贝，例如：

java 复制代码

List<EmapMachineDTO> distinctList = machineVOList.stream()
        .map(dto -> {
            EmapMachineDTO copy = new EmapMachineDTO();
            BeanUtils.copyProperties(dto, copy); // 或者其他拷贝方式
            return copy;
        })
        .collect(Collectors.toMap(...))
        ...

这样修改 machineVOList 就不会影响 distinctList，反之亦然。

【×会影响】方案2：直接在 distinctList 上设置状态

如果你只需要 distinctList 带状态，那可以不修改 machineVOList，而是在得到 distinctList 后遍历它：

java 复制代码

for (EmapMachineDTO distinct : distinctList) {
    if (distinct.getMachineId() != null) {
        // 从 statusMap 中获取状态并设置到 distinct 上
    }
}

**【×会影响到原数据，我自己亲测，原数据就会带上distinctList遍历时设置的数据】**这样既避免了对原列表的副作用，逻辑也更清晰。

额外提醒

当前循环里可能出现一个 machineId 对应多条不同 key 的记录，它们会拿到同一个 status，这看起来是符合业务假设的。但要注意 statusMap 如果缺失某个 machineId，你用了 randomWeightedStatus() 作为默认值，这可能会导致同一个 machineId 在不同的调用里出现不同的随机状态（如果去重后有多条记录，它们的状态也可能不一致）。如果这是临时测试代码（// todo 暂时写死），上线前别忘记替换回真实数据。