質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.61%

  • PHP

    19851questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

行儀のいい中断処理は?

解決済

回答 2

投稿

  • 評価
  • クリップ 2
  • VIEW 229

KazuhiroHatano

score 1428

複数の配列のすべての値に対して条件を満たす場合にのみ処理を行う
というようなことをする場合、一個でも条件を満たさなければ
その時点で以降の判定処理もすっとばしたいわけなのですが
その時の行儀のいいやり方について悩んでおります

while(true){
    foreach(〜 as 〜){
        if(!〜){break 2;}
    }
    foreach(〜 as 〜){
        if(!〜){break 2;}
    }
    〜処理〜
    break;
}
〜続き〜


continueで無限ループの可能性が微レ存

switch(true){
    case true:
    foreach(〜 as 〜){
        if(!〜){break 2;}
    }
    foreach(〜 as 〜){
        if(!〜){break 2;}
    }
    〜処理〜
}
〜続き〜


whileでいいんじゃないかな…

call_user_func(function(〜,〜)use(&〜,&〜){
    foreach(〜 as 〜){
        if(!〜){return false;}
    }
    foreach(〜 as 〜){
        if(!〜){return false;}
    }
    〜処理〜
    return true;
},〜,〜);


安全っぽいけどめんどい

foreach(〜 as 〜){
    if(!〜){goto hoge;}
}
foreach(〜 as 〜){
    if(!〜){goto hoge;}
}
〜処理〜
hoge:
〜続き〜


PHPにもgotoはあるんだよなぁ

個人的にはcall_user_funcが一番いいのかな、とも思ってるんですが
条件判定のためだけに関数を作っちゃうのは
結構負荷が大きいから避けるべきなのか?とかも思っちゃいます

使い分けた方がいいのか、やり方を統一しておいたほうがいいのか
あるいはこれら以外にもっと良いやり方があるのか

よろしくお願いします

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • KazuhiroHatano

    2017/12/28 14:09

    特に縛りがあったり、具体例があるわけではありません、 ただ、何度となくこういうif(〜)にまとめられない複合的条件判定を行うことがあったので どういう方針を持って臨むべきかといった悩みです

    キャンセル

  • yambejp

    2017/12/28 14:27

    直接回答ではないですがwhile(true)で1順目で強制breakするなら、do{}while(false)の方がまだましかと。それも昔どこかの技術板で書いたら「そんなクズみたいな回答するな」と怒られたやつですけどね

    キャンセル

  • KazuhiroHatano

    2017/12/28 14:37

    なるほどdo{}while(false)の方が無限ループのリスクが少ないですね

    キャンセル

回答 2

checkベストアンサー

+5

条件判定のためだけに関数を作っちゃうのは
結構負荷が大きい

ここでいう負荷といのが何かわかりませんが、関数を作るのには処理に名前が付けられるという
メリットがあります。
きちんと命名すれば、メンテナンスする人のコードリーディングの負荷は下がるかと

if (複数の配列のすべての値に対して条件を満たす(xxxx) ) {
    // 処理
} 

ところで、具体例がないのでなんともいえないですが、条件判定は2つ同時にできないのですか?

foreach(〜 as 〜){
        if(!〜 || !〜){return false;}
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/12/28 14:35

    回答ありがとうございます

    今の自分の方針は割とcall_user_func(無名関数)寄りなんですが
    「メソッド内で無名関数使うとメソッド実行のたびに関数作られちゃってないか?」
    「無名関数はPHP5.2では使えないわけだけど、PHP5.2なんてもう想定しなくていいのか?」
    といったところが悩みのタネです

    しかも複雑な条件判定というのが「判定に利用する変数がめちゃくちゃ多い」
    「使う変数が減ったり増えたりすることが割とある」ということでもあったりして
    関数・メソッドにすると引数がとんでもなく多かったり、変更に伴う修正箇所が増えたりするために
    普通に関数・メソッドを作るのではなく無名関数にしていたりします

    キャンセル

  • 2017/12/28 20:40

    引数オブジェクトの導入を検討してはどうでしょうか?抽象化された情報の塊であるオブジェクトを受け渡すのであれば、多くの値を1つの変数で受け渡せますし、呼び出し部分に手を加えることなく判定に利用する変数の増減に対応できます。

    キャンセル

+3

処理の意味合いにもよりますが、例外を使うか、関数化するのが自然に思います。

一か所でしか使わない処理に対して関数化することを負担に感じているのかもしれませんが、momon-gaさんの書かれているように処理が明確で名前が付けられることのほうが重要です。ソースコードの量だとか関数の数を減らしたいというのは書き手の都合に思います。

関数が小さくなればテストもしやすくなりますよ。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/12/28 14:20

    回答ありがとうございます

    今回の質問の契機になったものについては
    「粒度」「テストコードの書きやすさ」
    というものも考えるべきだった
    という後悔をしている最中です…

    渡すダミーデータを作るのさえ
    面倒くさいような代物だったので
    その辺をないがしろにして
    今まさにツケを食らってます…

    キャンセル

  • 2017/12/28 15:02

    ちなみにだけ書いておくと、学校の予約システムの予約制限の処理です
    講師(1〜2人)と教室によって受け入れ可能な授業の種類や人数が違い
    授業によって一度に数種類受け持てたり持てなかったり…

    最初は授業の種類は講師一人につき1種類って話だったんですけどねぇ
    「いや、この授業なら2〜3個は大丈夫です、実は」
    みたいな話になって、αテストを目前に控えた今から
    授業ごとに負荷値を設定して、予約制限の処理方法をイチから見直し…

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • PHP

    19851questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。