ページングで標準で1ページ目が表示されるようにしたい

Question

前提・実現したいこと

データベースからのデータを表にする画面を写経してます
件数の多いレコードをページ分けするページングの貢で、
pageパラメータを指定されていないとき、変数が空なので、
1を代入するようにしたいのですが参考書と同じように記述しても動きませんでした

該当のソースコード

    <?php
    $page = $_REQUEST['page'];　//line12はここです
    if($page==''){
      $page=1;
    }                           //$pageが空の場合1を代入するようにしたい
    $page = max($page,1);
    $start = ($page -1)*10;
    $recordset = mysqli_query($link,'select * from test order by id desc limit '.$start.',10');
     ?>

    <table><!--省略--></table>

    <ul class="paging">
      <li>
        <a href="test.php?page=<?php print($page -1) ?>">前のページへ</a>
      </li>
      <li>
        <a href="test.php?page=<?php print($page +1) ?>">次のページへ</a>
      </li>
    </ul>

発生している問題・エラーメッセージ

Notice: Undefined index: page in C:\xampp\htdocs\test.php on line 12

試したこと

http://localhost/test.php?page=1
に接続するとエラーメッセージは表示されません

ならそれでいいじゃんと思うかもですが、
http://localhost/test.php
に接続してもエラーメッセージが出ない方法があれば教えてください

Answer 1

配列のキーが存在しないためです。
なので、そのキーが存在しない場合＋空の場合に１を指定してやれば良いです。

おおよそ同じような結果にできる書き方は結構あります。
試して自身の要件に合うものを選んでください。

if(!isset($_REQUEST['p']) || (isset($_REQUEST['p']) && trim($_REQUEST['p']) == "")){
    $p = 1;
}else{
    $p = $_REQUEST['p'];
}
echo $p;

//↑の三項演算子版
$p = (!isset($_REQUEST['p']) || (isset($_REQUEST['p']) && trim($_REQUEST['p']) == "")) ? 1:$_REQUEST['p'];
echo $p;



if(!array_key_exists('p',$_REQUEST) || (array_key_exists('p',$_REQUEST) && trim($_REQUEST['p']) == "")){
    $p = 1;
}else{
    $p = $_REQUEST['p'];
}
echo $p;


//↑の三項演算子版
$p = (!array_key_exists('p',$_REQUEST) || (array_key_exists('p',$_REQUEST) && trim($_REQUEST['p']) == "")) ? 1:$_REQUEST['p'];
echo $p;

また、$_REQUESTだとPOSTの場合も値をとってしまうのでこういう書き方もあります。

$p = filter_input(INPUT_GET,'p');
$p = (is_null($p) || (!is_null($p) && trim($p) == "")) ? 1:$p ;
echo $p;

※ただし、数値以外の場合の対応、データが存在しないページのときの対応は別途考慮が必要です。

Comments

[m.ts10806]

結構ありますと書いたものの、おおよそ簡単な書き方２種類しか提示できてなかったりしますけど・・・。

[aluminuser]

想定通り動きました。
複数種の解決法をご教授いただきありがとうございました。

Answer 2

勉強中の未熟者ですが質問にトライさせていただきました。

if(is_null($_REQUEST['page'])){
$page=1;
}else{
$page = $_REQUEST['page'];
}

上記「is_null]だと結果は返るのですが「Notice」エラーが出るのは回避できず

if(empty($_REQUEST['page'])){
$page=1;
}else{
$page = $_REQUEST['page'];
}

上記「empty]だと結果は返り「Notice」エラー回避

と質問者様の要望にはかなったと思いますが…
m.ts10806さんほどのお方が「empty]の案を出しておられないので
他の問題があるのかもしれません

Comments

[aluminuser]

現状emptyを利用したものでエラーを回避できたので解決したと考えました。
m.ts10806様の回答より簡潔でわかりやすかったのでベストアンサーとさせていただきます。

[m.ts10806]

jinbaさん
empty()は0も空と見なすので要件次第と思ってはずしました。
そこは私の回答の後半に書いたバリデーションでどこまでやるか次第だと思ってます。

[jinba]

m.ts10806さん
返答ありがとうございます
いつも勉強させて頂いてます