Übergeben des gesamten Objekts vs. Übergeben der Eigenschaft des Objekts

Ich denke über die Lösung für meine Anwendung nach. Hier ist die Situation: Ich habe eine Klasse mit einer Methode, die ObjectA als Eingabeparameter nimmt und mehrere kleine Methoden aufruft. Jede dieser Methoden benötigt einige Teile von ObjectA (sie überlappen sich nicht, dh method1() braucht ObjectA.field1 Und ObjectA.field2, method2() braucht ObjectA.field3 usw…)

Ist es angesichts der allgemeinen guten Codepraktiken und der Leistung besser, ObjectA an jede dieser Methoden zu übergeben, damit sie den benötigten Wert selbst extrahieren können, oder ist es besser, ihnen einfach Werte zu übergeben? Ich meine:

method1(ObjectA);
method2(ObjectA);

oder

method1(Object1.getField1(), ObjectA.getField2());
method2(ObjectA.getField3());

Denken Sie daran, dass Sie mit Ihrem Code nicht wirklich bestehen ObjectA. Sie übergeben nämlich den Referenztyp an ObjectAalso auf einer Leistungsnote den Unterschied zwischen dem Bestehen von a String Objektreferenz und a ObjectA Objektreferenz wäre vernachlässigbar.

So wie ich es schreiben würde

Ich würde das gesamte Objekt übergeben, wenn die Methode für die Klasse relevant ist. Meine Begründung dafür ist, das Klassenwissen so weit wie möglich aufzuteilen. Was ich damit meine ist folgendes.

public void doSomethingRelatedToTheClass(String param)
{
    // Do something with param.
}

Meine erste Kritik hier ist, dass diese Methode davon ausgeht, dass die Eingabe das richtige Feld ist. Meine zweite ist, dass die Klasse, die diesen Code aufruft, jetzt ein bisschen mehr über diese Methode wissen muss, weil sie sie so aufrufen muss:

doSomethingRelatedToTheClass(myObject.getValue());

Und das bedeutet, wenn Sie das bei einem anderen Mitglied finden ObjectA funktioniert innerhalb dieser Methode besser, oder Sie möchten auf andere Mitglieder von zugreifen ObjectAund du veränderst dich doSomething() Um diese Änderung widerzuspiegeln, müssen Sie auch den Methodenaufruf ändern in:

doSomethingRelatedToTheClass(myObject.getOtherValue(), myObject.getValue());

Indem Sie also das gesamte Objekt übergeben, abstrahieren Sie dieses Detail, und die Methode kann damit umgehen. nämlich:

doSomethingRelatedToTheClass(myObject); // Doesn't need to know what you do with it.

public void doSomethingRelatedToTheClass(ObjectA object)
{
    String val = object.getValue();

    String otherVal = object.getOtherValue();
}

Wenn eine Änderung an einer Klasse zu einer Änderung an anderen Klassen führt, wird dies ein Anti-Pattern genannt Schrotflinten-Chirurgie.

Bearbeiten

Ich hatte Gelegenheit, meine Antwort hier zu überprüfen, und ich habe meine ursprüngliche Antwort leicht geändert, weil ich glaube, dass dies nicht die beste Lösung für ist alle Situationen. Wenn sich eine Methode wie oben speziell auf eine Klasse bezieht, dann wird die Instanziierung dieser Klasse (oder bevorzugter ihrer Oberklasse oder implementierten Schnittstelle[s]) sollte der Parameter sein.

Wenn dies nicht der Fall ist, kann die Funktionalität generisch sein. Ein Beispiel für eine generische Funktion könnte sein:

public String[] findNouns(String sentence);

In diesem Fall kann das Auffinden der Substantive in einem Satz für viele Anwendungsfälle geeignet sein und nicht nur für die Anwendungsfälle, die Sie definiert haben. Daher ist das Übergeben des Werts der einzig sinnvolle Ansatz, da Sie sonst zwei logische Teile miteinander verbinden, die keine direkte Beziehung haben. Das Finden von Substantiven und dem beliebigen Objekt, das Sie definiert haben.

In Summe

• Wenn die Methode Logik ist, die sich auf das Objekt bezieht, übergeben Sie das Objekt

• Wenn die Methode nichts mit dem Objekt zu tun hat und das Objekt sie nur als Hilfsfunktion verwendet, übergeben Sie den Wert und benennen Sie die Funktion generisch.

Lassen Sie uns ein Szenario untersuchen. Nun, dies kann Ihr Szenario sein oder auch nicht, aber es veranschaulicht einen Punkt.

Nehmen wir an, field1 und field2 sind in Ihrem Fall zwei ganze Zahlen und method1 summiert sie und gibt das Ergebnis zurück.

Wenn Sie die Objekte übergeben, kann diese Methode immer nur diese beiden Felder summieren. Die Methode ist nun auch stark mit diesen Objekten gekoppelt.

Übergeben Sie dagegen nur die Felder, die beiden Integer, wird Ihre Methode in diesem Fall generischer. Sie können jetzt 2 beliebige ganze Zahlen summieren, unabhängig davon, auf welchen Objekten sie sich befinden.

Im Allgemeinen sollten Sie jedoch immer so wenig Ihrer Objekte anderen Methoden und Klassen aussetzen. Dies fördert eine lose Kopplung.

Ausnahmen

AS maaartinus weist darauf hin, wenn zum Beispiel field1 und field2 Points wären und method1 den Abstand zwischen diesen beiden Punkten berechnet, dann müsste ich zustimmen, dass das Übergeben von zwei Points besser wäre als das Übergeben von 2 xy Integer-Paaren (4 Parameter).

Hoffe das hilft

Ich würde sagen, es kommt darauf an. Eine Methode kann klarer und allgemeiner sein, wenn sie auf den Argumenten operiert, anstatt ein ganzes Objekt zu benötigen. Manchmal haben Sie die Argumente parat (z. x Und y) und müssten diese erst zu zB a aggregieren Point um die Methode aufrufen zu können. Manchmal haben Sie ein anderes, nicht verwandtes Objekt (zB einige ImmutablePointoffensichtlich nicht verlängern java.awt.Point) und müsste die Koordinaten extrahieren und ein zu übergebendes Objekt erstellen.

Wenn das übergebene Objekt die richtige Abstraktion ist, ist es normalerweise der richtige Weg, es als Ganzes zu übergeben. Es geht nicht um die Leistung, sondern um Lesbarkeit und Wartbarkeit. Siehe auch die Gesetz der Demeter was zu einer lockereren Abhängigkeit vom übergebenen Objekt führen kann.

Wie andere gesagt haben, hängt es davon ab, aber meiner Erfahrung nach erschwert das Übergeben ganzer Objekte das Lesen und Verwalten von Code.

Betrachten wir diese Methode getUserDetails(User user) die sich auf wenige Methoden wie stützt getUserAddress(User user) getUserFamilyInfo(User user) usw., die sich weiter mit verschiedenen Datenquellen verbinden können, um die Informationen abzurufen.

Es gibt keinen einfachen Weg, das zu wissen getUserFamilyInfo braucht nur userId oder es braucht userId Und lastName oder etwas anderes aus user wenn das gesamte Objekt übergeben wird. Es macht es schwierig, Abhängigkeiten zwischen verschiedenen Diensten zu verstehen und Refactoring durchzuführen.

Ich bevorzuge es, einzelne Argumente zu übergeben, wenn count weniger als 3 ist, oder ein dto zu erstellen, wenn eine Handvoll Eigenschaften von einem sehr großen Objekt benötigt werden.