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

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

我假设他们要求您保留给定宠物 Collection 的不可变副本。

this.petNames = List.copyOf(p);

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

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

您不应该为 ls 会发生什么而烦恼，但您应该注意您可以用 p1 做什么。

您制作一份副本以不再担心输入是否会在某个时候更改。

您使集合不可变，这与 2 个最后的普通字段一起使类不可变。