浅拷贝在stream中未使用map等方式此时是浅拷贝ListEmapMachineDTO 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中的对象是同一个引用修改其中一个就会影响到另一个。为什么会这样看这段去重逻辑ListEmapMachineDTO 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里包含的那些对象上因为它们是同一个实例。这是问题吗从当前逻辑看很可能正是你想要的效果——你最终输出的是distinctListresp.setEmapMachineList(distinctList)所以让去重后的数据也带上状态是正确的。但如果你的设计期望是两个列表互相独立那就会出现副作用。何避免如果需要独立方案1拷贝后去重在去重前就先做一份浅拷贝例如ListEmapMachineDTO 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 暂时写死上线前别忘记替换回真实数据。
浅拷贝在stream中未使用map等方式此时是浅拷贝ListEmapMachineDTO 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中的对象是同一个引用修改其中一个就会影响到另一个。为什么会这样看这段去重逻辑ListEmapMachineDTO 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里包含的那些对象上因为它们是同一个实例。这是问题吗从当前逻辑看很可能正是你想要的效果——你最终输出的是distinctListresp.setEmapMachineList(distinctList)所以让去重后的数据也带上状态是正确的。但如果你的设计期望是两个列表互相独立那就会出现副作用。何避免如果需要独立方案1拷贝后去重在去重前就先做一份浅拷贝例如ListEmapMachineDTO 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 暂时写死上线前别忘记替换回真实数据。
