遍历 map 时若需同时访问键与值，应直接使用 `entryset()` 迭代，避免通过 `keyset()` 遍历后再调用 `get()` 查找值——后者会带来冗余哈希查找开销，降低性能且触发 sonarqube 等工具的代码异味警告（rspec-2864）。
在 Java 中，Map 接口提供了两种主流遍历方式：keySet() 和 entrySet()。表面上看，二者都能完成遍历任务；但从性能和语义清晰度角度，当循环体内需要访问键对应的值时，entrySet() 是更优、更推荐的选择。
为什么 keySet + get 是低效的？
以原始代码为例：
for (String accounts : shiftDatesMap.keySet()) {
Set<String> shiftDates = shiftDatesMap.get(accounts); // ❌ 每次都触发一次哈希查找
// … 后续逻辑
}尽管 HashMap.get() 平均时间复杂度为 O(1)，但它仍需重新计算哈希值、定位桶位、处理可能的哈希冲突（如链表或红黑树查找）。在 keySet() 迭代中，JVM 已经在内部遍历了所有 Node（即键值对），却舍近求远地“丢弃”了现成的值，再额外发起一次查找——这属于典型的重复工作（redundant computation）。
正确做法：用 entrySet 直接解构键值对
改写为 entrySet() 遍历后，可一次性获取键与值，零成本访问：
for (Map.Entry<String, Set<String>> entry : shiftDatesMap.entrySet()) {
String accounts = entry.getKey(); // ✅ 直接获取键
Set<String> shiftDates = entry.getValue(); // ✅ 直接获取值，无额外开销
// 后续逻辑可直接使用 accounts 和 shiftDates
}该方式不仅提升性能（尤其在大数据量或高频调用场景下），还增强了代码可读性与意图表达：明确表明“我需要处理每一个键值对”，而非“我先取所有键，再逐个查值”。
实际应用示例（修复原始方法）
将您原方法中的低效循环替换如下：
public Set<Date> getAccountShiftDate(Map<String, Set<String>> shiftDatesMap,
List<Groups> shiftSchedule) {
Set<Date> accountShiftDatesTemplate = new HashSet<>();
// ✅ 使用 entrySet 替代 keySet
for (Map.Entry<String, Set<String>> entry : shiftDatesMap.entrySet()) {
String accounts = entry.getKey();
Set<String> shiftDates = entry.getValue(); // 值已就绪，无需 get()
Optional<Groups> shiftOptional = shiftSchedule.stream()
.filter(g -> StringUtils.equalsIgnoreCase(accounts, g.getLongName()))
.findFirst();
if (shiftOptional.isPresent()) {
// 基于 shiftDates 和 shiftOptional 进行后续业务处理...
// 例如：解析日期、添加到 accountShiftDatesTemplate 等
}
}
return accountShiftDatesTemplate;
}注意事项与补充建议
- ✅ 适用前提：仅当循环内确实需要值（value）或同时需要键与值时，才必须优先选用 entrySet()；若仅需键（如做存在性校验），keySet() 仍合理。
- ✅ 泛型安全：务必声明完整泛型类型 Map.Entry
，避免原始类型警告；Java 10+ 可用 var 简化（for (var entry : map.entrySet())），但需确保可读性不受损。 - ⚠️ 并发 Map 注意：ConcurrentHashMap.entrySet() 返回的 Entry 在迭代期间不保证强一致性（可能反映部分更新状态），但这是其设计使然，非本优化范畴问题。
- ? 工具提示意义：SonarQube 的 RSPEC-2864 不仅是性能建议，更是代码质量信号——它提示开发者关注数据结构访问模式是否符合直觉与最佳实践。
总之，entrySet() 是 Map 遍历的“黄金路径”。养成习惯：只要涉及键值协同操作，首选 entrySet() ——简洁、高效、专业。
