遍历map时应优先使用entryset()而非keyset()，避免在循环中重复调用get()造成冗余哈希查找，提升性能并符合sonarqube等静态分析工具的规范要求。
在Java开发中，当需要同时访问Map的键（key）和值（value）时，一个常见但低效的写法是先遍历keySet()，再在循环体内通过map.get(key)获取对应值。这种模式看似直观，实则隐含性能损耗：每次get()调用都会触发一次哈希计算与桶内查找（即使底层是HashMap，平均时间复杂度为O(1)，但常数开销不可忽略），尤其在大数据量或高频调用场景下会累积明显开销。
更优的方式是直接遍历entrySet()——它返回的是Map.Entry<K,V>集合，每个Entry对象天然封装了键值对，可通过getKey()和getValue()零成本访问，无需额外查找。
以您提供的代码为例：
// ❌ 低效：keySet + get → 触发N次哈希查找
for (String accounts : shiftDatesMap.keySet()) {
Set<String> shiftDates = shiftDatesMap.get(accounts); // 不必要！
// 后续逻辑...
}应重构为：
deep-java-review
下载
Java项目代码review工具。分析Git变更+完整调用链路上下文，推断业务需求，进行多维度评分和分类汇总，生成完整PRD文档。包含细粒度Java代码审查清单（Null安全、异常处理、Streams、并发、equals/hashCode、资源管理、API设计、性能、MyBatis/ORM、事务边界、SQL/DD...
// ✅ 高效：entrySet → 键值对一步到位
for (Map.Entry<String, Set<String>> entry : shiftDatesMap.entrySet()) {
String accounts = entry.getKey();
Set<String> shiftDates = entry.getValue(); // 直接获取，无额外开销
// 后续逻辑（如匹配Groups、填充accountShiftDatesTemplate等）可在此继续
}若结合Stream API，亦可写作：
shiftDatesMap.entrySet().stream()
.forEach(entry -> {
String accounts = entry.getKey();
Set<String> shiftDates = entry.getValue();
// 处理逻辑...
});⚠️ 注意事项：
- 仅当确实需要值（value）时才适用此优化；若仅需键（如做存在性校验），keySet()仍是合理选择；
- entrySet()返回的Entry对象是Map的内部视图，修改Entry的value（如entry.setValue(newVal)）可能影响原Map（取决于具体Map实现，HashMap支持）；
- SonarQube规则RSPEC-2864正是基于此原理检测“低效键值访问”，修复后不仅消除代码异味，也体现对数据结构特性的正确理解。
综上，养成优先使用entrySet()的习惯，既是性能敏感型编码的体现，也是专业Java工程实践的重要一环。
