phpでこのswitch構文おかしくないですか？

質問

phpでこのswitch構文おかしくないですか？

https://www.flatflag.nir87.com/switch-622

59と60を変数に入れても出力できないですよねこれだと？

回答No.2

t_ohta · ベストアンサー

59 と 60 は判定外なので出力できませんね。
因みに 18 も出力されませんね。
コメント通りに出力したいのであれば

switch ($age) {
　// 年齢が18歳未満だったら…
　case $age < 18;
　　echo '18歳未満';
　　break;
　// 年齢が18歳〜59歳だったら…
　case $age > 17 && $age < 60;
　　echo '18~59歳';
　　break;
　// 年齢が60歳以上だったら…
　case $age > 59;
　　echo '60歳以上';
　　break;
}

と書くのが正解でしょうね。

お礼

ありがとうございます
１８歳もですね
すごくわかりやすいし納得できたのですっきりしました

回答No.5

dell_OK

回答No.3です。
くうう、むずかしすぎましたか、私の落ち度ですみません。
「59と60を変数に入れても出力できない」事とは無関係なので、回答ではなくなりますが、今一度チャンスをください。

caseに画像のような式を書くと、それぞれが評価されるとtrueかfalseになります。
その値とswitchの括弧内の値が等しいかどうかで分岐されます。
なので以下のように書くべきだと言いました。
$age = 41;
switch (true) {//$ageを書かない
case $age < 18://「41 < 18」なのでfalse
echo '18歳未満';
break;
case $age > 18 && $age < 59://「41 > 18 かつ 41 < 59」なのでtrue
echo '19~59歳';
break;
case $age > 60://「41 > 60」なのでfalse
echo '60歳以上';
break;
}

そしてswitchの括弧内に$ageを書いてはいけないことも付け加えました。
なぜならcaseの評価値の型に合わせて、switchの括弧内の値が等しいかどうかで分岐されるからです。（これは私の憶測なのでPHPの仕様とは異なるかも知れません）
caseの評価値はtrueかfalseなので、$ageが0の場合は$ageがfalseとみなされてしまいます。
$age = 0;
switch ($age) {//0なのでfalseとみなされる
case $age < 18://「0 < 18」なのでtrue
echo '18歳未満';
break;
case $age > 18 && $age < 59://「0 > 18 かつ 0 < 59」なのでfalse
echo '19~59歳';
break;
case $age > 60://「0 > 60」なのでfalse
echo '60歳以上';
break;
}
結果『「0 > 18 かつ 0 < 59」なのでfalse』と『0なのでfalseとみなされる』でどちらもfalseで等しいので、ここへ分岐されてしまいます。
「break;」があるので最初に見つかったここだけで、次の「false」は無視されます。
期待としては、0歳は'18歳未満'であって欲しいはずなのに、そうなりません。

少しはわかりやすくなりましたでしょうか。

お礼

私の質問は、
59と60を変数に入れても出力できないですよねこれだと？
これだけなので、

回答No.4

dell_OK

回答No.3です。
興奮して読み返すのを怠りましたので訂正します。
「浅井署」は「最初」に読みかえてください。

お礼

ありがとうございます

回答No.3

dell_OK

おかしいです。
いろいろとおかしいです。
それに、投稿者は勘違いされていると思います。
見てわかる簡単なことは割愛しますが、私が重視する点だけ説明させていただくと、caseに式を書く場合は、switchの括弧内はtrueを書くべきです。でなければせめてfalseです。
投稿者自身も書かれていますが「ダブって使っているので冗長なコードになっている」と言うのはswitch括弧内の「$age」のことだと思います。
これはゼロ以外の場合は暗にtrueであり、ゼロの場合は暗にfalseとなります。switch(true)やswitch(false)と書いたのと同じと言うことです。
そのうえで、caseの「$age < 18」や「$age > 18 && $age < 59」や「$age > 60」がそれぞれtrueかfalseか評価され、switchの括弧内と等しいか否かが判定されるものです。
$age=41ならswitch括弧内はtrueなので、caseのうち最初に見つかったtrueの条件に分岐します。
それを、投稿者は、switchに「$age」を書いて、さらにcaseにも「$age」を書くのが書き方だと思っているようです。
$age=0で試すとわかりますが、これはfalseになるので、「$age < 18」がtrueのためここは処理されず、次の「$age > 18 && $age < 59」が浅井署に見つかったfalseのためここが処理されます。
switchはあくまで、括弧内の値とcaseの値が等しいかどうかを見ているのであって、大なり小なりを見ているわけではありません。

お礼

ありがとうございます
むずかしすぎてさっぱりわかりません

回答No.1

GENESIS

もしかすると13行目の59を60にする必要があるのかなと思います。
そうすれば60歳以下なので59歳は引っかかるのでは。
ただそうなると60歳は何処へ行った？ってな事に。
60歳は60歳以上に含まれるので>=にするべかなと。

まあ、あくまでもサンプルなので…。

お礼

このページの作者の人がまちがっていますよね