保存せずに新しいオブジェクトを作成するのは悪い習慣ですか？

オブジェクトへの参照を保存せずにJavaコードで作成されたオブジェクトを見てきました。たとえば、Eclipseプラグインで、次のように作成されたSWTシェルを見ました。

new Shell();

この新しいShellオブジェクトは変数に格納されませんが、ウィンドウが破棄されるまで参照されたままになります。ウィンドウが閉じられると、デフォルトで[信じる？]が発生します。

参照を保存せずにこのようなオブジェクトを作成するのは悪い習慣ですか？または、ライブラリの設計が不十分でしたか？参照の必要はないが、オブジェクトの「副作用」のみが必要な場合はどうなりますか？とにかく参照を保存する必要がありますか？

更新：

確かに、私の上記の例は貧弱です。このように作成されたUI要素を見てきましたが、Shellインスタンスでopenメソッドを呼び出す必要があるため、このようなSWTシェルの作成はおそらく無意味です。Java並行性チュートリアルの次のような、aixが提供するより良い例があります。

(new HelloThread()).start();

このプラクティスは多くのコンテキストで見られるため、疑問が残ります。それは良い習慣ですか？

これには個人的な好みの要素がありますが、参照を保存しないことは必ずしも悪い習慣ではないと思います。

次の仮想的な例を考えてみましょう。

new SingleFileProcessor().process(file);

すべてのファイルに対して新しいプロセッサオブジェクトを作成する必要process()があり、呼び出し後に必要ない場合、そのオブジェクトへの参照を保存しても意味がありません。

Java 同時実行性チュートリアルから取った別の例を次に示します。

(new HelloThread()).start();

参照が保存されていないときに、他の多くの例を見てきました。

String str = new StringBuilder().append(x).append(y).append(z).toString();

（StringBuilderオブジェクトは保持されません。）

common.lang'sなどを含む同様のパターンがありますHashCodeBuilder。

作成したオブジェクトへの参照が必要ない場合は、参照を保持しないでください。それはそれと同じくらい簡単です。

一般に、参照を合理的にリリースできる限り迅速にリリースすることをお勧めします。次の間に違いはありません。

HelloThread thread = new HelloThread();
thread.start();
// where thread is never used for the rest of the method

そして

(new HelloThread()).start();

コードに関する限り、変数名の使用を避けているだけであり、これは肯定的なことかもしれません。JITコンパイラーは一般に、最後に使用した後にスレッドをガベージコレクションできることを認識できるほど賢いので、おそらくパフォーマンスの観点から違いはありません。

回避すべき唯一の本当の問題は、Constructor Anti-Patternのコードですが、それがあなたが求めているもののようには聞こえません。

SWTを使用している場合、これは悪い場合があります。自分でクリーンアップする必要があるためです（dispose（）メソッドを呼び出します）。しかし、他のクラス（非SWT）では問題ありません。

オペレーティングシステムリソースの管理に関する記事を次に示します。

まず第一に、Javaの前にCまたはC ++を学んだ人を困らせることになります。読者がそれが長所か短所かを判断するための演習として、それを残しておきます。

オブジェクトが非常に短命になるように設計されている状況は確かにありますが、ほとんどの場合、オブジェクトを作成するのに十分な複雑さがある場合、参照を保持する十分な理由があるほど複雑なので、少なくとも何かを見逃したかどうかを再確認するように警告してください。

たとえば、チュートリアルでは誰もスレッド参照を保持することを気にしませんが、実際には、スレッドを生成するのに時間がかかる場合、通常はスレッドをキャンセルできるようになるのに十分な時間がかかります。または、寿命が短い場合は、通常、join続行する前に必要な多数のスレッドを生成します。または、スレッドの作成が実際に成功したことを確認する必要があります。または、終了する前にスレッドが完全に終了することを確認する必要があります。写真が撮れます。