ifの条件が2つ以上だと頭がパンクする

if(param==0 ＆＆ 判定(param2) || !param3){ //したいこと } みたいに3つぐらい条件がある場合、頭がパンクしそうになる。 この時こう考えている。 1:したいことをする条件は 2:paramが0 3：かつ 4：判定関数でparam2の条件を返して、true 5：またはparam3がfalseのときがtrueだから 6：trueだとだめなんだよね。falseが正しいんだ。でもfalseは偽なのに正？ 8:あれ、何したいんだっけ。まぁいいや、とりあえず実行してエラーなら直そう。 7：実行。 8:あれ？なんか違う。何がおかしいんだ 1へ戻る。 これを3,4回繰り返してウアアアアアアアアアアアってなってしまう。 誰か助けて

[AQM]

ANDとORと繋がったままif実行するなんて頭がフットーしそうだよおっっ

[yamamoto-h]

しそうってなんだよ。パンクする条件定義しっかりしろよ。

[fuji_haruka]

if (!条件1) return; if (!条件2) return; if (!条件3) return; って感じで条件に合わないものを少しずつ除外するようなコードを書けば可読性があがるしデバッグしやすいよ。

[kuro_pp]

困難は分割せよ。条件が増えたらもっと単純な条件文に分割すべき。

[shiraivin]

ベン図を書こう

[htnmiki]

サブウェイ「ご注文は？パンの種類は？苦手なものは？トッピングは？ソースは？サイドメニューは？ドリンクは？」　たかがサンドイッチ食うために2、3回の転生が必要なのか……

[aukusoe]

notが絡むorだと一瞬テンパる。大学のテストの「超簡単だけど配点デカい留年を出さないためのサービス問題」をケアレスミスしたときの苦い経験を思い出すからだと、自己分析してる。

[Palantir]

一旦ORとAND使うのやめればいいのでは？ 静的解析上は深度も変わんないはずだし。 IFの上に目的をコメントすることはある。

[sonots]

rubyにはunlessというのがありまして、あれ使われるとな…

[irasally]

実コードでこんな条件分岐書いてあるの出てきたら、これは筋が悪いな...と他の部分も合わせて警戒する。

[sennkjp]

わかる

[ch1248]

ブコメでも言われてるが、リファクタリング案件。パッと見て理解しにくいコードは保守性が低い。

[trashtoy]

自分だったら if 文の中身を bool 型の値を返す関数として切り出しておく

[HalSakura]

無駄だと思っても()使う。（後から読む人の為にも）

[vvakame]

こんなん俺だってわからんわ リファクタリングしよ

[otiai10]

早期returnが好きです

[koyancya]

unless ガード節をキメる場面だ

[junorag]

自分が頭がパンクするなら、その箇所は他人が後で呼んでもパンクする可能性がある。括弧でくくれ。ifを分割してもいい。わかりやすく書け。

[t-wada]

こういうときはリファクタリング「条件記述の分解」をしましょう

[minoru0707]

コメントを書け

[masaru_b_cl]

param == 0、判定関数 param2、param3に全部ちゃんと名前つけてあげよう

[b_taro]

一行ずつif文作れや

[nabe1121sir]

このコードはセンスがない。

[uunfo]

「2つ以上」と「2つ超」を区別しない思考をしている限り無理だな／結合順序は忘れて常に括弧で優先順位を指定すべき

[kamayan1980]

プログラマの必須能力である「情報を整理する能力」が欠けているので、もっとレベルあげよう。具体的に言うと、基本情報処理技術者試験の午後のプログラム問題がオススメ。

[ka-ka_xyz]

頭の中でテンポラリにまとめられないなら、メモ紙に真偽値表を書き出せばいいだけでは。（この程度の条件であればコードの書き方を変えてもあまり意味が無いと思う）

[pmint]

||はね、別の行に書くんだよ。で、各行ごとに正しいか検証する。if ... else if ... に置き換えることもできるけど、同じコードを複数書くことになる。別段複雑ではないので、コードはこのままでも可。

[babi1234567890]

分割し統治せよってローマ人が言ってた

[yusuke-k]

こういう時は条件に名前を付けるといい。hoge = param == 0; fuga = param2 || !param3;とすれば if (hoge && fuga) {...} と簡潔になる。hogeとfugaに適切な名前を割り振れないならそもそも書きたいプログラムを理解できてない。

[orenonihongogayabai]

マジレスするとこのif文をtrue/falseのみ返すprivateな判定用関数にして、関数内部ではor条件は使わず別のifとして記述すると読みやすくて楽。リファクタリング楽しいです。

[naka-06_18]

わかる。しんどい

[houyhnhm]

一息ついてから、逆の、「処理しない条件」を並べてbreakなりreturnなりするように書き換える。構造化プログラミングくらいの次元の話。

[hayashikousun]

判定用の関数に括りだせ，ベン図かカルノー図か真理値表を書いて脳内を整理しろ，後で読むときにはそのベン図を見ろ

[anoncom]

条件毎に()で括らないから悪い。そういうコードのメンテナンスを任されたなら、ご愁傷様としか言いようがない。

[katzchang]

ド・モルガンの呪いだ

[HHH8383]

修正行数をできるだけ少なくしなきゃいけない雰囲気が強い時は、こう直すかな。 !param3 || param==0 && 判定(param2) これならまだ理解しやすいし、判定メソッドの呼び出し回数減らせる

[hush_puppy]

ネストを増やせ。ネストを浅くしろ。

[atrandom2520]

括弧つけよう