Kopie oder Referenz ?

[Carcharoth]

Hallo

Ich habe eine Klasse Graph welche manchmal grosse Datenmengen halten kann.
Eine Klasse GrafischeDarstellung benötigt Zugriff auf diese Daten. Ich gebe also ein Graph als Parameter über den Konstruktor von GrafischeDarstellung weiter.

public class GrafischeDarstellung {
    Graph graph;
    
    GrafischeDarstellung(Graph graph) {
        this.graph=graph;
    }
}

Frage 1: macht das Sinn der Klasse GrafischeDarstellung so den Zugriff auf die Daten von Graph zu geben oder gibt es einen besseren Weg?
Frage 2: wird in einer Instanz von GrafischeDarstellung jeweils eine Kopie oder eine Referenz auf die jeweilige Instanz von Graph gemacht? Spielt aus meiner Sicht eine Rolle für den Speicherplatz.

Danke und Gruss

 

[kneitzel]

Ja, das ist der richtige Weg und es wird nur eine Referenz weiter gegeben.

Das kannst du auch testen, indem du z.B. Daten veränderst im Graphen - die Änderungen siehst du über beide Referenzen.

 

[Carcharoth]

OK, danke.

Zusatzfrage: wie würde ich es anstellen, damit effektiv eine Kopie erstellt wird? Müsste ich im Konstruktor von GrafischeDarstellung die Kopie des Parameters erstellen und dann dem Feld "graph" in GrafischerDarstellung zuweisen? so was wie :

public class GrafischeDarstellung {
    Graph graph;
    
    GrafischeDarstellung(Graph graph) {
        this.graph=graph.kopieren();
    }
}

(Methode kopieren würde ich in der Klasse Graph implementieren)

 

[White_Fox]

Ich würde sagen es kommt darauf an. Wenn du z.B. auf keinen Fall willst, daß 'GraphischeDarstellung' in 'Graph' irgendwelche Änderungen vornehmen kann, würde ich es nicht so machen.
Dann würde ich 'Graph' ein Interface 'DataPresentationer' mitgeben, das ausschließlich Lesemethoden enthält. Eine andere Möglichkeit wäre, daß 'Graph' auf Anfrage von 'GraphischeDarstellung' ein 'GraphModel' mit den enthaltenen Daten liefert.

 

[kneitzel]

Also eine Kopie müsstest Du selbst erzeuge. Dein Ansatz mit der Methode ist also vom Prinzip her richtig. Die Methode, die man da oft findet, ist clone() und es gibt auch ein Interface Clonable. Das ist aber ein Ansatz, der so eher unüblich ist, da dies im Bereich der Vererbung zu Problemen führen kann.

Die Implementierung, die daher gebräuchlicher ist, ist ein "Copy Constructor", also ein Konstruktor in Graph, der einen Graph als Parameter nimmt.

So wie @White_Fox aber auch schon etwas ausgeführt hat, sollte man überlegen, ob man dies wirklich benötigt.

 

[Carcharoth]

Danke euch beiden. Habe nun Futter um weiter zu experimentieren.
Gruss