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

Question

お世話になっております。
単純な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文の力技で書いた次第です。

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

Answer 1

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

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

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

Comments

[Mcgregor]

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

Answer 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

• PHP: PHP マニュアル - Manual
• PHP: is_int - Manual
• PHP: array_map - Manual
• PHP: list - Manual

Comments

[Mcgregor]

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

Answer 3

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

Comments

[Mcgregor]

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

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

[退会済みユーザー]

うーん。正直見る気になりません＾＾；

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

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

[退会済みユーザー]

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

<?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');

Answer 4

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

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

Comments

[Mcgregor]

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

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