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

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

ただいまの
回答率

89.19%

配列に特定の複数の値が入っているかチェックする処理

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 5,742

Tiktak

score 54

お世話になっております。
この度は、タイトル通りですが、「配列に特定の複数の値が入っているかチェックする処理」に関してお聞きしたいと思います。

私が考えるコードでも上の処理は実現可能だと思うのですが、他の経験のある方たちはどのような書き方、処理を選ばれるのか知りたいので、ご教授いただければ幸いです。

この処理の具体的シチュエーション
・チェックボックスから「初級(beginner),中級(middle),上級(advanced)」を複数選べるようになっており、選ばれた値が入った配列を、どの値が入っているかを調べて処理を分けたい。
考えられる組み合わせは、(初級、中級、上級、初級中級、初級上級、中級上級、初級中級上級)

以下私が考えた処理です。

$difficulty = array('beginner'); /* これは初級というチェックボックスのみチェックされた時の例です*/ 

/*初級のみの場合*/
if(in_array('beginner', $difficulty)){

}/* 中級のみの場合 */
elseif(in_array('middle', $difficulty)){

}/* 上級のみの場合 */
elseif(in_array('advanced', $difficulty)){

}/* 初級と中級の場合 */
elseif(in_array('beginner', $difficulty) && in_array('middle', $difficulty)){

}/* 初級と上級の場合 */
elseif(in_array('beginner', $difficulty) && in_array('advanced', $difficulty)){

}/* 中級と上級の場合 */
elseif(in_array('middle', $difficulty) && in_array('advanced', $difficulty)){

}/* すべてチェックされた場合 */
elseif(in_array('beginner', $difficulty) && in_array('middle', $difficulty) && in_array('advanced', $difficulty)){

}


というコードを考えたのですが、もっとスマートな書き方、もしくはこの書き方に問題がある場合は指摘して頂けますと有り難いです。

初歩的な質問ですが、どうぞよろしくお願い致します。
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+1

まず、元のコードだと、初級が入っていれば、最初の「/*初級のみの場合*/」の部分に入り込んでしまうので、想定した操作をしません。基本的に条件は「厳しい方」から判断しましょう。

で、自分なら最初にin_arrayの結果を抽出して、その上で結果変数を組み合わせて条件を表現していくかなと思います。

$is_beginner = in_array('beginner', $difficulty);
$is_middle = in_array('middle', $difficulty);
$is_advanced = in_array('advanced', $difficulty);

/*3つともの場合*/
if($is_begin && $is_middle && $is_advnced) {

}

//後略

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2015/10/02 09:43

    ご回答ありがとうございます。よく考えたらそもそも私のコードが正しくありませんでしたね。。
    ご指摘ありがとうございます!
    あと、in_arrayの結果を抽出して、その変数で判定するという方法も私には新しくとても勉強になりました。またこちらで質問させて頂く時はぜひよろしくお願いします!

    キャンセル

0

ご提示のコードは意図通りに動きません。

なぜなら、if文は上から順番に評価されるため、
"初級"と"中級"がチェックされた場合でも/*初級のみの場合*/のブロックが実行されてしまうからです。

if文の順番を逆にする(/* すべてチェックされた場合 */を最初に、条件が厳しい方からチェックする)と、うまく行くと思います。

"スマートな書き方"については、私に妙案はありませんw

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2015/10/02 09:44

    ご回答ありがとうございます。
    そうですね。書き方についてのアドバイスを頂くつもりがかなり初歩的なところでミスをしていました。。
    こちらで質問しなかったら気づかなかったと思いますので、ありがとうございます!

    キャンセル

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

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