使用addRandomValuesToList生成新Map时部分列表为空的问题排查

你的外层遍历处理Map的思路完全没问题，新Map中出现空列表的问题，100%出在addRandomValuesToList方法的内部实现上——外层循环的逻辑是合理的：遍历原TreeMap的每个键值对，调用方法处理对应的List，再将结果存入新的TreeMap。

可能导致空列表的常见原因 #

以下是addRandomValuesToList方法里最容易踩的坑：

• 没有正确执行添加元素的逻辑：比如循环次数不足（比如只循环了3次而不是4次），或者生成的随机索引无效，导致无法从输入List中取出元素添加到新列表。
• 未处理输入List的边界情况：虽然你的原Map中每个List都有10个元素，但如果方法里没有判断输入List是否为空/长度不足，极端情况下可能返回空列表（不过你的场景里暂时不会触发，但这是潜在问题）。
• 异常未捕获：比如生成随机索引时出现越界异常，导致方法提前终止，返回未添加任何元素的空列表。

正确的addRandomValuesToList实现示例 #

这里给你一个安全可靠的实现，确保能从输入List中随机选取4个元素（允许重复选取，和你给出的示例结果一致）：

import java.util.ArrayList;
import java.util.List;
import java.util.concurrent.ThreadLocalRandom;

public static List<Integer> addRandomValuesToList(List<Integer> input) {
    // 先判断输入List是否有效，避免空指针或长度不足的问题
    if (input == null || input.size() < 1) {
        throw new IllegalArgumentException("Input list cannot be empty or null");
    }
    
    List<Integer> randomValues = new ArrayList<>();
    ThreadLocalRandom random = ThreadLocalRandom.current();
    
    // 循环4次，每次随机选取一个元素添加到新列表
    for (int i = 0; i < 4; i++) {
        // 生成0到输入List长度-1之间的随机索引
        int randomIndex = random.nextInt(input.size());
        randomValues.add(input.get(randomIndex));
    }
    
    return randomValues;
}

对你的外层代码的小优化 #

你的外层代码里newList没有声明类型，虽然在Java 8+的隐式类型推断下可以运行，但显式声明类型会让代码更清晰：

Map<String, List<Integer>> test = new TreeMap<>();
for (Map.Entry<String, List<Integer>> entry : map.entrySet()) {
    List<Integer> newList = addRandomValuesToList(entry.getValue());
    test.put(entry.getKey(), newList);
}

用上面的方法替换你原来的addRandomValuesToList实现后，新Map里的列表就不会再出现空的情况了。

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