非ArrayList/LinkedList的List执行addAll后清空原列表会丢失元素吗？

标准JDK实现下完全安全 #

你提到的ArrayList、LinkedList等JDK自带的List实现，这段逻辑100%符合预期，可以通过代码评审。

核心原理 #

Java集合的操作都是基于引用值传递，两个关键行为的定义非常明确：

• addAll的接口契约要求：将指定集合当前时刻的所有元素，按迭代顺序追加到当前列表。JDK标准实现都会遍历传入集合的迭代器，把每个元素的引用复制一份存入自身的内部存储，不会直接绑定原集合的内部结构。
• clear()的作用：仅清空当前列表自身持有的引用集合（将内部存储的引用置为null、重置size为0），不会销毁实际的元素对象。

你给出的测试逻辑里，newList.addAll(original)执行完成后，newList已经持有了所有MyObject实例的独立引用副本，original.clear()只会清空original自己的引用，完全不会影响newList中的数据。哪怕主动触发GC，只要newList还持有这些实例的强引用，实例就不会被回收，遍历断言不会出现任何问题。

仅有的风险边界 #

只有当你使用违反List接口契约的特殊自定义实现时，才可能出现问题：比如少数为特殊场景设计的视图类List，addAll时没有复制元素引用，而是直接关联原集合的内部存储，此时原集合的修改会同步到新列表。但这类实现都会在文档中明确标注特殊行为，常规业务开发几乎不会遇到，也不属于通用规范的覆盖范围。

代码评审建议 #

• 如果项目中仅使用JDK标准List实现，或所有用到的第三方List实现都遵守addAll的标准契约，这段代码完全合理，没有问题。
• 如果要进一步提升可维护性，可以加一行注释说明此处依赖addAll复制引用的标准行为即可。
• 注意：只有当你后续需要修改原MyObject实例，且不希望修改同步到newList时，才需要对元素做深拷贝，你的场景只是清空原列表，完全不需要额外的深拷贝操作。

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