这个clone()有什么问题？

Question

我已经编写了这个克隆方法,当Employee类的父类是抽象的,而父类中的clone()方法是abstract.I想用这个代码复制Employee的对象的原始数据类型,而不是复制每个原始数据单独键入,但此代码与我调用clone()方法的行有问题.(此代码在Employee类中)

public Object clone() {
    Object obj = new Object();
    Object object = obj.clone();  //Emphasis here
    return object;

}

错误是:Object类型的方法clone()不可见.

但是我的Employee类在类层次结构中,可以访问Object类中的protected clone()方法.

这是我简单的Employee类:

public class Employee extends Person implements Cloneable {
private int ID;

public Employee() {
    ID = 0;
}

public void setID(int ID) {
    this.ID = ID;
}

public int getID() {
    return ID;
}

public Object clone1() throws CloneNotSupportedException {
    try {
        Object obj = new Object();

        Object object = obj.clone();
        return object;
    } catch (CloneNotSupportedException ex) {
        return null;
    }
}

Answer 1

使类可克隆的标准模式是:

1. 实行 Cloneable
2. 覆盖该clone()方法并将其公开
3. 在clone()调用super.clone()然后复制任何可变对象的状态

你应该不使用创建新的对象new.正确的方法是调用super.clone()一个新实例.Object的clone()是特殊的,将创建该对象的新副本,并复制其原始字段和引用.

例如:

public class Person implements Cloneable {
    protected String name;
    // Note that overridden clone is public
    public Object clone() {
        Person clone = (Person)super.clone();
        // No need to copy name as the reference will be
        // copied by Object's clone and String is immutable
        return clone;
    }
}

public class Employee extends Person {
    protected int id;
    protected java.awt.Point location;
    public Object clone() {
        Employee  clone = (Employee )super.clone();
        // No need to copy id as Object's clone has already copied it
        // Need to clone location as Point is mutable and could change
        clone.location = location.clone();
        return clone;
    }
}

Answer 2

Java的克隆机制有点尴尬.为了能够克隆自己,类必须做两件事.首先,它必须实现Clonable.其次,它必须覆盖clone()并使其公开.

在您的示例中,您覆盖了clone(),但是您不是在Employee类上调用clone()而是在Object.class()上调用clone(),其中clone()仅受保护.

Answer 3

我认为目前的绿色答案很糟糕,为什么你会问？

• 它增加了很多代码
• 它要求您列出要复制的所有字段并执行此操作
• 当使用clone()时,这对于列表不起作用(这是HashMap的clone()所说的:返回此HashMap实例的浅表副本:键和值自身未被克隆.)所以你最终手动完成它(这使得我哭了)

哦顺便说一下序列化也很糟糕,你可能不得不在整个地方添加Serializable(这也让我哭).

那么解决方案是什么:

Java深度 克隆库克隆库是一个小型的开源(apache许可证)java库,它深入克隆对象.对象不必实现Cloneable接口.实际上,这个库可以克隆任何java对象.如果您不希望修改缓存对象或者只是想要创建对象的深层副本,则可以在缓存实现中使用它.

Cloner cloner=new Cloner();
XX clone = cloner.deepClone(someObjectOfTypeXX);

请访问http://code.google.com/p/cloning/查看

Answer 4

您是否在对象上实现了 Cloneable 接口？

\n

但是，很少有情况我会使用克隆来复制对象。array.clone() 就是这样一个安全的例子。我宁愿使用复制构造函数习惯用法或手动复制/分配值。

\n

《Effective Java》 （第 2 版）中有 Item#11关于背景问题。Cloneable 接口是一种特殊的接口，因为它修改了Object类有关克隆的行为。基本上，它是 Java 中启用类接口的功能。

\n

编辑：根据您的示例，在一般情况下，您可能需要将 clone() 调用包装在 CloneNotSupportedException 的 try-catch 中。

\n

Edit2：改写我的答案

\n

Edit3：您是否覆盖了上下文中的clone() public？在您提供的示例中，您尝试克隆一个对象，该对象位于 java.lang 包中 - 几乎不是您的代码所在的包。

\n

Edit4：我认为答案已经在其他帖子中，只是想反思根本问题。

\n

编辑5：试试这个：

\n

public Object clone1() throws CloneNotSupportedException {        \n    return super.clone();        \n}\n

\n

Edit6然后命名您的方法public abstract Object copy()，并在实现中使用 super.clone() - 以避免混淆。

\n

Edit7我做了一些调整并得出了以下解决方案：

\n

public class Cloner {\n    public static abstract class Person {\n       protected abstract Object clone1() throws CloneNotSupportedException;\n       public Object copy() throws CloneNotSupportedException {\n           return clone1();\n       }\n    }\n    public static class Employee extends Person implements Cloneable {\n        @Override\n        protected Object clone1() throws CloneNotSupportedException {\n            return super.clone();\n        }\n    \n    }\n    public static void main(String[] args) throws Exception {\n        new Employee().copy();\n    }\n}\n

\n

但基本上，它与将抽象方法重命名为clone() 之外的其他名称是相同的概念。

\n

Edit8：修复了我的示例，现在它可以正常工作了。

\n

（但实际功劳归于G \xc3\xa1bor Hargitaisuper.clone()）

\n