浅拷贝
在stream中未使用map等方式，此时是浅拷贝
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 这个数据的
for (EmapMachineDTO vo : machineVOList) { if (vo.getMachineId() != null) { ..... vo.setStatus(statusEnum.getCode()); vo.setColor(statusEnum.getColor()); } }问题分析
你观察得很准确。原因在于distinctList中的对象和machineVOList中的对象是同一个引用，修改其中一个就会影响到另一个。
为什么会这样？
看这段去重逻辑：
List<EmapMachineDTO> distinctList = machineVOList.stream() .collect(Collectors.toMap( dto -> buildDedupKey(...), Function.identity(), (existing, replacement) -> existing )) .values() .stream() .collect(Collectors.toList());Function.identity()直接使用的就是machineVOList里的对象本身，没有做任何拷贝。所以最终distinctList只是原列表中某些元素的引用集合。
后续的循环：
for (EmapMachineDTO vo : machineVOList) { vo.setStatus(code); vo.setColor(color); }遍历的是同一个对象集合，当然就会把状态和颜色写到distinctList里包含的那些对象上（因为它们是同一个实例）。
这是问题吗？
从当前逻辑看，很可能正是你想要的效果——你最终输出的是distinctList（resp.setEmapMachineList(distinctList)），所以让去重后的数据也带上状态是正确的。但如果你的设计期望是两个列表互相独立，那就会出现副作用。
何避免（如果需要独立）？
方案1：拷贝后去重
在去重前就先做一份浅拷贝，例如：
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后遍历它：
for (EmapMachineDTO distinct : distinctList) { if (distinct.getMachineId() != null) { // 从 statusMap 中获取状态并设置到 distinct 上 } }【×会影响到原数据，我自己亲测，原数据就会带上distinctList遍历时设置的数据】这样既避免了对原列表的副作用，逻辑也更清晰。
额外提醒
当前循环里可能出现一个machineId对应多条不同 key 的记录，它们会拿到同一个status，这看起来是符合业务假设的。但要注意statusMap如果缺失某个machineId，你用了randomWeightedStatus()作为默认值，这可能会导致同一个machineId在不同的调用里出现不同的随机状态（如果去重后有多条记录，它们的状态也可能不一致）。如果这是临时测试代码（// todo 暂时写死），上线前别忘记替换回真实数据。