使用构造函数复制对象，Java

Question

假设我想制作一个对象的深层副本，但使用它的构造函数。所以我有：

public class PositionList {
    private Position[] data = new Position[0];
    private int size = 0;

public PositionList(PositionList other, boolean deepCopy) {
    if (deepCopy==true){
        size=other.getSize();
        for (int i=0;i<data.length;i++)
        data[i]=other.data[i];
    }
    else {
        data=other.data;
        size = other.size;

所以说我有这个被称为：

PositionList list = new PositionList();
PositionList acopy = new PositionList(list, true);

然而，我正在做的事情是不正确的，我不确定为什么..

Answer 1

问题在于您的深度复制逻辑：

size=other.getSize();
for (int i=0;i<data.length;i++)
    data[i]=other.data[i];

您正在设置 size 字段（这与 data 数组是多余的），但没有将新数组分配给 data 字段，这可能是“深层”副本的全部要点。您应该将 data 初始化为对方的 size （或 other.data.length）：（

data = new Position[other.data.length];
for (int i=0;i<data.length;i++)
    data[i]=other.data[i];

并去掉 size 全部在一起）