三項演算子は可読性を落とすか - Qiita

しばしば三項演算子やdo-while構文が禁止されているコーディング規約に遭遇する。それは今すぐ撤廃すべきという論理武装を考えてみよう。事例はPHPだが、このことはすべての言語にいえる。 例 あるサービスから何らかのデータを取り出し、それをクライアント側で使うという手続きを考える。 ここで、サービス側のAPIとクライアント側のAPIに仕様ギャップがあったとしよう。$service は古く、うまく行かなかったとき例外ではなく null 的なもの(PHPではもしかするとfalseかもしれない)を返す習慣で作られていた。いっぽう $client は null を入力される想定がない。何もしないで欲しいときは空の array を入力する仕様だった。 以下の実装例を見てほしい。 <?php $data = $service->getSomeData(); if ($data === null ||

[at_yasu]

三項演算子の入れ子はアレだけど、入れ子になってないならアリだと思う。

[nutahuate]

“動くだけのコードから、動かなくなる余地のないコードへ”

[decoy2004]

1行に詰め込んでるとカバレッジやステップ実行する時にやや不便。演算子使うのはいいけど、メソッドならこの不便はない。

[rajahbrooke]

三浦理恵子に空目した

[otherworld]

$bar = $foo ? true : falseという使い方はやめてほしい

[PowerEdge]

エルビス演算子いいなー知らんかった

[sugawara1991]

“ミュータビリティを式の中に封印できる 変数がイミュータブルになる”保守のことは特段に考慮しないけど、このへん常には意識してる

[karasu9113]

三項演算は使うべきだと思う。たった一行200文字以内で済むところに5行も使いたく無いし、200超えてても改行ポイント間違わなければ可読性は保たれる。if文内に変な処理仕込まれてリファクタリングが複雑化もしないし

[n314]

自分なら $client->useSomeData($serviceWrapper->getSomeData()); こうするかな。

[JULY]

もっと古典的に「goto 文を使うな」の同じで、それを使っていたずらに複雑な構造を作り込むことがダメなのであって、機械的に禁止するのは間違い。あと、根底に、保守はワンランク下、という偏見があると思う。

[k-holy]

あれってElvis Operatorっていうのね。これが語源？ http://stackoverflow.com/questions/1993409/operator-php