任意の１件を表示した後、「前へ」「次へ」で古い記事 、新しい記事へ移動したい。

Question

お世話になります。

やりたいこと：
イベントのリストから任意の１件を選択表示後、
「前へ」「次へ」　で古い記事　、新しい記事へ移動する。

条件：event_date 順　（event_dateは手入力）
主キー、作成日、更新日　はありますが並びには関係ありません。

質問：日付を遡ったり、進んだり移動するが、レコードの最後までいくと当然表示されない状態で終わる
その先にレコードがない場合「前へ」「次へ」を表示させないための処理がわかりません。
また、このやり方ではできない場合、他に良い方法があれば教えて下さい。

１ページ〇件表示するページャーはサンプル等がたくさん出てきて他のページで利用できました。
よろしくお願いいたします。

<?php
if(!empty($_GET['id'])){	// 一覧から選択された場合
    $sql = 'SELECT * FROM article
    where article.id='.$_GET['id'].' AND category="event"';

}elseif(!empty($_GET['do'])){
    if($_GET['do'] == "next"){		// 次へが押された場合
        $sql = 'SELECT * FROM article
        where event_date > "'.$_GET['date'].'" AND category="event" ORDER BY event_date ASC LIMIT 1';

    }elseif($_GET['do'] == "prev"){	// 前へが押された場合
        $sql = 'SELECT * FROM article
        where event_date < "'.$_GET['date'].'" AND category="event" ORDER BY event_date DESC LIMIT 1';

    }else{	// 最新記事が押された場合							
        $sql = 'SELECT * FROM article
        where category="event" ORDER BY event_date DESC LIMIT 1';
    }
}

// ～～～～～～～～～～～～～～～～　省略　～～～～～～～～～～～～～～～～～～～～～//

echo "<div>";
echo '<a href="event.php?do=prev&date='.$result["event_date"].'">«前へ</a> ';
echo '<a href="event.php?do=top">最新記事</a>';
echo ' <a href="event.php?do=next&date='.$result["event_date"].'">次へ»</a>';
echo '</div>';

echo '<h2>'.htmlspecialchars($result["title"], ENT_QUOTES).'</h2>';
echo '<p>'.$result["event_date"].'</p>';
$contents = htmlspecialchars($result["contents"], ENT_QUOTES);
$contents = nl2br($contents);	
echo '<div>'.$contents.'</div>';

Answer 1

固定で

echo "<div>";
echo '<a href="event.php?do=prev&date='.$result["event_date"].'">«前へ</a> ';
echo '<a href="event.php?do=top">最新記事</a>';
echo ' <a href="event.php?do=next&date='.$result["event_date"].'">次へ»</a>';
echo '</div>';

を表示するんじゃなくて、表示する前に前記事/後記事が存在することを確認するべきです。
つまり、do=prevやdo=nextのパラメータをもらってから判断しては遅いということです。

あと、こういうのは、表示するべき記事idを直接もらって表示するものなので、
リンクurl上でid=いくつ みたいにするべき。

Comments

[reiko_y]

とても参考になりました。
ありがとうございました。

Answer 2

SELECT文発行することで対象IDを取得するのが王道ですね。
一括でSELECTする方法が分からないのでこれで終了とさせていただきます。

//ページ表示データ

if(!empty($_GET['id'])){   
    $sql = 'SELECT * FROM article
    where article.id='.$_GET['id'].' AND category="event"';
}

// ～～～～～～～～～～～～～～　省略　～～～～～～～～～～～～～～～～//

//以下を追加（現在表示中のデータより前、後にデータがあるかどうか）

$sql = 'SELECT id FROM article where category="event" 
AND event_date < "'.$result["event_date"].'" ORDER BY event_date DESC LIMIT 1';

$stmt = $db->prepare($sql);
$stmt->execute();
$prev = $stmt->fetchColumn(0);

$sql = 'SELECT id FROM article where category="event" 
AND event_date > "'.$result["event_date"].'" ORDER BY event_date ASC LIMIT 1';

$stmt = $db->prepare($sql);
$stmt->execute();
$next = $stmt->fetchColumn(0);

$sql = 'SELECT id FROM article 
where category="event" ORDER BY event_date DESC LIMIT 1';

$stmt = $db->prepare($sql);
$stmt->execute();
$top = $stmt->fetchColumn(0);	

//ページャー表示部分
if(!empty($prev)){
echo '<a href="event.php?id='.$prev.'" class="page_next">«前の記事</a>　';
}

echo '<a href="event.php?id='.$top.'" class="page_top">最新記事</a>';

if(!empty($next)){
echo '　<a href="event.php?id='.$next.'" class="page_next">次の記事へ»</a> ';
}


				

Comments

[m.ts10806]

unionすればSQL何回も実行しなくていいです。

[退会済みユーザー]

fetchColumn()って引数無しで使うと、1つ目のカラムを読み出す仕組みってのをわかって使っているんならいいけど、本当はデータ構造の変更にも堪えられるよう fetchColumn('id') みたいに引数を補うべきではないかと思ったり。
そして、前記事や後記事が存在しないときに $prev や $next にどういう値が入っているかまでしっかり確認したなら、文句はないかな。

[reiko_y]

お二人ともご教授ありがとうございました。
union 調べてみます。
$prev や $next　は　if(!empty($prev)){}　でした。
あと、fetchColumn('id')　だと　
fetchColumn() expects parameter 1 to be int, string エラーになります。
fetchColumn(0)でいいですか？
超初心者なのでまだまだ、勉強が必要です・・・。

[退会済みユーザー]

fetchColumn()、引数は数値のみですね、失礼しました。
なので、SELECT * FROM～じゃなく、SELECT id FROM ～なら、確実に0でidの取得になりますよね。

[m.ts10806]

あでも、UNIONだとレコードとれないときに重ならないから、
サブクエリにして１行で取った方がいいかもしれないですけど。

[reiko_y]

サブクエリ、、、分かりませんでした。
SELECTで取得した結果を利用してSELECTするようなものは見つけましたが、、
難しいのでひとまず、ほかの部分を進めたいと思います><;
ORDER BY句もASCとDESC混在でもいけますか？
間違っていると思いますが、こんな感じでしょうか。

$sql = 'SELECT a.id
, (SELECT id FROM article where category="event"
AND event_date > "2021-05-01" ORDER BY event_date DESC LIMIT 1)as prev
, (SELECT id FROM article where category="event"
AND event_date > ""2021-05-01"" ORDER BY event_date ASC LIMIT 1)as next
, (SELECT * FROM article
where category="event" ORDER BY event_date DESC LIMIT 1)as top
from (select id from article group by id) a';

[m.ts10806]

PHPからいきなり実行ではなく、DBに対して直に実行して想定の結果が得られることを確認してください。
サブクエリ組むときも全部一気にではなく、小さなまとまりごとに作っていって後で組み合わせます。
order byは表に対して行うのでサブクエリやunionのように個別に表ができるのであれば可能です。
あくまで表と表の結合です。

[m.ts10806]

SQLiteだともしかしたらそこまで込み入ったことはできないかもしれないので、やはりMySQLやPostgreSQLなど機能が揃ったもののほうがアプリケーション側の負担を減らすことができると思います。

[reiko_y]

ありがとうございます。
SQLiteで対象テーブルは1つ　絞り込み条件のみが違う　結果＝id　の取得です。
ひとつひとつのクエリーでは正しい結果を取得されています。(phpLiteAdmin)

私の頭ではちょっと無理そうですのでひとまず終了させていただきます。
いろいろとありがとうございました。

Answer 3

取得結果あるときだけhtmlに出力すれば良いのでは。
$resultがどのように定義されるのか書かれてないのでわかりませんが、例えばこう

echo '<a href="event.php?do=top">最新記事</a>';
if(!is_null($result)){
    echo ' <a href="event.php?do=next&date='.$result["event_date"].'">次へ»</a>';
}
echo '</div>';

Comments

[reiko_y]

参考にさせていただきました。
ありがとうございます。