4

考虑到对象是可变的,我正在尝试创建一个复制构造函数。我的复制构造函数是错误的;我似乎无法弄清楚我做错了什么。

请不要告诉我使用clone(). 在这种情况下,我将如何完成复制构造函数?我是 Java 新手,非常感谢任何帮助。

public class MyList {


public ArrayList<Cool> people;

/**
 * "people" variable as a new (empty) ArrayList of Cool objects.
 */
public MyPersonList()
{
    people = new ArrayList<Cool>(0);    
}


/**
 * A copy constructor which makes the right kind of copy considering
 * a Cool is mutable.
 */
public MyList(MyList other) 
{
    people = new ArrayList<Cool>(); 

    for(Cool p:people)
    {   
        people.add(p);
    }

}
4

3 回答 3

11

注意:克隆列表与克隆列表中的元素不同。

这些方法都没有按照您希望的方式工作:

//1
people = new ArrayList<Cool>(other.people);

//2
people = new ArrayList<Cool>();
for(Cool p : other.people) {
    people.add(p);
}

上述方法将填充people,使其包含与 相同的元素other.people

但是,您不希望它包含相同的元素。你想用other.people.

最好的方法是这样的:

people = new ArrayList<Cool>(other.people.size());
for(Cool p : other.people) {
    people.add((Cool)p.clone());
}

确保Cool实现Cloneable. clone()必要时覆盖。

于 2013-05-02T01:13:59.053 回答
4

简单地说:你正在迭代,people但你应该迭代other.people变量。

请注意:ArrayList已经提供了一个构造函数来添加另一个集合的所有项目:

ArrayList(Collection<? extends E> c)  

所以:

people = new ArrayList<Cool>(other.people); 

足够的。

于 2013-05-02T00:37:40.797 回答
2
public MyList(MyList other) 
{
    people = new ArrayList<Cool>(); 

    for(Cool p:people)
    {   
        people.add(p);
    }

}

将其更改为:

public MyList(MyList other) 
{
    people = new ArrayList<Cool>(); 

    for(Cool p : other.people)
    {   
        people.add(p);
    }

}
于 2013-05-02T01:42:32.240 回答