为什么这个类不是不可变的？

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. 您使集合不可变，这与 2 个最后的普通字段一起使类不可变。

Answer 2

为了使列表不可变，我相信你应该使用

       this.petnames=  ImmutableList.copyOf(p);

或者

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

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

 this.petnames = List.copyOf(p)

如果您返回它，则无法向其中添加或删除元素。但是，它们可以从中获取元素。