Implementieren von clone() für unveränderliche Klassen

Question

Ich entwickle eine Klassenbibliothek.

1. Ich habe eine abstrakte Basisklasse Matrix für Matrizen, die Implementierungen für einige der grundlegenden Methoden bietet.
2. Abgeleitet von Matrix sind konkrete Unterklassen für verschiedene Arten von Matrizen.
3. Ich habe die Anforderung für Matrizen klonbar sein, so implementiert Matrix die Cloneable-Schnittstelle.
4. Einige der von Matrix abgeleiteten Klassen sind unveränderlich

wäre es für die unveränderlichen Klassen Klon Methoden akzeptabel sein, dass stattdessen ein Klon des Objekts zurückzugeben, das Objekt selbst zurückgegeben wird?

Einige (stark vereinfacht) Code zur Klarstellung:

abstract class Matrix implements Cloneable { 
    ... 
} 

class ImmutableMatrix extends Matrix { 
    ImmutableMatrix clone() { 
     return this; 
    } 
    ... 
} 

class SomeOtherMatrix extends Matrix { 
    SomeOtherMatrix clone() { 
     SomeOtherMatrix other = super.clone(); 
     ... 
     return other; 
    } 
    ... 
} 

Answer 1

ich super.clone() ausreichend wäre Aufruf gedacht hätte.

Wenn Ihre Klasse unveränderlich ist, sollte sie bereits alle änderbaren Klassen geklont haben, als sie erstellt wurde. Daher würde ich denken, es wäre sicher, flache Kopien von allen Feldern Ihrer Klasse zu haben.

Der JavaDocs-Status x.clone() != x wird bevorzugt. Obwohl dies keine absolute Anforderung ist, würde es sicherlich durch Ihren Plan verletzt werden, einfach this zurückzugeben.

Answer 2

Ja Wenn wir das Verhalten von String (Unveränderbare Klasse) sehen, Wenn der Inhalt identisch ist, wird das gleiche Objekt zurückgegeben, also habe ich recht, die clone() Methode sollte nur das zurückgeben.

Answer 3

Geben Sie einfach this in der clone() - Implementierung der unveränderlichen Klassen zurück.

Answer 4

Während Sie die unveränderliche Klassen haben könnten einfach clone implementieren Verweise auf sich selbst zurück, ich bei der Verwendung clone auf die Dinge viel Wert nicht wirklich sehen, die nicht wandelbar sein kann oder nicht, abwesend unveränderlich gewisse Art und Weise der Herstellung wandelbar wird in von Dinge und umgekehrt.

Ich würde denken, dass es für Ihre Basis Matrix Klasse besser wäre, Methoden umfassen IsImmutable und IsWritable, zusammen mit AsImmutable, AsMutable und AsNewMutable Methoden; Es sollte auch Methoden zum Lesen und Schreiben der Matrix enthalten (obwohl das Aufrufen der "Write" -Methode auf einer nicht beschreibbaren Matrix eine Ausnahme auslösen sollte).

definieren statische Methoden CreateImmutableMatrix und CreateMutableMatrix die gegeben, ein Matrix, wird eine neue unveränderliche oder veränderliche Matrix erzeugen, die mit den richtigen Daten vorinitialisiert ist.

Mutable Klassen sollten AsImmutable implementieren selbst passieren zu CreateImmutableMatrix, AsMutable selbst zurückzukehren und AsNewMutable sich CreateMutableMatrix passieren.

Unveränderliche Klassen sollten implementieren AsImmutable selbst zurückzukehren, AsMutableAsNewMutable zu nennen, und AsNewMutable sich CreateMutableMatrix passieren.

Read-only-Wrapper sollte AsImmutable implementieren AsImmutable auf den gewickelten Objekte zu nennen, und AsMutable und AsNewMutableAsNewMutable auf den gewickelten Objekte zu nennen.

Ein Objekt, das eine Matrix empfängt, die es zu kopieren oder zu mutieren braucht, kann es einfach in einem Feld speichern (z. B. Foo). Wenn es die Matrix mutieren muss, kann es Foo durch Foo.AsMutable() ersetzen. Wenn das Objekt, das die Matrix enthält, kopiert werden muss, sollte das Feld in der Kopie entweder durch Foo.AsImmutable() oder Foo.AsNewMutable() ersetzt werden, je nachdem, ob das Feld in der Kopie wahrscheinlich mutiert werden muss.

Answer 5

Ihre Klasse ist nicht streng unveränderlich, da sie nicht endgültig ist: Es kann veränderbare Unterklassen geben.

Wenn jemand will, ImmutableMatrix Unterklasse, wird er nicht in der Lage sein clone() zu implementieren, indem super.clone() Aufruf, wenn Sie nur this zurück. Also in diesem Fall sollten Sie super.clone() anrufen.

Wenn Sie jedoch Ihre Klasse endgültig machen, sehe ich keinen Grund, warum nicht einfach this zurückgeben.