• Twitterでシェア
  • Facebookでシェア

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

テクノロジー カテゴリーの変更を依頼 記事元:anond.hatelabo.jp
適切な情報に変更
329 usersがブックマーク コメント169
    共有

    • 記事へのコメント169件

    • 注目コメント
    • 新着コメント
    その他
    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
    実コードでこんな条件分岐書いてあるの出てきたら、これは筋が悪いな...と他の部分も合わせて警戒する。

    その他
    good2nd
    自分の頭の悪さをどうカバーするかを考えない人は向いてないよな。複雑なことはすぐ間違えるって前提でやらないと。

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

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

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

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

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

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

    その他
    naka-06_18
    わかる。しんどい

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

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

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

    その他
    katzchang
    ド・モルガンの呪いだ

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

    その他
    hush_puppy
    ネストを増やせ。ネストを浅くしろ。

    その他
    atrandom2520
    括弧つけよう

    その他
    inaba629
    そもそも書き方に問題がある気がする

    その他

    注目コメント算出アルゴリズムの一部にLINEヤフー株式会社の「建設的コメント順位付けモデルAPI」を使用しています

    アプリのスクリーンショット
    いまの話題をアプリでチェック！
    • バナー広告なし
    • ミュート機能あり
    • ダークモード搭載
    アプリをダウンロード

    関連記事

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

    if(param==0 ＆＆ 判定(param2) || !param3){ //したいこと } みたいに3つぐらい条件がある場合、頭がパ...

    ブックマークしたユーザー

    すべてのユーザーの
    詳細を表示します

    同じサイトの新着

    同じサイトの新着をもっと読む

    いま人気の記事

    いま人気の記事をもっと読む

    いま人気の記事 - テクノロジー

    いま人気の記事 - テクノロジーをもっと読む

    新着記事 - テクノロジー

    新着記事 - テクノロジーをもっと読む

    同時期にブックマークされた記事

    いま人気の記事 - 企業メディア

    企業メディアをもっと読む

    はてなブックマーク

    公式Twitter

    はてなのサービス

    • App Storeからダウンロード
    • Google Playで手に入れよう
    Copyright © 2005-2025 Hatena. All Rights Reserved.