Collections.fill(list, obj) 填充的是同一对象引用而非副本,导致多个位置共享同一实例,修改一处影响全局;安全做法是逐个新建对象、用Stream.generate或确保obj不可变。

Collections.fill(list, obj) 看似简单,但若填充的是可变对象引用,极易引发隐性共享问题——多个位置指向同一对象实例,一处修改,处处生效。
填充的是引用,不是副本
fill() 不会为每个位置创建新对象,而是把同一个 obj 的引用复制到 list 的每个索引上。例如:
List<list>> rows = new ArrayList(Collections.nCopies(3, new ArrayList())); Collections.fill(rows, new ArrayList()); // 所有元素都指向同一个 ArrayList 实例</list>
此时 rows.get(0) == rows.get(1) == rows.get(2) 为 true。后续执行:
你接手一个需求时,是不是经常这样:"改一个小字段 → 结果把不相关的代码也改了"、"需求说得很模糊 → 做着做着发现跑偏了"、"数据库加个字段 → 上线才发现漏了数据兼容"、"做了个新功能 → 一个月后自己都忘了当时怎么设计的"、"写到一半被中断 → 再回来不知道从哪继续";这个 Skill 就是专门解决这些问题的。 它会自动探测你的项目环境(JDK、Spring Boot、ORM、数据库、前端框架),记在
rows.get(0).add("A");
rows.get(1).add("B");
结果是:三个子列表都包含 ["A", "B"] —— 因为它们根本就是同一个对象。
常见高危场景
- 初始化二维结构(如
List<list>></list>)时直接fill空列表 - 用
fill(list, new Date())初始化时间戳列表 → 所有元素共享同一Date实例 - 填充自定义 POJO(如
fill(list, new User()))→ 所有索引共用一个User对象
安全替代方案
- ✅ 逐个新建对象填充(推荐):
List<list>> rows = new ArrayList(); for (int i = 0; i ()); // 每次 new 一个新实例 }</list>
- ✅ 使用
Stream.generate()(Java 8+):List<list>> rows = Stream.generate(ArrayList::new) .limit(3) .collect(Collectors.toList());</list> - ✅ 若必须用
fill,确保obj是不可变类型(如String,Integer,LocalDateTime),或本身设计为只读
判断是否安全的小技巧
问自己:obj 是否会被后续代码调用其可变方法(如 add(), set(), clear(), setTime())?
如果是 → fill() 就不安全;
如果只是读取或作为标识符(如状态码 "PENDING")→ 通常无风险。
不复杂但容易忽略
Java免费学习笔记:立即使用
解锁 Java 大师之旅:从入门到精通的终极指南










