Passe 'this' dans un appel de méthode pratique acceptée en Java

Est-ce une pratique bonne / mauvaise / acceptable de passer l'objet courant dans un appel de méthode. Un péché:

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);
    }
}

Plus précisément, la ligne est-elle bar.foo(this)acceptable?

Il n'y a aucune raison de ne pas l'utiliser, thisc'est l'instance actuelle et son utilisation est parfaitement légitime. En fait, il n'y a souvent aucun moyen propre de l'omettre.

Alors utilisez-le.

Comme il est difficile de convaincre que c'est acceptable sans exemple (une réponse négative à une telle question est toujours plus facile à argumenter), je viens d'ouvrir l'une des java.langclasses les plus courantes , Stringcelle, et bien sûr j'ai trouvé des instances de cette utilisation, par exemple

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

Cherchez (thisdans les grands projets «acceptés», vous ne manquerez pas de le trouver.

Il n'y a rien de mal à cela. Ce qui n'est PAS une bonne pratique, c'est de faire la même chose à l'intérieur des constructeurs, car vous donneriez une référence à un objet pas encore complètement initialisé.

Il y a une sorte de post similaire ici: Java divulgue ceci dans le constructeur où ils expliquent pourquoi ce dernier est une mauvaise pratique.

Oui , mais vous devez faire attention à deux choses

1. Passer ceci lorsque l'objet n'a pas encore été construit (c'est-à-dire dans son constructeur)
2. Passer ceci à un objet de longue durée, qui maintiendra la référence en vie et empêchera cet objet d'être ramassé.

C'est parfaitement normal et parfaitement acceptable.

cela représente l'objet actuel. Ce que vous faites est statiquement correct, mais je ne vois pas la nécessité de cela si vous appelez la méthode dans la même classe.

C'est une mauvaise pratique de passer l'objet courant dans un appel de méthode s'il existe des alternatives moins complexes pour obtenir le même comportement.

Par définition, une association bidirectionnelle est créée dès qu'elle thisest passée d'un objet à un autre.

Pour citer Refactoring, par Martin Fowler:

Changer l'association bidirectionnelle en unidirectionnel (200)

Les associations bidirectionnelles sont utiles, mais elles ont un prix. Le prix est la complexité supplémentaire de maintenir les liens bidirectionnels et de s'assurer que les objets sont correctement créés et supprimés. Les associations bidirectionnelles ne sont pas naturelles pour de nombreux programmeurs, elles sont donc souvent une source d'erreurs

...

Vous devez utiliser des associations bidirectionnelles lorsque vous en avez besoin, mais pas lorsque vous ne le faites pas. Dès que vous voyez qu'une association bidirectionnelle ne tire plus son poids, laissez tomber l'extrémité inutile.

Donc, théoriquement, nous devrions entendre la sonnette d'alarme lorsque nous constatons que nous devons passer thiset essayer vraiment de penser à d'autres moyens de résoudre le problème en cours. Il y a, bien sûr, des moments où, en dernier ressort, il est logique de le faire.

De plus, il est souvent nécessaire de corrompre temporairement votre conception, en faisant des «mauvaises pratiques», lors d'une refactorisation à plus long terme de votre code pour une amélioration globale. (Un pas en arrière, deux pas en avant).

Dans la pratique , j'ai trouvé mon code a amélioré massivement en évitant des liens bidirectionnels comme la peste.

Oui. vous pouvez l'utiliser.Il est juste courant dans la programmation de passer this.Mais il y a des avantages et des inconvénients à l'utiliser.Mais ce n'est pas dangereux de le faire.

Juste pour ajouter un autre exemple où le passage thisest correct et suit une bonne conception: modèle de visiteur . Dans le modèle de conception de visiteur, la méthode accept(Visitor v)est généralement implémentée d'une manière qu'elle appelle simplement v.visit(this).

Acceptable

Extrait de la documentation Oracle JAVA:

Dans une méthode d'instance ou un constructeur, il s'agit d'une référence à l'objet actuel - l'objet dont la méthode ou le constructeur est appelé. Vous pouvez faire référence à n'importe quel membre de l'objet actuel à partir d'une méthode d'instance ou d'un constructeur en utilisant this.

Utiliser ceci avec un champ

La raison la plus courante d'utilisation du mot-clé this est qu'un champ est masqué par une méthode ou un paramètre de constructeur.

Tout en java est passé par valeur. Mais les objets ne sont JAMAIS passés à la méthode!
Lorsque java passe un objet à une méthode, il effectue d'abord une copie d'une référence à l'objet, pas une copie de l'objet lui-même. C'est pourquoi cette méthode est parfaitement utilisée en java. Et l'utilisation la plus couramment suivie.