为什么这些 Java 枚举的值会改变？

Question

我在根据 enum 上的条件创建对象列表时遇到一些问题。 看来我完成列表后，列表中的每一项都相当于最后一项。

这是不同引用指向同一对象的经典情况，但我不知道如何避免它：

我在保持可读性的同时尽可能地减少了内容：

public class Foo {
  Digit[] array = new Digit[2];
  ArrayList<Foo> foozlets;

  Foo() {
    array[0] = Digit.ZERO;
    foozlets = new ArrayList<Foo>();
  }

  Foo(Foo old, Digit num) {
    this.array = old.array;  \\This line is a problem, what should it say?
    array[1] = num;
  }

  public static void main(String[] args) {
    Foo f = new Foo();
    System.out.println("Initial Foo:");
    System.out.println(f);
    f.listFoozlets();
  }

  void listFoozlets() {
    for (Digit k : Digit.values()) {
      if (k == Digit.TWO || k == Digit.FIVE) {
        foozlets.add(new Foo(this, k));
        System.out.println("** Foozlet being added **");
        Foo foo = new Foo(this, k);
        System.out.println(foo);
      }
    }
    System.out.println("** List of Foozlets **");
    for (Foo foo : foozlets) {
        System.out.println(foo);
    }
  }

  public String toString() {
    return array[0].toString() + " " + array[1].toString();
  }
}

enum Digit { ZERO, ONE, TWO, THREE, FOUR, FIVE }

这是输出：

Initial Foo:
ZERO NULL
** Foozlet being added **
ZERO TWO
** Foozlet being added **
ZERO FIVE
** List of Foozlets **
ZERO FIVE
ZERO FIVE

如果有人可以解释为什么列表中 Foo 的第一个实例发生变化，以及如何制作一个不改变的列表，我将不胜感激。

编辑：好的，我知道问题出在哪里了。 在真正的、更大的程序中，我有一个更大的数组，并且我想在为列表创建新的 Foo 时保留旧信息。 我更改了代码以反映有我想要维护的其他信息。 我该如何实现这个目标？

编辑 2021： 十二年后，我想到了这个，这是我的第一个 SO 问题。 如果这对您有帮助，我很高兴。 如果你想知道我到底想做什么，答案是“我不知道”。

Answer 1

这一点就是罪魁祸首：

Foo(Foo old, Digit num) {
  this.array = old.array;
  array[0] = num;
}

您将引用复制到旧的 Foo 数组，然后更改该数组中的值。

为什么你有一个大小为 1 的数组而不是一个 Digit ？ 如果您确实想要该数组，您可能想要克隆它，而不仅仅是复制引用，但我们无法真正判断其意图是什么。

这是一个较短的示例：

enum Digit { ZERO, ONE, TWO, THREE, FOUR, FIVE }

public class Foo {
  Digit[] array = new Digit[1];

  Foo() {
    array[0] = Digit.ZERO;
  }

  Foo(Foo old, Digit num) {
    this.array = old.array;
    array[0] = num;
  }

  public String toString() {
    return array[0].toString();
  }

  public static void main(String[] args) {
    Foo f = new Foo();
    System.out.println(f);
    Foo other = new Foo(f, Digit.ONE);
    System.out.println(f);
  }
}

Answer 2

在第二个构造函数中：

Foo(Foo old, Digit num) {
  this.array = old.array;
  array[0] = num;
}

您正在重复使用 old 中的列表。 您想要创建该列表的副本而不是使用相同的列表。 您可以通过将分配更改为：

Foo(Foo old, Digit num) {
  this.array = new ArrayList<Foo>(old.array);
  array[0] = num;
}

Answer 3

问题似乎出在这一行：

    this.array = old.array;

您正在共享数组引用，因此每个 Foo 共享相同的数组，因此它们在 array[0] 处都有相同的值。

要解决此问题，请尝试：

  this.array = old.array.clone();

Answer 4

本例中的对象引用是数组。 foo 的构造函数是您遇到问题的地方，具体来说：

Foo(Foo old, Digit num) {
   this.array = old.array; // reference the old array 
   array[0] = num; // set the first element of the array (for this AND the old array) to num
}

您需要将数组复制到 foo 构造函数中的新数组。

Answer 5

而不是

Foo(Foo old, Digit num) {
  this.array = old.array;
  array[0] = num;
}

简单地执行

Foo(Foo old, Digit num) {
  array[0] = num;
}

如果您确实需要数组， ...
（最好是使用另一个回复中建议的简单数字）