三項演算子は、割り当てステートメントの外で使用する必要がありますか？

編集これは、3項演算子が有害かどうかの問題ではありません。それは、割り当てステートメントの外で使用する必要があるかどうかについてコンセンサスがあるかどうかについてです。/編集

ブール値に基づいて、2つの異なる関数（戻り値なし）の1つを呼び出します。私はこのJavaScriptのビットを書きました：

(condition) ? doThis(arg) : doThat(arg)

if-elseステートメントを書くよりも見た目がきれいだと思います。

if(condition) {
    doThis(arg);
}
else {
    doThat(arg);
}

私の同僚は、三項ステートメントは割り当てステートメントでのみ使用するべきだと強く信じています。これについて説明しているスタイルガイドが見つかりません。おそらく、JavaやC＃などの言語ではこれがまったく許可されていないためです（コンパイラは値を返すために3項演算子を必要とします）。これは三項ステートメントの悪用と見なされるべきですか？どうして？

一般的に言えば、条件演算子は、値を持つ ifステートメントを作成することを目的としています。w = x ? y : z。したがって、それを副作用に使用している場合、直感に反します。有効ですが直感に反します。チームメイトが読むためのコードを書いていることを思い出してください。

割り当てではないいくつかのケースで、3項演算子が役立つと思います。

関数の引数で：私の意見では

f( e1, .. cond ? em : dm, .. en);

エラーが発生しにくい

if ( cond) f( e1, .. em, .. en); else f( e1, .. dm, .. en);

同じ引数を使用して関数を呼び出したい場合：もう一度見つけます

(cond ? f : g)( /* long and complex argument list */);

エラーが発生しにくい

if ( cond) f( /* long and complex argument list */);
else g( /* long and complex argument list */);

私が見ているように、それぞれのケースで重要なことは、フォームの繰り返しが少ない。繰り返しにより、1つのインスタンスを変更し、もう1つのインスタンスを変更し忘れる可能性が開かれます。