使用Collections.addAll还是Arrays.asList生成List，当修改原始数据时，均会影响到List里的数据

感谢作者能翻译并且分享这么优质的文章。不过文章中有一处表述欠妥：

如果希望避免这两个坑，请改用这个方式
Collections.addAll(arraylist, array);

从字面上理解，作者的意思应该是指避免上述的两个坑，定长 && 修改原数据会影响list里的对象。
欠妥的主要是后者：修改原数据，list中的值也会变

环境：

java version "1.8.0_25"
Java(TM) SE Runtime Environment (build 1.8.0_25-b18)
Java HotSpot(TM) 64-Bit Server VM (build 25.25-b02, mixed mode)

---

Collections.addAll方法的源码如下所示：

@SafeVarargs
public static <T> boolean addAll(Collection<? super T> c, T... elements) {
    boolean result = false;
    for (T element : elements)
        result |= c.add(element);
    return result;
}

---

Arrays.asList方法相关源码如下所示：

@SafeVarargs
@SuppressWarnings("varargs")
public static <T> List<T> asList(T... a) {
    return new ArrayList<>(a);// 内部私有类
}

ArrayList(E[] array) {
    a = Objects.requireNonNull(array);
}

public static <T> T requireNonNull(T obj) {
    if (obj == null)
        throw new NullPointerException();
    return obj;
}

---

从源码来看，不管是Collections.addAll，还是Arrays.asList，均是引用传递，修改source object会影响到list里的target object。下面是我测试的代码：

public static void main(String[] args) {
    User user_1 = new User();
    user_1.put("name", "user_1");
    user_1.put("value", "v_1");

    User user_2 = new User();
    user_2.put("name", "user_2");
    user_2.put("value", "v_2");

    List<User> asList = Arrays.asList(user_1, user_2);
    List<User> arraylist = new ArrayList<>();
    Collections.addAll(arraylist, user_1, user_2);

    user_1.put("value", "v_a_1");
    user_2.put("value", "v_a_2");

    System.out.println("-------- asList --------");
    System.out.println(JsonKit.toJson(asList));
    System.out.println("\n-------- arraylist --------");
    System.out.println(JsonKit.toJson(arraylist));
}

运行结果

-------- asList --------
[{"name":"user_1","value":"v_a_1"},{"name":"user_2","value":"v_a_2"}]

-------- arraylist --------
[{"name":"user_1","value":"v_a_1"},{"name":"user_2","value":"v_a_2"}]

---

结论，不管是Collections.addAll还是Arrays.asList，修改原始数据时，均会影响到List里的数据

原帖地址