MySQLを利用したページング

Question

前提・実現したいこと

PHPを勉強している者です。
MySQLからデータを取得して、5行ずつページに表示させるページングを作りたいのですが、わからないことが多く、躓いております。select文のlimitとoffsetに$nowを使ってページごとに変わる計算結果を入れて、１ページ目なら１行目～５行目、２ページ目なら６行目～１１行目というように表示させたいと考えております。
様々な参考サイトのコードを参照して、つぎはぎ状態になっている部分があるので、お見苦しい部分多いかと思いますが、ご容赦いただけますと幸いです。

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

該当のソースコード

<?php
$db=new PDO('mysql:dbname=$dbname;host=$dbhost','$user','$pass');

$offset=($_GET['page_id'] - 1) * 5;
$limit= $offset+5;

$query="select * from touroku order by date desc limit :limit offset :offset;";
$saisho="select * from touroku order by date desc limit :limit offset :offset;";
$zenbu="select * from touroku order by date desc;";

$meirei_limit=$db->prepare($query);
$meirei=$db->query($zenbu);
$meirei_saisho=$db->prepare($saisho);

$meirei_saisho->bindvalue(':limit',5,PDO::PARAM_INT);
$meirei_saisho->bindvalue(':offset',0,PDO::PARAM_INT);

$meirei_limit->bindvalue(':limit',$limit,PDO::PARAM_INT);
$meirei_limit->bindvalue(':offset',$offset,PDO::PARAM_INT);

$meirei_limit->execute();
$meirei_saisho->execute();

$a=$meirei_limit->fetchAll();
$first=$meirei_saisho->fetchAll();
$result=$meirei->fetch();

$total=count($result);

define('max','5');

$max_page = ceil($total/ max);

if(!isset($_GET['page_id'])){
    	$now = 1;
	foreach($first as $value) {
		print($value["date"].'　'.$value["title"].'<br />');
	}
}else{
  	$now = $_GET['page_id'];

	foreach($a as $value) {
		print($value["date"].'　'.$value["title"].'<br />');	
	}
}

print('全件数'.$total.'件.　');

if($now > 1){
    echo '<a href="/pager.php?page_id='.($now - 1).'")>前へ</a>". "　';
} else {
    echo '前へ'. '　';
}
 
for($i = 1; $i <= $max_page; $i++){
    if ($i == $now) {
        echo $now. '　'; 
    } else {
        echo '<a href="/pager.php?page_id='. $i. '")>'. $i. '</a>". "　';
    }
}
 
if($now < $max_page){
    echo '<a href="/pager.php?page_id='.($now + 1).'")>次へ</a>". "　';
} else {
    echo '次へ';
}

unset($db);

?>

試したこと

最初に
https://www.sejuku.net/blog/70234
こちらのサイトを参考にさせていただきまして、後半部分に関してはほとんどコピペです。

修正依頼をいただいたのち、PDOに変えて修正しました。

修正し、やりたかった動きにできました。削れる部分はまだあるかと思いますが、javahack様、skys215様　のお力添えのおかげです。ありがとうございました！

補足情報（FW/ツールのバージョンなど）

Comments

[Orlofsky]

URLは https://teratail.com/help/question-tips#questionTips3-7 の [リンク] で [リンク先の説明](http...) に修正してください。

[mame9]

失礼しました。修正いたしました。

[skys215]

foreach($data as $value) {
var_dump($value); //<--でvalueの値を見る
で$valueにdataやtitleがあるかどうか確認してみましょう

[skys215]

$meirei_limit = $db->prepare("select * from list order by date desc limit ? offset ?;");
$result_a=mysqli_query($db,$meirei);

limitとoffsetでページングをしようと思ってますけど、bind_paramせずにqueryを行ったのでmysql側ではページングできてませんと思います。
queryを実行する前にbind_paramを用いてlimitとoffsetに値を与えましょう。

[javahack]

limit、offsetを入れたSQLの方は（まともに動いてなさそうですが）そもそも実行結果をつかっていません。今のコードは全件取得して配列に入ったデータをページ毎に5件ずつ取り出す仕組みになっています。
（リンク先のコード自体がそのように作られています）
5件ずつのデータ取得をPHP側でやるのかDB側でやるのかはっきりさせてください。

コードのコピペは別に構いませんがそのコードが何をやっているのか理解するようにしてください。

[mame9]

skys215様、javahack様
修正依頼ありがとうございます。
プリペアドステートメントについてよく理解できておらず、調べながらのため修正に少し時間がかかるかもしれませんが、また修正させていただきます。本当にありがとうございます。

[mame9]

skys215様、javahack様
SQL部分、修正させていただきました。MySQLのほうはどうにかlimit、offsetに値を設定できているかなと勝手に思っていて、自分の中では、$nowからうまく値をとれていないための失敗かなと思っているので、現在そこで試行錯誤しています。もし気が向かれましたら、またご助力いただけると幸いです。

[javahack]

コードを修正されたようなのでヒントを追加します。
・全件数は別途SQLを発行して取得する。
（'select count(*) as cnt from touroku' とか）
・作成したSQLを実際にクライアントDBツールで実行してどのような結果になるか確認する。
（SQL発行できっかり5件取得ならその後のarray_slice()は不要）

[mame9]

javahack様
ありがとうございます！
別途SQLを取得したほうが良いのかは悩んでいた部分だったので、とても勉強になります。
また見直し、修正させていただきます。

[mame9]

修正後のソースコードで、なんとか望んでいた動きに出来ました！
まだ削れる部分はあるかと思いますが、
javahack様、skys215様、お力添えをいただき、本当にありがとうございました。

[javahack]

解決したのなら、ご自分で回答を書いてベストアンサーをクリックし解決済みにしてください。
https://teratail.com/help#resolve-myself

Answer 1

解決済みです。
ありがとうございました。