Klon yöntemi nasıl düzgün şekilde geçersiz kılınır?

Üst sınıfı olmayan nesnelerimden birinde derin bir klon uygulamam gerekiyor.

CloneNotSupportedExceptionSüper sınıf tarafından atılan kontrolleri ele almanın en iyi yolu nedir Object?

Bir iş arkadaşım bunu şu şekilde halletmemi tavsiye etti:

@Override
public MyObject clone()
{
    MyObject foo;
    try
    {
        foo = (MyObject) super.clone();
    }
    catch (CloneNotSupportedException e)
    {
        throw new Error();
    }

    // Deep clone member fields here

    return foo;
}

Bu benim için iyi bir çözüm gibi görünüyor, ancak ekleyebileceğim başka içgörüler olup olmadığını görmek için StackOverflow topluluğuna atmak istedim. Teşekkürler!

Kesinlikle kullanmak zorunda clonemısın? Çoğu insan Java'nın clonebozuk olduğu konusunda hemfikir .

Josh Bloch on Design - Oluşturucuyu Kopyala ve Klonlama

Kitabımdaki klonlama ile ilgili maddeyi okursanız, özellikle satır aralarını okursanız clone, derinden kırık olduğunu düşündüğümü anlayacaksınız . [...] CloneableKırılan bir utanç ama oluyor.

Etkili Java 2. Baskı, Madde 11: Mantıklı bir cloneşekilde geçersiz kıl kitabında konu hakkında daha fazla tartışma okuyabilirsiniz . Bunun yerine bir kopya oluşturucu veya kopyalama fabrikası kullanılmasını önerir.

Yapmanız gerektiğini düşünüyorsanız, nasıl uygulamanız gerektiğine dair sayfalarca sayfa yazmaya devam etti clone. Ama bununla kapattı:

Tüm bu karmaşıklıklar gerçekten gerekli mi? Seyrek. Uygulayan bir sınıfı genişletirseniz Cloneable, iyi huylu bir cloneyöntemi uygulamaktan başka seçeneğiniz yoktur . Aksi takdirde, alternatif nesne kopyalama yöntemleri sağlamanız veya bu özelliği sağlamamanız daha iyi olur .

Vurgu onundu, benim değil.

Uygulama dışında çok az seçeneğiniz olduğunu açıkça belirttiğinize göre clone, işte bu durumda yapabilecekleriniz: emin olun MyObject extends java.lang.Object implements java.lang.Cloneable. Durum buysa, ASLA bir CloneNotSupportedException. AssertionErrorBazılarının önerdiği gibi atmak mantıklı görünüyor, ancak bu özel durumda catch bloğunun neden asla girilmeyeceğini açıklayan bir yorum da ekleyebilirsiniz .

Alternatif olarak, başkalarının da önerdiği gibi, belki de clonearamadan uygulayabilirsiniz super.clone.

Bazen bir kopya oluşturucuyu uygulamak daha kolaydır:

public MyObject (MyObject toClone) {
}

Sizi kullanım zahmetinden kurtarır CloneNotSupportedException, finaltarlalarla çalışır ve geri dönmek için tür konusunda endişelenmenize gerek yoktur.

Kodunuzun çalışma şekli, onu yazmanın "kanonik" yoluna oldukça yakındır. AssertionErrorYine de yakalamaya bir tane atardım . O hatta asla ulaşılmaması gerektiğini gösterir.

catch (CloneNotSupportedException e) {
    throw new AssertionError(e);
}

Atılacağı iki durum vardır CloneNotSupportedException:

1. Klonlanmakta olan sınıf uygulanmıyor Cloneable(gerçek klonlamanın sonunda Objectklon yöntemini ertelediği varsayılarak ). Uygulamalarda bu yöntemi yazdığınız sınıf Cloneable, bu asla olmayacaktır (çünkü herhangi bir alt sınıf onu uygun şekilde miras alacaktır).
2. İstisna, bir uygulama tarafından açıkça fırlatılır - bu, üst sınıf olduğunda bir alt sınıfta klonlanabilirliği önlemek için önerilen yoldur Cloneable.

İkinci durum sınıfınızda oluşamaz (doğrudan trybloktaki süper sınıf yöntemini çağırdığınız için, bir alt sınıf çağrısından çağrılsa bile super.clone()) ve sınıfınızın açıkça uygulaması gerektiği için ilk durum olmamalıdır Cloneable.

Temel olarak, hatayı kesin olarak kaydetmelisiniz, ancak bu belirli durumda yalnızca sınıfınızın tanımını bozarsanız gerçekleşir. Bu nedenle, onu kontrol edilmiş bir sürümü NullPointerException(veya benzeri) gibi ele alın - kodunuz işlevselse asla atılmayacaktır.

Belirli bir nesne hiçbir garantisi yoktur - diğer durumlarda bu olasılığa karşı hazırlıklı olmak gerekir ise istisna yakalamak zaman, mevcut nesne ile devam (bu durumuna bağlı olarak uygun eylemi alternatif klonlama stratejisi almalı nedenle, kopyalanabilen Örneğin, seri hale getirme-seri IllegalParameterExceptiondurumdan çıkarma, yönteminiz parametreyi klonlanabilir, vb. gerektiriyorsa atın.).

Düzenleme : Genel olarak şunu belirtmeliyim ki, evet, clone()doğru bir şekilde uygulanması gerçekten zor ve arayanlar için dönüş değerinin istedikleri gibi olup olmayacağını bilmesi zor, derin ve sığ klonları düşündüğünüzde iki katına çıkın. Her şeyden tamamen kaçınmak ve başka bir mekanizma kullanmak genellikle daha iyidir.

Derin kopyalar yapmak için serileştirmeyi kullanın . Bu en hızlı çözüm değildir, ancak türe bağlı değildir.

Korunan kopya oluşturucuları şu şekilde uygulayabilirsiniz:

/* This is a protected copy constructor for exclusive use by .clone() */
protected MyObject(MyObject that) {
    this.myFirstMember = that.getMyFirstMember(); //To clone primitive data
    this.mySecondMember = that.getMySecondMember().clone(); //To clone complex objects
    // etc
}

public MyObject clone() {
    return new MyObject(this);
}

Buradaki yanıtların çoğu geçerli olsa da, çözümünüzün gerçek Java API geliştiricilerinin de bunu nasıl yaptığını söylemem gerekiyor. (Josh Bloch veya Neal Gafter)

İşte openJDK, ArrayList sınıfından bir alıntı:

public Object clone() {
    try {
        ArrayList<?> v = (ArrayList<?>) super.clone();
        v.elementData = Arrays.copyOf(elementData, size);
        v.modCount = 0;
        return v;
    } catch (CloneNotSupportedException e) {
        // this shouldn't happen, since we are Cloneable
        throw new InternalError(e);
    }
}

Sizin de fark ettiğiniz ve diğerlerinin de bahsettiği gibi, arayüzü CloneNotSupportedExceptionuyguladığınızı beyan ederseniz hemen hemen atılma şansı yoktur Cloneable.

Ayrıca, geçersiz kılınan yöntemde yeni bir şey yapmazsanız, yöntemi geçersiz kılmanıza gerek yoktur. Yalnızca nesne üzerinde fazladan işlemler yapmanız gerektiğinde veya onu herkese açık hale getirmeniz gerektiğinde onu geçersiz kılmanız gerekir.

Nihayetinde, bundan kaçınmak ve başka bir şekilde yapmak en iyisidir.

public class MyObject implements Cloneable, Serializable{   

    @Override
    @SuppressWarnings(value = "unchecked")
    protected MyObject clone(){
        ObjectOutputStream oos = null;
        ObjectInputStream ois = null;
        try {
            ByteArrayOutputStream bOs = new ByteArrayOutputStream();
            oos = new ObjectOutputStream(bOs);
            oos.writeObject(this);
            ois = new ObjectInputStream(new ByteArrayInputStream(bOs.toByteArray()));
            return  (MyObject)ois.readObject();

        } catch (Exception e) {
            //Some seriouse error :< //
            return null;
        }finally {
            if (oos != null)
                try {
                    oos.close();
                } catch (IOException e) {

                }
            if (ois != null)
                try {
                    ois.close();
                } catch (IOException e) {

                }
        }
    }
}

Java'nın Cloneable uygulamasının kırılmış olması, kendinizinkini yaratamayacağınız anlamına gelmez.

OP'nin gerçek amacı derin bir klon oluşturmaksa, bunun gibi bir arayüz oluşturmanın mümkün olduğunu düşünüyorum:

public interface Cloneable<T> {
    public T getClone();
}

sonra onu uygulamak için daha önce bahsedilen prototip yapıcısını kullanın:

public class AClass implements Cloneable<AClass> {
    private int value;
    public AClass(int value) {
        this.vaue = value;
    }

    protected AClass(AClass p) {
        this(p.getValue());
    }

    public int getValue() {
        return value;
    }

    public AClass getClone() {
         return new AClass(this);
    }
}

ve bir AClass nesne alanına sahip başka bir sınıf:

public class BClass implements Cloneable<BClass> {
    private int value;
    private AClass a;

    public BClass(int value, AClass a) {
         this.value = value;
         this.a = a;
    }

    protected BClass(BClass p) {
        this(p.getValue(), p.getA().getClone());
    }

    public int getValue() {
        return value;
    }

    public AClass getA() {
        return a;
    }

    public BClass getClone() {
         return new BClass(this);
    }
}

Bu şekilde, @SuppressWarnings veya diğer hileli kodlara gerek kalmadan BClass sınıfının bir nesnesini kolayca derinlemesine klonlayabilirsiniz.