报错内容
报错信息显示ID重复。
原因分析
在最终添加的方法中，出现了重复ID的报错。对于为什么ID会重复，我思考了很久也没有想通。我沿着代码往上找到了saveList。
int batchSize = 5000; if (CollUtil.isNotEmpty(saveList)) { List<List<Map<String, Object>>> batchList = ListUtil.partition(saveList, batchSize); for (List<Map<String, Object>> batch : batchList) { if (CollUtil.isNotEmpty(batch)) { firstRecordMapper.insertJghzLdryList(batch.stream().filter(v -> v != null && ObjectUtil.isNotEmpty(v.get("ID"))).collect(Collectors.toList())); } } }
代码大致如上所示。
for (Map<String, Object> item : list) { ... Map<String, Object> oneCellMap = new HashMap<String, Object>(); // 遍历这一类型的10个二级指标，每个指标存成一行数据 for (int i = 0; i < secIndexNameList.size(); i++) { ... oneCellMap.clear(); oneCellMap.put("ID", IdUtilV2.getInstance().getId()); ... saveList.add(oneCellMap); px++; } // 保存这一行数据的 综合得分 oneCellMap.clear(); oneCellMap.put("ID", IdUtilV2.getInstance().getId()); ... saveList.add(oneCellMap); }
我也不是很确定，但我怀疑问题出在这里，于是我做了以下修改：
for (Map<String, Object> item : list) { ... // 遍历这一类型的10个二级指标，每个指标存成一行数据 for (int i = 0; i < secIndexNameList.size(); i++) { ... Map<String, Object> oneCellMap = new HashMap<String, Object>(); oneCellMap.put("ID", IdUtilV2.getInstance().getId()); ... saveList.add(oneCellMap); px++; } // 保存这一行数据的 综合得分 Map<String, Object> oneCellMap = new HashMap<String, Object>(); oneCellMap.put("ID", IdUtilV2.getInstance().getId()); ... saveList.add(oneCellMap); }
然后问题就解决了。我感到很奇怪，后来才反应过来...原来是使用了同一个对象。
解决方案
问题的根本原因是在循环中重复使用了同一个oneCellMap对象，导致了ID重复的问题。在第一个版本的代码中，每次循环都会清空oneCellMap对象并重新添加ID，但是由于saveList中保存的是引用，所以最终saveList中的所有oneCellMap对象的ID都是相同的。
在修改后的代码中，每次循环都创建了一个新的oneCellMap对象，确保了每个oneCellMap对象的ID都是唯一的，解决了ID重复的问题。
