現在のページに応じた条件分岐を作りたいです。

Question

現在のページに応じた条件分岐を作っているのですが、変なことが起こりました。

現在のページは$url = `$_SERVER['REQUEST_URI'];で取得しますが、今回はひとまず
$url = '/topic/dog/';としています。

で、この$urlは「topicのページ」なので、<p>topicのページ</p>だけをechoしたいのですが、、

なぜか、<p>membersのページ</p>までがechoされるという状態です。

<?php
function my_is_page( $one, $three=null ) {
	
	//URLを取得する
	//$url = $_SERVER['REQUEST_URI']; 
	$url = '/topic/dog/';
	
	//URLに応じてページ分岐
	$arr = explode( '/', $url );	
	
	if( isset($arr[1]) && is_null($three) ){
		if( $arr[1] == $one ){ 
			return true;
		}
	}
	
	if( isset($arr[3]) ){
		if( $arr[3] == $three ){
			return true;
		}
	}

}


if( my_is_page('topic') ){
    echo '<p>topicのページ</p>';
}

if( my_is_page('members') ){
    echo '<p>membersのページ</p>';
}

if( my_is_page('members','settings') ){
    echo '<p>members/settingsのページ</p>';
}

?>

私の考えでは、$arr[1]と引数$oneが等しいときだけtrueが返るハズなので、$url = '/topic/dog/';のときにif( my_is_page('members') ){}は反応せず、<p>membersのページ</p>までがechoされるのはおかしいのでは？と思うのですが、、

どうすればページ分岐が適切にできるでしょうか？

Answer 1

function my_is_page( $one, $three=null ) {

    //URLを取得する
    //$url = $_SERVER['REQUEST_URI']; 
    $url = '/topic/dog/';

    //URLに応じてページ分岐
    $arr = explode( '/', $url );    

    if( isset($arr[1]) && is_null($three) ){
        if( $arr[1] == $one ){ 
            return true;
        }
    }

    if( isset($arr[3]) ){
        if( $arr[3] == $three ){
            return true;
        }
    }

}

問題は二つ目の判定部分です。
explode('/', '/topic/dog/') の結果は、

array(4) {
  [0]=>
  string(0) ""
  [1]=>
  string(5) "topic"
  [2]=>
  string(3) "dog"
  [3]=>
  string(0) ""
}

となりますから、$arr[3] は空文字列です（意図としては $arr[2] をチェックしたいのでしょうか？）。そして PHP では論理値への変換を行う場合、次のものは FALSE となります。

• 整数値の 0
• 実数値の 0.0
• 空文字列
• 文字列としての "0"
• 空の（要素数0の）配列
• NULL

つまりこのif文が

if ($arr[3] == $three) {
↓
if ("" == NULL)
↓
if (FALSE == FALSE)

と変換されるので、成立するのです。

厳密な（型変換をしない）判定である === を使うと意図した動きになります。

Comments

[irukairuka]

おおおお！わかりやすかったです。どうもありがとうございます。

[irukairuka]

メンバーごとのフロントページはどのようにしたらいいと思いますか？

[tacsheaven]

フロントページと言うことは、
・arr[1] が "members" で、
・arr[3] が空文字列で、
・arr[2] がメンバー名
という条件で判定すればよいのかと思います（2と3の順番が逆な理由は後述）。
ただこれを一つの if 文にまとめるのではなく、ネストさせた方がよいでしょう。1 と 3 は同時にチェックして、2 だけその中でチェックするようにします。そうするとメンバー名が増えても対応がしやすいです。

[irukairuka]

ありがとうございます。さきほどはemptyが思いつかず追加で聞いてしまいましたが、下記で解決致しました。（メンバー名は飛ばしましたが、だいじょうぶですよねたぶん笑）

➀この条件を追加し
if( isset($arr[1]) && $arr[1] == $one && $three == 'front' && empty($arr[3]) ){
return true;
}

➁このように判定しました
if( my_is_page('members','front') ){
echo '<p>membersのfrontページ</p>';
}

[tacsheaven]

ん？　なんか変なような……
if ($arr[1] == 'members' && $arr[3] == '') {
if ($arr[2] == 'john') {
return true;
}
if ($arr[2] == 'mike') {
return true;
}
}
return false;

こんな感じですよ？

[irukairuka]

あ、なるほど。そうすれば「メンバーそれぞれのフロントページ」が判定ができますね。私のはとりあえず「メンバーのフロントページ」としか考えておりませんでした。ありがとうございます。

[tacsheaven]

もう少し判るようになると、メンバー名のリストを別のところから持ってきて、ループを組み合わせてチェックするとかいろいろやれるようになります。が、それはおいおい理解していけばいいでしょう。

[irukairuka]

既存の関数があるのでなんとなくできそうです。今回はこれにて。引き続き挑戦してまいります。最後までお付き合い頂きまして、どうもありがとうございました。がんばります！

Answer 2

こういうときこそデバッグです。
__LINE__は「現在のプログラムの行数」を出力してくれますので、
そちらも利用します。

function my_is_page( $one, $three=null ) {

    echo $one."\n";
    //URLを取得する
    //$url = $_SERVER['REQUEST_URI']; 
    $url = '/topic/dog/';

    //URLに応じてページ分岐
    $arr = explode( '/', $url );

    if( isset($arr[1]) && is_null($three) ){
        echo __LINE__."\n";
        if( $arr[1] == $one ){ 
        echo __LINE__."\n";
            return true;
        }
    }

    if( isset($arr[3]) ){
        echo __LINE__."\n";
        if( $arr[3] == $three ){
        echo __LINE__."\n";
            return true;
        }
    }
}

出力結果：

topic
13
15
<p>topicのページ</p>members
13
21
23
<p>membersのページ</p>members
21

2つ目、「my_is_page('members')」を実行したときに
21行目、23行目を通ってtrueを返していることが分かります。

    if( isset($arr[3]) ){
        echo __LINE__."\n";
        if( $arr[3] == $three ){
        echo __LINE__."\n";
            return true;
        }
    }

$arrの[3]が存在していて、$threeと同じだったら、、、ですね。

では、membersのときだけここで何がきているか出すようにしてみましょう。

    if($one === "members"){
        var_dump($arr);
        if( isset($arr[3]) ){
            echo __LINE__."\n";
            if( $arr[3] == $three ){
            echo __LINE__."\n";
                return true;
            }
        }
    }

出力結果：

members
array(4) {
  [0]=>
  string(0) ""
  [1]=>
  string(5) "topic"
  [2]=>
  string(3) "dog"
  [3]=>
  string(0) ""
}
23
25
<p>membersのページ</p>

はい。
$arrの3は""です。
一応、==で比較していると""とnullは同じとみなされます。
下記のように修正することで対応可能

        if( isset($arr[3]) ){
            if( $arr[3] === $three ){
                return true;
            }
        }

なぜか？
/topic/dog/を/で分割されているわけですが、

[]/topic/dog/[]
↑[]で囲っているところが含まれるからなんですね
ミニマムコードで試すとこんな感じ。

$url = '/';
$arr = explode( '/', $url );
var_dump($arr);
/*
array(2) {
  [0]=>
  string(0) ""
  [1]=>
  string(0) ""
}
*/

もしこの現象をなくしたければ「最初と最後は除く」みたいなのが必要かと思います。

---

蛇足：分岐入れてて更にifとか入れるのはいいですが、何も処理がないならまとめたほうが良い

if( isset($arr[1]) && is_null($three) && $arr[1] == $one){
  return true;
}
if( isset($arr[3]) && $arr[3] == $three){
  return true;
}

「その関数の全ての分岐を実行されない」こともあるので、一応明示的に
最後にreturn false は入れておいた方が良い。

Comments

[irukairuka]

これはすごい！デバッグの方法と流れを見せて頂き納得できました。__LINE__も便利ですね。ご丁寧にありがとうございます。大変勉強になりました。

[m.ts10806]

解決されたようで何よりです。
デバッグは今後も活用してください。質問する前に解決できることも多くなるはずです

[irukairuka]

はい！どうもありがとうございました。

Answer 3

$arr[3] === $three 

Comments

[irukairuka]

できました。ありがとうございます。