Está pasando 'esto' en una llamada a método práctica aceptada en java

¿Es una práctica buena / mala / aceptable pasar el objeto actual en una llamada a un método? Como en:

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

Específicamente, ¿es bar.foo(this)aceptable la línea ?

No hay razón para no usarlo, thises la instancia actual y su uso es perfectamente legítimo. De hecho, a menudo no existe una forma clara de omitirlo.

Así que úsalo.

Como es difícil convencer de que es aceptable sin un ejemplo (una respuesta negativa a esa pregunta siempre es más fácil de argumentar), acabo de abrir una de las java.langclases más comunes , la Stringúnica, y por supuesto encontré casos de este uso, por ejemplo

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

Busque (thisen grandes proyectos "aceptados", no dejará de encontrarlo.

No hay nada de malo en eso. Lo que NO es una buena práctica es hacer lo mismo dentro de los constructores, porque daría una referencia a un objeto que aún no está completamente inicializado.

Hay una especie de publicación similar aquí: Java filtrando esto en el constructor donde dan una explicación de por qué esta última es una mala práctica.

Sí , pero debes tener cuidado con dos cosas

1. Pasando esto cuando el objeto aún no se ha construido (es decir, en su constructor)
2. Pasar esto a un objeto de larga duración, mantendrá viva la referencia y evitará que este objeto sea recolectado como basura.

Es perfectamente normal y perfectamente aceptable.

esto representa el objeto actual. Lo que está haciendo es sistemáticamente correcto, pero no veo la necesidad de esto si está llamando al método en la misma clase.

Es una mala práctica pasar el objeto actual en una llamada a un método si existen alternativas menos complejas para lograr el mismo comportamiento.

Por definición, una asociación bidireccional se crea tan pronto como thisse pasa de un objeto a otro.

Para citar Refactoring, de Martin Fowler:

Cambiar asociación bidireccional a unidireccional (200)

Las asociaciones bidireccionales son útiles, pero tienen un precio. El precio es la complejidad adicional de mantener los enlaces bidireccionales y garantizar que los objetos se creen y eliminen correctamente. Las asociaciones bidireccionales no son naturales para muchos programadores, por lo que a menudo son una fuente de errores.

...

Debe utilizar asociaciones bidireccionales cuando sea necesario, pero no cuando no lo necesite. Tan pronto como vea que una asociación bidireccional ya no está tirando de su peso, deje caer el extremo innecesario.

Entonces, teóricamente, deberíamos escuchar las alarmas cuando nos damos cuenta de que necesitamos aprobar thisy esforzarnos mucho para pensar en otras formas de resolver el problema en cuestión. Por supuesto, hay ocasiones en las que, como último recurso, tiene sentido hacerlo.

Además, a menudo es necesario corromper su diseño temporalmente, haciendo 'cosas de malas prácticas', durante una refactorización a largo plazo de su código para una mejora general. (Un paso atrás, dos pasos adelante).

En la práctica, he descubierto que mi código ha mejorado enormemente al evitar los enlaces bidireccionales como la plaga.

Si. puede usarlo. Es común en la programación pasar. thisPero hay pros y contras acerca de usarlo. Aún así, no es peligroso hacerlo.

Solo para agregar un ejemplo más en el que pasar thises correcto y sigue un buen diseño: Patrón de visitante . En el patrón de diseño de visitantes, el método accept(Visitor v)generalmente se implementa de una manera que simplemente llama v.visit(this).

Aceptable

Fragmento de documentos de Oracle JAVA:

Dentro de un método de instancia o un constructor, esta es una referencia al objeto actual, el objeto cuyo método o constructor se está llamando. Puede hacer referencia a cualquier miembro del objeto actual desde dentro de un método de instancia o un constructor usando this.

Usando esto con un campo

La razón más común para usar la palabra clave this es porque un campo está sombreado por un método o parámetro de constructor.

Todo en Java se pasa por valor. ¡Pero los objetos NUNCA se pasan al método!
Cuando Java pasa un objeto a un método, primero hace una copia de una referencia al objeto, no una copia del objeto en sí. Por lo tanto, este es un método perfectamente utilizado en Java. Y el uso más seguido.