Как правильно сделать пагинацию?

Question

[Анастасия]

У меня есть база данных где находятся проекты, и есть на странице навигация по этим проектам с помощью двух кнопок "Вперед" и "Назад"

<?php
$nextpost = next_post($post_id);
$prevpost = prev_post($post_id);
?>

<div>
<a href="/project?post_id=<?= $prevpost['id'] ?>"><button>Prev</button></a>
<a href="/project?post_id=<?= $nextpost['id'] ?>"><button">Next</button></a>
</div>

function next_post($post_id)
{
    global $link;
    $sql = "SELECT * FROM projects WHERE id > " . $post_id . " ORDER BY id ASC LIMIT 1";
    $result = mysqli_query($link, $sql);
    $nextpost = mysqli_fetch_assoc($result);
    return $nextpost;
}

function prev_post($post_id)
{
    global $link;
    $sql = "SELECT * FROM projects WHERE id < " . $post_id . " ORDER BY id DESC LIMIT 1";
    $result = mysqli_query($link, $sql);
    $prevpost = mysqli_fetch_assoc($result);
    return $prevpost;
}

Проблема в том что, если я, например, нахожусь на проекте с id=5 (а id=6 еще нет) и при этом я нажимаю на кнопку "Вперед", то меня переносит на страницу с id=6, которой не существует и я просто получаю ошибку.

Как сделать так, чтобы при настику на кнопку "Вперед", при условии что у меня нет следующего проекта, меня перекидувало на страницу с id самого первого проекта в базе данных. (Так же и с кнопкой "Назад")

Comments

[Дмитрий]

По хорошему надо запрос на существование или на количество добавить

Answer 1

[Дмитрий Энтелис]

Из приведенного кода не понятно как происходит

если я, например, нахожусь на проекте с id=5 (а id=6 еще нет) и при этом я нажимаю на кнопку "Вперед", то меня переносит на страницу с id=6, которой не существует и я просто получаю ошибку.

Общая идея пагинации: смотрите какие у вас есть данные в базе, исходя из этого формируете ссылки на возможные страницы.
Если вы хотите сделать цикличную - ну так и сделайте её, добавьте в свою логику проверку "id больше чем текущий не найдено" и корректно обрабатывайте.

Answer 2

[Ильдар Сарибжанов]

Да простят меня боги чистого кода.

/**
 * Следующий пост
 *
 * @param $post_id
 *
 * @return string[]|null
 */
function next_post($post_id)
{
    global $link;
    $sql      = "SELECT * FROM projects WHERE id > " . $post_id . " ORDER BY id ASC LIMIT 1";
    $result   = mysqli_query($link, $sql);
    $nextpost = mysqli_fetch_assoc($result);

    if (!$nextpost) {
        return first_post();
    }

    return $nextpost;
}

/**
 * Предыдущий пост
 *
 * @param $post_id
 *
 * @return string[]|null
 */
function prev_post($post_id)
{
    global $link;
    $sql      = "SELECT * FROM projects WHERE id < " . $post_id . " ORDER BY id DESC LIMIT 1";
    $result   = mysqli_query($link, $sql);
    $prevpost = mysqli_fetch_assoc($result);

    if (!$prevpost) {
        return last_post();
    }

    return $prevpost;
}

/**
 * Первый пост
 *
 * @return string[]|null
 */
function first_post()
{
    global $link;
    $sql    = "SELECT * FROM projects WHERE ORDER BY id ASC LIMIT 1";
    $result = mysqli_query($link, $sql);
    $post   = mysqli_fetch_assoc($result);

    return $post;
}

/**
 * Последний пост
 *
 * @return string[]|null
 */
function last_post()
{
    global $link;
    $sql    = "SELECT * FROM projects WHERE ORDER BY id DESC LIMIT 1";
    $result = mysqli_query($link, $sql);
    $post   = mysqli_fetch_assoc($result);

    return $post;
}

Comments

[Анастасия]

Warning: mysqli_fetch_assoc() expects parameter 1 to be mysqli_result, boolean given in C:\OSPanel\domains\project\include\functions.php on line 182

182: $post = mysqli_fetch_assoc($result);

[Ильдар Сарибжанов]

Анастасия, значит вот так =)

/**
 * Следующий пост
 *
 * @param $post_id
 *
 * @return string[]|null
 */
function next_post($post_id)
{
    global $link;
    $sql      = "SELECT * FROM projects WHERE id > " . $post_id . " ORDER BY id ASC LIMIT 1";
    $result   = mysqli_query($link, $sql);

    if (!$result) {
        return first_post();
    }

    $nextpost = mysqli_fetch_assoc($result);

    return $nextpost;
}

/**
 * Предыдущий пост
 *
 * @param $post_id
 *
 * @return string[]|null
 */
function prev_post($post_id)
{
    global $link;
    $sql      = "SELECT * FROM projects WHERE id < " . $post_id . " ORDER BY id DESC LIMIT 1";
    $result   = mysqli_query($link, $sql);

    if (!$result) {
        return last_post();
    }

    $prevpost = mysqli_fetch_assoc($result);

    return $prevpost;
}

/**
 * Первый пост
 *
 * @return string[]|null
 */
function first_post()
{
    global $link;
    $sql    = "SELECT * FROM projects WHERE ORDER BY id ASC LIMIT 1";
    $result = mysqli_query($link, $sql);
    $post   = mysqli_fetch_assoc($result);

    return $post;
}

/**
 * Последний пост
 *
 * @return string[]|null
 */
function last_post()
{
    global $link;
    $sql    = "SELECT * FROM projects WHERE ORDER BY id DESC LIMIT 1";
    $result = mysqli_query($link, $sql);
    $post   = mysqli_fetch_assoc($result);

    return $post;
}

[Анастасия]

Ильдар Сарибжанов, Теперь ошибки нет, но все равно не работает. То есть не происходит переход до первой записи, или последней : (

[Анастасия]

Ильдар Сарибжанов, А возможно ли сделать так, что если нет следующего id, чтобы конкретная кнопка навигации становилась не активной?

[Ильдар Сарибжанов]

Анастасия, можно. В функция next/prev, там где добавил условие на проверку существования следующего/предыдущего поста вместо вызова функций last_post()/first_post(); можно сделать возврат null. А снаружи, там где выводится сама разметка кнопок вперёд/назад, добавить проверку на нулл.

ну типа так

/**
 * Следующий пост
 *
 * @param $post_id
 *
 * @return string[]|null
 */
function next_post($post_id)
{
    global $link;
    $sql    = "SELECT * FROM projects WHERE id > " . $post_id . " ORDER BY id ASC LIMIT 1";
    $result = mysqli_query($link, $sql);

    if (!$result) {
        return null;
    }

    $nextpost = mysqli_fetch_assoc($result);

    return $nextpost;
}

/**
 * Предыдущий пост
 *
 * @param $post_id
 *
 * @return string[]|null
 */
function prev_post($post_id)
{
    global $link;
    $sql    = "SELECT * FROM projects WHERE id < " . $post_id . " ORDER BY id DESC LIMIT 1";
    $result = mysqli_query($link, $sql);

    if (!$result) {
        return null;
    }

    $prevpost = mysqli_fetch_assoc($result);

    return $prevpost;
}

И разметка

<div class="paginator">
    <?php if (!is_null(prev_post())): ?>
        <a href="#" class="prev"></a>
    <?php endif; ?>

    <?php if (!is_null(next_post())): ?>
        <a href="#" class="next"></a>
    <?php endif; ?>
</div>