Kopier-Konstruktor

[Kalkulator]

Hallo zusammen,

ich habe eine Frage zu Kopier-Konstruktoren.

Es gibt folgenden parametrisierten Konstruktor:

[CODE lang="java" title="Java"] public Wohnung(int neupreis, int baujahr, int zimmer, int wohnflaeche) {
super(neupreis, baujahr);
this.zimmer= zimmer;
this.wohnflaeche= wohnflaeche;
}[/CODE]

Zusätzlich soll ein Kopier-Konstruktor erstellt werden. Mein Versuch schaut aktuell so aus:

[CODE lang="java" title="Java"] public Wohnung(Wohnung wohnung) {
super(wohnung.getNeupreis(), wohnung.getBaujahr());
this.zimmer= wohnung.zimmer;
this.wohnflaeche= wohnung.wohnflaeche;
}[/CODE]

Allerdings bin ich durch Internetrecherche auch auf folgende Möglichkeiten gestoßen:

[CODE lang="java" title="Java"] public Wohnung(Wohnung wohnung) {
this(wohnung.getNeupreis(), wohnung.getBaujahr(), wohnung.zimmer, wohnung.wohnflaeche);
}[/CODE]

bzw.

[CODE lang="java" title="Java"] public Wohnung(Wohnung wohnung) {
super(wohnung);
}[/CODE]

Meine Frage: Was ist der Unterschied zwischen diesen 3 Kopier-Konstruktoren?
Welcher sollte am besten verwendet werden?
(Hinweis, falls das relevant ist: Objekte sollen mit dem Datentyp Haus der Oberklasse gespeichert werden. Jede Wohnung wird also als Haus gespeichert.)

In welchen Fällen wird in der Unterklasse zusätzlich der Konstruktor der Oberklasse benötigt?
[CODE lang="java" title="Java"] public Wohnung(Haus haus) {
super(haus);
}[/CODE]

Vielen Dank im Voraus für eure Hilfe!

 

[kneitzel]

Hier ist die Frage, was vorhanden ist. Wenn Du den Konstruktor mit 4 Parametern hast, dann wird die Variante, die den Konstruktor mit nur 2 Parametern aufruft, ggf. nicht funktionieren.

Und der letzte Fall sieht dubios aus - ein Kopier-Konstruktor von Wohnung wird in der Regel nicht einen Parameter der Oberklasse entgegen nehmen, denn die Oberklasse wurde ja doch erweitert. Und es kann ja noch andere abgeleitete Klassen geben. Und spätestens da macht dann der Copy Konstruktor ja keinen Sinn. Es mag so einen Konstruktor geben, aber das ist dann kein Copy Konstruktor. Es kann also einen Konstruktor geben, der eine Wohnung erstellt und ein Hochhaus als Parameter bekommt. Aber die Wohnung ist dann natürlich keine Kopie vom Hochhaus.

 

[Kalkulator]

Ok, und welcher der Kopierkonstruktoren ist dann zu verwenden? Ich steh gerade etwas auf dem Schlauch...

Den letzten Fall lassen wir mal weg, das hätte mich nur grundsätzlich interessiert.

 

[LimDul]

   public Wohnung(Wohnung wohnung) {
        this(wohnung.getNeupreis(), wohnung.getBaujahr(), wohnung.zimmer, wohnung.wohnflaeche);
    }

Den finde ich den angenehmsten, weil der an den bestehenden Konstruktor der Klasse Wohnung delegiert. Klappt halt nur, wenn es einen Konstruktor gibt, der auch diese 4 Parameter entgegen nehmen kann.

   public Wohnung(Wohnung wohnung) {
        super(wohnung.getNeupreis(), wohnung.getBaujahr());
        this.zimmer= wohnung.zimmer;
        this.wohnflaeche= wohnung.wohnflaeche;
    }

Der hat den Nachteil das du 2 Codezeilen hast, die identisch sind zum normalen Konstruktor. Duplizierter Code ist generell etwas, was erstmal immer ein Code-Smell ist. Wenn du z.B. dich irgendwann entscheidest und sagst, für Zimmer möchte ich aber mindestens immer eine 1 drinstehen habe bei der Erzeugung, muss du beide Zeilen anpassen mit this.zimmer = Math.max(1, ....); Im ersten Fall musst du nur einen anpassen.

Aber es kann gut sein, dass es oftmals gar keinen Konstruktor gibt, der alle Parameter entgegen nimmt oder man eh im Copy-Konstruktor Sonderlogik für die Kopierlogik braucht (wenn man z.B. eine Deep-Copy machen will, also Unterstrukturen auch kopieren will). Dann würde man eher zu der 2ten Variante tendieren.

   public Wohnung(Wohnung wohnung) {
        super(wohnung);
    }

Das wird zu 99% nicht kompilieren.

 

[Blender3D]

Meine Frage: Was ist der Unterschied zwischen diesen 3 Kopier-Konstruktoren?
Welcher sollte am besten verwendet werden?

Ein Kopierkonstruktor in einer Java-Klasse ist ein Konstruktor, der ein Objekt mit einem anderen Objekt derselben Java-Klasse erstellt.
Wichtig dabei ist auf tiefe Kopien zu achten.
Dieser Konstruktor passt wahrscheinlich am ehesten.

public Wohnung(Wohnung wohnung) {
        super(wohnung.getNeupreis(), wohnung.getBaujahr());
        this.zimmer= wohnung.zimmer;
        this.wohnflaeche= wohnung.wohnflaeche;
    }

Der Kopierkonstruktor sollte allerdings sicherstellen, dass das gesamte Objekt kopiert wird. In diesem Beispiel ist das aber nicht sicher gestellt.
Warum? Die Elternklasse könnte weitere Attribute besitzen, die im aufgerufenen Konstruktor nur mit Default Werten gesetzt werden.
Die Elternklasse müsste ebenfalls einen Kopierkonstruktor besitzen, der aufgerufen wird um eine vollständige Kopie zu gewärleisten.
z.B.
[CODE lang="java" title="Wohnung" highlight="6-8"]public class Wohnung extends BauObjekt {
public final int zimmer;
public final int wohnflaeche;

public Wohnung(Wohnung w) {
super(w);
this.zimmer = w.zimmer;
this.wohnflaeche = w.wohnflaeche;
}

public Wohnung(int neupreis, int baujahr, int zimmer, int wohnflaeche) {
super(neupreis, baujahr);
this.zimmer = zimmer;
this.wohnflaeche = wohnflaeche;
}
}[/CODE]
[CODE lang="java" title="BauObjekt" highlight="11"]public class BauObjekt {
public final int neuPreis;
public final int bauJahr;

public BauObjekt(int neuPreis, int bauJahr) {
this.neuPreis = neuPreis;
this.bauJahr = bauJahr;
}

public BauObjekt(BauObjekt o) {
this(o.neuPreis, o.bauJahr);
}
}[/CODE]

 

[fhoffmann]

Wichtig dabei ist auf tiefe Kopien zu achten.

Das muss nicht immer korrekt sein sondern hängt von der Anwendung ab.

Hat beispielsweise eine Wohnung einen besitzer von der Klasse Person, so stellt sich die Frage, ob ich wirklich eine tiefe Kopie möchte.
Inhaltlich bedeutet dies:
Wenn ich von dem besitzer der ersten Wohnung die Telefonnummer ändere, soll sich dann auch die Telefonnummer des besitzer der kopierten Wohnung automatisch mit ändern?

 

[Blender3D]

Das muss nicht immer korrekt sein sondern hängt von der Anwendung ab.

Das ist richtig. Mein Statement bezieht sich beim Kopieren immer auf tiefe Kopien zu achten. Selbstverständlich abhängig vom Kontext.
Anfänger sind sich aber beim Kopieren der Bedeutung von flachen und tiefen Kopien nicht bewusst. Es wird automatisch eine flache Kopie gemacht.
Und sie wundern sich dann wenn zwei unabhängige Objekte erzeugt werden sollen, dass sich Werte im kopierten Objekt auch ändern.