メソッド呼び出しで 'this'を渡しているJavaの慣習

現在のオブジェクトをメソッド呼び出しで渡すことは良い/悪い/許容できる方法ですか？のように：

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

具体的には、ラインはbar.foo(this)受け入れ可能ですか？

これを使用しない理由はありません。this現在のインスタンスであり、使用することは完全に正当です。実際、それを省略する明確な方法がないことがよくあります。

だからそれを使う。

例がなければ受け入れられると納得させるのは難しいため（このような質問に対する否定的な答えは常に議論しやすい）、私は最も一般的なjava.langクラスの1つを開きStringました。もちろん、この使用例を見つけました。

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

探し(this大きな「受け入れ」プロジェクトでは、あなたはそれを見つけるために失敗することはありません。

何も問題はありません。まだ完全に初期化されていないオブジェクトへの参照を与えるため、コンストラクター内で同じことを行うのは良い習慣ではありません。

似たような投稿がここにあります。Javaがコンストラクター でこれをリークし、後者がなぜ悪い習慣であるかを説明しています。

はい、しかしあなたは2つのことに注意する必要があります

1. オブジェクトがまだ構築されていないときに（つまり、コンストラクターで） これを渡す
2. これを長期間存続するオブジェクトに渡すと、参照が存続し、このオブジェクトがガベージコレクションされるのを防ぎます。

それは完全に正常で完全に許容できます。

これは現在のオブジェクトを表します。あなたがやっていることは文法的に正しいですが、同じクラスのメソッドを呼び出している場合、これは必要ないと思います。

同じ振る舞いを実現するためのより簡単な代替手段がある場合、メソッド呼び出しで現在のオブジェクトを渡すことは悪い習慣です。

定義により、双方向の関連付けはthis、あるオブジェクトから別のオブジェクトに渡されるとすぐに作成されます。

Martin Fowlerによるリファクタリングを引用すると：

双方向の関連付けを単方向に変更（200）

双方向の関連付けは便利ですが、コストがかかります。価格は、双方向リンクを維持し、オブジェクトが適切に作成および削除されることを保証するために追加される複雑さです。双方向の関連付けは多くのプログラマにとって自然ではないため、多くの場合、エラーの原因になります。

...

必要な場合は双方向の関連付けを使用する必要がありますが、使用しない場合は使用しないでください。双方向の関連付けが重要度を低下させていないことを確認したら、すぐに不要な端を削除します。

したがって、理論的にはthis、問題を解決するための他の方法を考え、真剣に取り組む必要があるとわかったときに、警報ベルを聞く必要があります。もちろん、最後の手段としてそれを行うことが理にかなっている場合もあります。

また、全体的な改善のためにコードを長期間リファクタリングする際に、設計を一時的に破壊して「悪い習慣」を実行することがしばしば必要になります。（一歩後退、二歩前進）。

実際に私は私のコードが改善されているを発見した大規模な疫病のような双方向リンクを避けることによって。

はい。合格するthisことはプログラミングで一般的ですが、それを使用することには賛否両論がありますが、そうすることは危険ではありません。

渡すことthisが正しく、優れたデザインに従うもう1つの例を追加します：訪問者パターン。ビジターデザインパターンでaccept(Visitor v)は、メソッドは通常、を呼び出すだけの方法で実装されますv.visit(this)。

許容できる

Oracle JAVAドキュメントのスニペット：

インスタンスメソッドまたはコンストラクター内では、これは現在のオブジェクト（メソッドまたはコンストラクターが呼び出されているオブジェクト）への参照です。これを使用して、インスタンスメソッドまたはコンストラクター内から現在のオブジェクトの任意のメンバーを参照できます。

これをフィールドで使用する

thisキーワードを使用する最も一般的な理由は、フィールドがメソッドまたはコンストラクターのパラメーターによって隠されているためです。

Javaのすべてが値によって渡されます。しかし、オブジェクトがメソッドに渡されることはありません！
Javaがオブジェクトをメソッドに渡すとき、最初にオブジェクト自体のコピーではなく、オブジェクトへの参照のコピーを作成します。したがって、これはJavaで完全に使用されるメソッドです。そして、最も一般的に従う使用法。