为什么这个类不可变？

Question

今天我参加了一个Java面试，我被要求创建一个名为Person的不可变类，并给了我一个骨架，里面有一些参数：年龄、姓名等。我创建了下面的类。

final class Person {
  private final int age;
  private final String name;
  private final List<String> petNames;


  public Person(int a, String n, List<String> p) {
    this.age = a;
    this.name = n;
    this.petNames = p;
  }

  int getAget() {
    return this.age;
  }

  String getName() {
    return this.name;
  }

  List<String> getPetnames() {
    return this.petNames;
  }

}

我被告知这并不完整，因为通过执行下面的代码序列，你可以证明它不是不可变的。

int x = 3;
String name = "Alex";
List<String> ls = new ArrayList<>();
Person p1 = new Person(x, name, ls);

我得到的提示是，它与列表有关，我应该在构造函数中改变一些东西。但是我真的不知道为什么，我也不太明白哪里出了问题。

谁能给我解释一下，应该添加什么，为什么？也许我没有完全理解不可变性。

Answer 1

我假设他们要求你为给定的宠物集合保留一个不可变的副本。

this.petNames = List.copyOf(p);

否则，你仍然可以修改一个人的宠物。

p1.getPetnames().add("newPet");

你不应该为以下情况的发生而烦恼 ls但你应该注意你能做的事情。p1.

1. 你做了一个副本，不再担心输入是否会在以后的某个时候被改变。
2. 你使集合不可变，加上最后的两个普通字段，使类不可变。

Answer 2

要使 list 不可变，我相信你应该使用

       this.petnames=  ImmutableList.copyOf(p);

或

     this.petnames  = Collections.unmodifiableList(new ArrayList<>(p));

在java 9中，你可以使用copyOf()方法来创建不可变的列表。

 this.petnames = List.copyOf(p)

如果你返回这个，那么它不可能添加或删除元素。但是，他们可以从其中获取元素。