Está passando 'isto' em uma prática aceita de chamada de método em java

É uma prática boa / ruim / aceitável passar o objeto atual em uma chamada de método. Como em:

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

Especificamente, a linha é bar.foo(this)aceitável?

Não há razão para não usá-lo, thisé a instância atual e é perfeitamente legítimo para usar. Na verdade, geralmente não há uma maneira limpa de omiti-lo.

Então use.

Como é difícil convencer que é aceitável sem exemplo (uma resposta negativa a essa pergunta é sempre mais fácil de argumentar), acabei de abrir uma das java.langclasses mais comuns , a String, e é claro que encontrei exemplos desse uso, por exemplo

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

Procure (thisem grandes projetos "aceitos", você não deixará de encontrá-lo.

Não há nada de errado nisso. O que NÃO é uma boa prática é fazer o mesmo dentro dos construtores, porque você daria uma referência a um objeto ainda não completamente inicializado.

Há um tipo de postagem semelhante aqui: Java vazando isso no construtor, onde eles dão uma explicação de por que o último é uma prática ruim.

Sim , mas você deve ter cuidado com duas coisas

1. Passando este quando o objecto não tiver sido ainda construído (ou seja, no seu construtor)
2. Passar isso para um objeto de longa vida, que manterá a referência viva e impedirá que este objeto seja coletado como lixo.

É perfeitamente normal e perfeitamente aceitável.

isso representa o objeto atual. O que você está fazendo é sistematicamente correto, mas não vejo necessidade disso se você estiver chamando o método na mesma classe.

É uma má prática passar o objeto atual em uma chamada de método se houver alternativas menos complexas para obter o mesmo comportamento.

Por definição, uma associação bidirecional é criada assim que thisé passada de um objeto para outro.

Para citar Refatoração, por Martin Fowler:

Alterar associação bidirecional para unidirecional (200)

As associações bidirecionais são úteis, mas têm um preço. O preço é a complexidade adicional de manter os links bidirecionais e garantir que os objetos sejam criados e removidos corretamente. As associações bidirecionais não são naturais para muitos programadores, então muitas vezes são uma fonte de erros

...

Você deve usar associações bidirecionais quando for necessário, mas não quando não for necessário. Assim que você perceber que uma associação bidirecional não está mais exercendo sua influência, descarte a extremidade desnecessária.

Então, teoricamente, deveríamos estar ouvindo alarmes quando acharmos que precisamos passar thise tentar realmente pensar em outras maneiras de resolver o problema em questão. É claro que há momentos em que, em último recurso, faz sentido fazer isso.

Também é frequentemente necessário corromper seu design temporariamente, fazendo 'coisas de prática ruim', durante uma refatoração de longo prazo de seu código para uma melhoria geral. (Um passo para trás, dois passos para a frente).

Na prática, descobri que meu código melhorou muito , evitando links bidirecionais como a praga.

Sim. você pode usá-lo. É comum na programação passar this. Mas há prós e contras em usar isso. Ainda assim, não é perigoso fazer isso.

Apenas para adicionar mais um exemplo em que a passagem thisé correta e segue um bom design: Padrão de visitante . No padrão de design do Visitante, o método accept(Visitor v)é geralmente implementado de uma maneira que ele apenas chama v.visit(this).

Aceitável

Snippet de documentos Oracle JAVA:

Dentro de um método de instância ou construtor, esta é uma referência ao objeto atual - o objeto cujo método ou construtor está sendo chamado. Você pode se referir a qualquer membro do objeto atual de dentro de um método de instância ou um construtor usando this.

Usando isso com um campo

O motivo mais comum para usar a palavra-chave this é porque um campo é sombreado por um método ou parâmetro do construtor.

Tudo em java é passado por valor. Mas os objetos NUNCA são passados ​​para o método!
Quando java passa um objeto para um método, ele primeiro faz uma cópia de uma referência ao objeto, não uma cópia do próprio objeto. Portanto, este é um método perfeitamente usado em java. E o uso mais comumente seguido.