Übergibt 'dies' in einem Methodenaufruf, der in Java akzeptiert wird

Ist es eine gute / schlechte / akzeptable Praxis, das aktuelle Objekt in einem Methodenaufruf zu übergeben? Wie in:

public class Bar{
    public Bar(){}

    public void foo(Baz baz){
        //  modify some values of baz
    }
}

public class Baz{
    //constructor omitted

    public void method(){
        Bar bar = new Bar();
        bar.foo(this);
    }
}

Ist die Leitung konkret bar.foo(this)akzeptabel?

Es gibt keinen Grund, es nicht zu verwenden, es thisist die aktuelle Instanz und es ist absolut legitim, es zu verwenden. Tatsächlich gibt es oft keinen sauberen Weg, dies wegzulassen.

Also benutze es.

Da es schwer zu überzeugen ist, dass es ohne Beispiel akzeptabel ist (eine negative Antwort auf eine solche Frage ist immer leichter zu argumentieren), habe ich gerade eine der häufigsten java.langKlassen eröffnet, die Stringeine, und natürlich habe ich zum Beispiel Beispiele für diese Verwendung gefunden

1084        // Argument is a String
1085        if (cs.equals(this))
1086            return true;

Suchen Sie (thisin großen "akzeptierten" Projekten, Sie werden es nicht verfehlen, es zu finden.

Daran ist nichts auszusetzen. Was NICHT eine gute Praxis ist, ist, dasselbe innerhalb von Konstruktoren zu tun, da Sie einen Verweis auf ein noch nicht vollständig initialisiertes Objekt geben würden.

Hier gibt es eine Art ähnlichen Beitrag: Java leckt diesen im Konstruktor, wo erklärt wird, warum letzteres eine schlechte Praxis ist.

Ja , aber Sie sollten in zwei Punkten vorsichtig sein

1. Vorbei an diesen , wenn das Objekt noch nicht gebaut worden ist (dh in seinem Konstruktor)
2. Wenn Sie dies an ein langlebiges Objekt weitergeben, bleibt die Referenz am Leben und es wird verhindert, dass dieses Objekt durch Müll gesammelt wird.

Es ist völlig normal und vollkommen akzeptabel.

Dies steht für das aktuelle Objekt. Was Sie tun, ist sytatisch korrekt, aber ich sehe keine Notwendigkeit dafür, wenn Sie die Methode in derselben Klasse aufrufen.

Es wird empfohlen , das aktuelle Objekt in einem Methodenaufruf zu übergeben, wenn weniger komplexe Alternativen vorhanden sind, um dasselbe Verhalten zu erzielen.

Per Definition wird eine bidirektionale Zuordnung erstellt, sobald sie thisvon einem Objekt zu einem anderen übergeben wird.

Um Refactoring von Martin Fowler zu zitieren:

Ändern Sie die bidirektionale Zuordnung in unidirektional (200).

Bidirektionale Assoziationen sind nützlich, aber sie haben einen Preis. Der Preis ist die zusätzliche Komplexität der Aufrechterhaltung der bidirektionalen Verbindungen und der Sicherstellung, dass Objekte ordnungsgemäß erstellt und entfernt werden. Bidirektionale Assoziationen sind für viele Programmierer nicht selbstverständlich, daher sind sie häufig eine Fehlerquelle

...

Sie sollten bidirektionale Assoziationen verwenden, wenn Sie müssen, aber nicht, wenn Sie dies nicht tun. Sobald Sie sehen, dass eine bidirektionale Assoziation nicht mehr an Gewicht verliert, lassen Sie das unnötige Ende fallen.

Theoretisch sollten wir also Alarmglocken hören, wenn wir feststellen, dass wir bestehen müssen, thisund uns wirklich bemühen, andere Wege zu finden, um das vorliegende Problem zu lösen. Es gibt natürlich Zeiten, in denen es letztendlich Sinn macht, dies zu tun.

Außerdem ist es häufig erforderlich, Ihr Design vorübergehend zu beschädigen und während einer längerfristigen Umgestaltung Ihres Codes „schlechte Praktiken“ auszuführen, um eine allgemeine Verbesserung zu erzielen. (Ein Schritt zurück, zwei Schritte vorwärts).

In der Praxis habe ich festgestellt, dass sich mein Code massiv verbessert hat, indem bidirektionale Links wie die Pest vermieden wurden.

Ja. Sie können es verwenden. Es ist nur üblich in der Programmierung zu bestehen. thisAber es gibt Vor- und Nachteile bei der Verwendung. Trotzdem ist es nicht gefährlich, dies zu tun.

Nur um ein weiteres Beispiel hinzuzufügen, bei dem die Übergabe thiskorrekt ist und einem guten Design folgt: Besuchermuster . Im Besucherentwurfsmuster wird die Methode accept(Visitor v)normalerweise so implementiert, wie sie nur aufgerufen wird v.visit(this).

Akzeptabel

Ausschnitt aus Oracle JAVA-Dokumenten:

Innerhalb einer Instanzmethode oder eines Konstruktors ist dies eine Referenz auf das aktuelle Objekt - das Objekt, dessen Methode oder Konstruktor aufgerufen wird. Mit dieser Option können Sie innerhalb einer Instanzmethode oder eines Konstruktors auf ein beliebiges Mitglied des aktuellen Objekts verweisen.

Verwenden Sie dies mit einem Feld

Der häufigste Grund für die Verwendung dieses Schlüsselworts ist, dass ein Feld von einer Methode oder einem Konstruktorparameter beschattet wird.

Alles in Java wird als Wert übergeben. Objekte werden jedoch NIEMALS an die Methode übergeben!
Wenn Java ein Objekt an eine Methode übergibt, erstellt es zuerst eine Kopie eines Verweises auf das Objekt, nicht eine Kopie des Objekts selbst. Daher ist dies eine perfekt verwendete Methode in Java. Und am häufigsten gefolgt Verwendung.