Java更新HashMap引用值时所有元素products被覆盖为相同值问题求助

• 最核心的常见原因：构造入参casesMap时，所有key对应的ExportDto value都是同一个对象的引用，你修改任意一个value的products属性，所有指向该对象的value都会同步变更，表现出来就是所有value的products完全相同。
• 第二版代码存在变量名拼写错误：调用service返回的产品列表变量名为customerProducts，但后续过滤时使用了未定义的products变量，若代码实际可运行则大概率是你贴代码时的笔误，但该错误会导致编译不通过或用了全局的products列表。
• 原有实现性能问题：遍历每个key时都全量扫描产品列表过滤，时间复杂度为O(n*m)，数据量大时效率极低，且重复过滤容易出现逻辑误差。

1. 先修正casesMap的构造逻辑，确保每个key对应独立的ExportDto实例
   反例（错误写法）：

ExportDto commonDto = new ExportDto();
for (String cin : cinList) {
    // 所有key都复用同一个ExportDto对象
    casesMap.put(cin, commonDto);
}

正例（正确写法）：

for (String cin : cinList) {
    // 每个key对应独立的新对象
    ExportDto dto = new ExportDto();
    casesMap.put(cin, dto);
}

2. 优化populateProducts的实现逻辑，提前按cin分组减少重复遍历，同时修正变量名问题：

@Transactional(readOnly = true)
public void populateProducts(Map<String, ExportDto> casesMap, List<String> cins) {
  try {
    List<ProductDto> customerProducts = service.retrieveProductsByCins(cins);
    // 提前按cin转小写后分组，一次遍历完成分组，后续查询时间复杂度为O(1)
    Map<String, List<ProductDto>> cin2ProductsMap = customerProducts.stream()
            .filter(p -> p.getCin() != null)
            .collect(Collectors.groupingBy(p -> p.getCin().toLowerCase(), Collectors.toList()));
    
    // 遍历Map直接给每个ExportDto设置对应产品
    casesMap.forEach((cinKey, exportDto) -> 
        exportDto.setProducts(cin2ProductsMap.getOrDefault(cinKey.toLowerCase(), Collections.emptyList()))
    );
  } catch (CustomerGenericException e) {
    log.error("查询用户产品信息异常", e);
  }
}

在populateProducts方法入口打调试断点，查看casesMap中各个value的对象内存地址，若地址完全一致则可确认是构造Map时复用了同一个ExportDto对象的问题。

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