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

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

ただいまの
回答率

91.04%

  • PHP

    17223questions

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

PHPの条件分岐で分岐が上手く機能しない

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 129

Mcgregor

score 4

お世話になっております。
単純なif文の条件分岐なのですが、何故か思い通りに動いてくれません。
教本やネットで調べて、色々試したのですが上手くいかず。
ご助言をいただけると助かります。
またシンプルな記述等あれば教えていただけると幸いです。

状況
変数3つ
$a,$b,$cの代入された値に応じて分岐をしたいと思っています。
それぞれの変数には以下の様に値が入ります。
$a = 0以上の数字、もしくはallという文字列
$b = 0以上の数字、もしくはallという文字列
$c = 文字列、もしくはallという文字列

以下、2つのコードを試したのですが、上手く分岐しませんでした。

if (($a >= 0) && ($b >= 0) && ($c != 'all')) {
  echo "int, int, not all";
} elseif (($a >= 0) && ($b == 'all') && ($c == 'all')) {
  echo "int, all, all";
} elseif (($a == 'all') && ($b >= 0) && ($c == 'all')) {
  echo "all, int, all";
} elseif (($a == 'all') && ($b == 'all') && ($c != 'all')) {
  echo "all, all, not all";
} elseif (($a >= 0) && ($b >= 0) && ($c == 'all')) {
  echo "int, int, all";
} elseif (($a >= 0) && ($b == 'all') && ($c != 'all')) {
  echo "int, all, not all";
} elseif (($a == 'all') && ($b >= 0) && ($c != 'all')) {
  echo "all, int, not all";
} elseif (($a == 'all') && ($b == 'all') && ($c == 'all')) {
  echo "all, all, all";
}


このコードだと$cの値が正常に分岐しれくれません。

if (($a == 'all') && ($b == 'all')) {
  if ($c == 'all') {
    echo "all, all, all";
  } else {
    echo "all, all, not all";
  }
} elseif (($a >= 0) && ($b >= 0)) {
  if ($c == 'all') {
    echo "int, int, all";
  } else {
    echo "int, int, not all";
  }
} elseif (($a >= 0) && ($b == 'all')) {
  if ($c == 'all') {
    echo "int, all, all";
  } else {
    echo "int, all, not all";
  }
} elseif (($a == 'all') && ($b >= 0)) {
  if ($c == 'all') {
    echo "all, int, all";
  } else {
    echo "all, int, not all";
  }
}


こちらのコードだと、$bの値が分岐してくれないという状況です。

最初はswitchで書くのが短く綺麗になるかと考えたのですが、
書き方が思いつかず、if文の力技で書いた次第です。

私がそもそもこの記述が間違ってるかどうか、あまり判断がついていないレベルで、
教本やネットの記述を真似て必要に応じ書き換えて試したのですが、
どうも解決に至らないので、お力添えをよろしくお願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 4

+3

解決済みではありますが、

正常分岐しない理由としては

PHP比較演算子

にある
== と ===の使い分けが出来ていないからの様に思われます。
 >= 0は整数値であるかどうかを判定するには不適切な条件です。

例えば、以下のコードの場合、二つ目のifではallが数値と判定されます。
比較演算子についてはリンクの公式マニュアルに詳細な記述があるので、参考になるかと思います。

<?php

$a = 0;
if($a >= 0){
        echo "a is int".PHP_EOL;
}

$a = "all";
if($a >= 0){
        echo "a is int".PHP_EOL;
}else{
        echo "a is all";
}

実行結果

a is int
a is int

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/29 20:27

    とても勉強になりました。
    原因がわかったので、とてもスッキリしました。
    単純な比較だけでも、しっかりと理解や知識がなかったことがわかりました。
    ありがとうございます。

    キャンセル

checkベストアンサー

+2

こういうことでしょうか。

echo ( $a === 'all' ? 'all' : 'int' ).', '.( $b === 'all' ? 'all' : 'int' ).', '.( $c === 'all' ? 'all' : 'not all' );

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/29 20:13

    ご回答ありがうございます。
    それぞれの処理をecho ”int, int, all”の箇所に記述つもりでいます。

    ご教示いただいたコードで変数の値を判定して、それを元に条件分岐を行う様に
    修正してみようと思います。

    キャンセル

+2

kei344 さんも書いてくれてますが、まとめて評価するより、それぞれを評価したものをくっつけるといったアプローチのほうがきれいになります。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/29 20:15

    ご教示いただいた方法で新たに試してみようと思います。

    因みにですが、上記の記述が何故正常に分岐できないか、
    よくわからないのですが、もしわかれば教えていただけると助かります。

    キャンセル

  • 2017/12/29 20:22

    うーん。正直見る気になりません^^;

    条件式が、ごちゃごちゃしている上、`==` が入っているので。。。

    こういった条件は「どうきれいに書くか」と条件式は「型変換の無い式で評価する」事が重要です。

    キャンセル

  • 2017/12/29 20:33

    これやってみるとイイですよ。

    <?php
    echo "0 == 'all' : ";
    var_export(0 == 'all');
    echo PHP_EOL;
    echo "1 == 'all' : ";
    var_export(1 == 'all');
    echo PHP_EOL;
    echo "1 == '1all' : ";
    var_export(1 == '1all');

    キャンセル

+2

「こういった書き方もあるんだな」くらいに思って流し読みしてみてください

<?php

function filterValue($ar) {
  // 配列に対して 関数(無名関数)の処理を適用
  // 値を if で判別して文字列に変換する
  $ret =
    array_map(function($elm) {
      if($elm === 'all') { return 'all'; }
      else if(is_int($elm) && $elm >= 0) { return 'int'; }
      else { return 'not all'; }
    }, $ar);
  // 配列を指定した文字列で区切りながら1つの文字列として結合
  return implode(', ', $ret);
}

// Memo:
// * $a, $b, $c 含めた取りうる値
//   - 0以上の数字, all あるいはその他の文字列
// * $a, $b, $c の値をもとに変換する値の種類
//   - 0以上 -> 'int'
//   - 'all' -> 'all'
//   - 'foo' -> 'not all' (all 以外の任意の文字列)

// list 関数で配列と対応付けた値の代入処理
list($a, $b, $c) = ['all', 1, 'foo'];

// 定義した関数の呼び出しと戻り値の出力
echo filterValue([$a, $b, $c]); // => all, int, not all

ポイントとして:

  • 「与えられた値をもとに別の値に変換する」というのは、それだけで関数を1つ設ける大きな処理の塊です
  • 幾つかの値を同じようなルールで処理するというのは、それらへの対応を個別に定義するのではなく、ループ処理の出番です

PHP が標準で用意している関数は数多くあるので、少しでもそれを知ることが簡潔なコードを書く近道なのかもしれません

何か参考になれば幸いです

 Links

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/29 20:23

    ありがとうございます。
    やはり教本に載ってることだけだと、
    少しでも複雑な動作をしようとした時に冗長なコードになりがちですが、
    おっしゃられるように、様々な関数を学ぶことが簡潔なコードを書く近道なのでしょう。
    引き続き精進してまいります。

    キャンセル

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

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

関連した質問

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

  • PHP

    17223questions

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