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

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

ただいまの
回答率

89.54%

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

解決済

回答 3

投稿

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

irukairuka

score 13

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

現在のページは$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されるのはおかしいのでは?と思うのですが、、

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

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 3

+3

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)


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

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

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/03/04 17:34

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

    キャンセル

  • 2019/03/04 17:38

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

    キャンセル

  • 2019/03/04 17:49

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

    キャンセル

checkベストアンサー

+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 は入れておいた方が良い。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/03/04 16:35

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

    キャンセル

  • 2019/03/04 16:36

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

    キャンセル

  • 2019/03/04 16:38

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

    キャンセル

+1

$arr[3] === $three 

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/03/04 16:06 編集

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

    キャンセル

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

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