列表迭代性能优化咨询：如何避免三次遍历列表实现LinkedHashMap有序插入

嘿，这个问题我太熟了——重复遍历列表确实是性能杀手，尤其是当categories数据量上去之后，三次遍历完全没必要。咱们把它改成只遍历一次的实现，既保持原逻辑不变，又能提升效率！

核心思路 #

先把categories列表一次性遍历完，构建一个临时的映射表（key是统一大小写的分类编码，value是对应的处理后布尔值），之后直接从这个映射表里取我们需要的三个分类的值，不用再碰原列表。

优化后的代码 #

public void prepareData(final List<Category> categories, final Map<String, Boolean> preferencesMap) {
    // 遍历一次列表，构建临时映射，只保留第一个符合条件的分类数据
    Map<String, Boolean> categoryLookup = new HashMap<>();
    for (Category category : categories) {
        String code = category.getCode();
        // 匹配原逻辑：code不为空且isActive不为null时才处理
        if (code != null && category.isActive() != null) {
            String standardizedCode = code.toUpperCase();
            // 只存第一个符合条件的，和原方法中"找到第一个就返回"的逻辑一致
            if (!categoryLookup.containsKey(standardizedCode)) {
                categoryLookup.put(standardizedCode, !category.isActive());
            }
        }
    }

    // 直接从映射表取值，不存在则返回默认的FALSE
    preferencesMap.put("ACCESSORIES", categoryLookup.getOrDefault("ACCESSORIES", Boolean.FALSE));
    preferencesMap.put("WATCHES", categoryLookup.getOrDefault("WATCHES", Boolean.FALSE));
    preferencesMap.put("PHONES", categoryLookup.getOrDefault("PHONES", Boolean.FALSE));
}

为什么这个方案更好？ #

1. 性能提升：只遍历categories一次，不管后续需要多少个分类的值，都不会重复遍历。数据量越大，提升越明显。
2. 逻辑完全一致：和你原来的代码逻辑完全对齐——只取第一个编码匹配（忽略大小写）且isActive不为null的分类的!isActive值，找不到则返回FALSE。
3. 代码更简洁：去掉了重复的方法调用和循环，可读性更强。

额外说明 #

• 关于LinkedHashMap的顺序：你只需要确保传入的preferencesMap是LinkedHashMap实例，我们按ACCESSORIES→WATCHES→PHONES的顺序put，就能保持插入顺序，和原代码的效果一致。
• 空值处理：加了code != null的判断，避免toUpperCase()抛出空指针异常，和原代码中val.equalsIgnoreCase(null)返回false的逻辑一致。

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