■http://anond.hatelabo.jp/20160919121645

追記：論理積と論理和の優先順位で間違いがあったため修正しました。

まず最初の条件をif分を一回終わらせて入れ子にするか、falseならreturn させるといい。

入れ子になってもとにかく自分が分かるようにするといいです。

一行で書くのはそのあとではどうだろう

if(param == 0 && test(param2)) {
   if(!param3) {
     something todo
   }
 }

また

if(!(param == 0 && test(param2))){
  return false;
}
if(test(param2) || !param3) {
  実行分
}

あと、一つの条件式で書きたいなら、括弧を使ったり、改行したりして、式の流れを細かくすると分かりやすくなる。

2:paramが0

3：かつ

4：判定関数でparam2の条件を返して、true

5：またはparam3がfalseのときがtrueだから

なのに　param==0 ＆＆ 判定(param2) || !param3　だと

param==0 ＆＆ 判定(param2)

と

!param3

で別れるはず（言語によって違う可能性もありますが）

だけど、日本語を言葉にすると「paramが0かつ判定関数でparam2の条件を返して、trueまたはparam3がfalseのときがtrueだから」

となって、かかる範囲が分かりづらくなるから、

だから、論理演算子が複数になるときは、日本語でかつ、または使うと逆に混乱するから、

trueになる一つ目の条件の時は「○○」またはといったように書くと、

冗長的ですが、言葉にしても分かりやすいかと思います。

それ以外には既に上げていた色んな増田の方のやり方もそうだし、

変数名で分かりやすくするというのもあるぞ。

is_children = age < 20
has_game    = check_game(param2)
if(is_children && has_game) {
  print ゲームをもったこども
}

条件式は思考にブレーキがかかるからこうやって、

変数名に置き換えること、文章として読めるからブレーキがかかりにくくなる。

と、無理にワンライナーで書かない方法を導入すると理解しやすくなると思います。

Permalink | 記事への反応(2) | 18:45

ツイートシェア