if条件付きでのset.add（）のブール戻り値？

セットクラスのadd演算子は、ブール値を返します。ブール値は、追加する要素がまだ存在しない場合はtrue、そうでない場合はfalseです。書いています

if (set.add(entry)) {
    //do some more stuff
}

きれいなコードを書くという点で良いスタイルと考えられていますか？あなたは一度に2つのことをするので、私は疑問に思っています。1）要素を追加し、2）要素が存在したかどうかを確認します。

はい、そうです。

オペレーションがブール値を返す通常のポイントは、それを使用して、つまり構造内で決定を下せることifです。この可能性を実現できる唯一の他の方法は、戻り値を変数に格納し、すぐにそれを再利用ifすることif(operation()) { ... }です。だから、先に進んでそれをしてください、私たちはあなたを判断しません（そのため）。

メンテナーが戻り値が意味するものをすでに知っているか調べることを強制するため、可能な限りクリーンではないと言います。値がすでに存在していた、まだ存在していなかった、正常に挿入されたことを意味しますか？あなたがそれをあまり使わないなら、あなたは知りません、そして、たとえあなたがそれをするとしても、それははるかに精神的な負荷です。

私は次を好むでしょう：

boolean added = set.add(entry);

if (added) {
    //do some more stuff
}

はい、少し冗長ですが、コンパイラはまったく同じバイトコードを生成する必要があります。Javaセットを何年も使用していない人でも、何も調べずにロジックに従うことができます。

trueが成功を意味する場合、それは良い、明確なコードです。

成功すると、関数またはメソッドがtrue（またはtrueと評価されるもの）を返すという一般的な規則があります。コードがそれに従う限り、条件にメソッドを入れることは問題ないと思います。

私の見解では、このようなコードは不必要に乱雑です：

boolean frobulate_succeeded = thing.frobulate();

if (frobulate_succeeded) {
    ...
}

自分を繰り返しているように感じます。

ただし、戻り値の意味についての質問はあいまいです。「追加された要素が既に存在しているかどうかを示すブール値」と言います。これは、trueが要素が存在したことを意味する可能性があります（追加は発生しませんでした）。その場合は、メソッドの戻り動作をより一般的なものに変更するのが理想的です。それが不可能な場合は、コードで返された結果に明確にラベルを付けることができる追加の中間変数を追加します（他の人が提案したとおり）。

私はそれが非常にCのようなものだと思います。たいていの場合、突然変異の結果には記述的に名前を付けた変数を使用し、if条件内で突然変異は発生しないようにします。

コンパイラは、すぐに再利用される場合、この変数を削除します。人間はソースを読むのが簡単になります。私にとって、それはより重要です。

if条件にand/ or句を追加して条件を拡張する必要.add()がある場合、短絡評価のために特定のケースで呼び出しを行わないことがあります。短絡が特に予想されない限り、これはバグになる可能性があります。

あなたのコードはCommand Query Separationを壊しているようです。これについては、クリーンコードブックと関数構造のビデオで説明しています。クリーンコードの観点からは、良いスタイルとは見なされないと思います。

「クリーンなコードはシンプルで直接的です。きれいなコードは、よく書かれた散文のように読みます。クリーンなコードは、デザイナーの意図を曖昧にすることはありませんが、むしろ鮮明な抽象化と簡単な制御ラインでいっぱいです。

-アプリケーション指向のオブジェクト指向分析および設計のGrady Booch著者」

私にとって、あなたのコードの意図は不明です。エントリが正常に追加されたとき、または既に存在するときにも両方が実行されますか？何がadd()戻りますか？アイテム？エラーコード？