php sql ページング機能 取得内容が重複してしまう

Question

現在、ECサイトの作成をしております。ユーザーごとに購入履歴を表示する履歴ページにてページング機能を実装したいのですがうまくいきません。
下記コードだと1ページあたり10件になりますが、各ページにて履歴内容が重複してしまいます。
（1ページ目（21-12件）、2ページ目（20-11件）のような表示になっている。
1ページ目（21-12件）、2ページ目（11-2件）にしたい）

関数部分

// 履歴テーブルから取得
function get_history($dbh, $user_id) {
    try {
        // SQL生成
        $sql = 'SELECT sake_master.sake_id, 
                    sake_master.name,
                    sake_master.price,
                    sake_master.img,
                    users_history.user_id,
                    users_history.amount,
                    users_history.buy_datetime
                    FROM sake_master
                    INNER JOIN users_history
                    ON sake_master.sake_id = users_history.sake_id
                    WHERE users_history.user_id = :user_id
                    ORDER BY buy_datetime DESC
                    LIMIT :start, :limit';
        // SQL文を実行する準備
        $stmt = $dbh->prepare($sql);
        $stmt->bindParam(':user_id', $user_id, PDO::PARAM_INT);
        $stmt->bindParam(':start', getCurrentPage(), PDO::PARAM_INT);
        $stmt->bindValue(':limit', 10, PDO::PARAM_INT);
        // SQLを実行
        $stmt->execute();
        // レコードの取得
        $rows = $stmt->fetchAll();
} catch (PDOException $e) {
        throw $e;
    }
    return $rows;
}

// 履歴テーブルの件数を取得
function get_historyCount($dbh, $user_id) {
    try {
        // SQL生成
        $sql = 'SELECT COUNT(*)
                FROM users_history
                WHERE user_id = :user_id';
        // SQL文を実行する準備
        $stmt = $dbh->prepare($sql);
        $stmt->bindParam(':user_id', $user_id, PDO::PARAM_INT);
        // SQLを実行
        $stmt->execute();
        // レコードの取得
        $count = $stmt->fetchColumn();
    } catch (PDOException $e) {
        throw $e;
    }
    return $count;
}

// 現在のページ番号を取得する
function getCurrentPage() {
    return (int) filter_input(INPUT_GET, 'page');
}

function pagination($count) {
    $limit = 10;

    //レコード総数がゼロのときは何も出力しない
    if (0 === $count) {
        return '';
    }
    
    //現在表示中のページ番号（ゼロスタート）
    $intCurrentPage = getCurrentPage();

    //ページの最大数
    $intMaxpage = ceil($count / $limit);

    //現在ページの前後３ページを出力
    $intStartpage = (2 < $intCurrentPage) ? $intCurrentPage - 3 : 0;
    var_dump($intCurrentPage);
    $intEndpage = (($intStartpage + 7) < $intMaxpage) ? $intStartpage + 7 : $intMaxpage;
    var_dump($count);

    //url組み立て
    $urlparams = filter_input_array(INPUT_GET);

    $items = array();
    
    //最初
    $urlparams['page'] = 0;
    $items[] = sprintf('<span><a href="?%s">%s</a></span>'
        , http_build_query($urlparams)
        , '最初'
    );

    //表示中のページが先頭ではない時
    if (0 < $intCurrentPage) {
        $urlparams['page'] = $intCurrentPage - 1;
        $items[] = sprintf('<span><a href="?%s">%s</a></span>'
            , http_build_query($urlparams)
            , '前へ'
        );
    }

    for ($i = $intStartpage; $i < $intEndpage; $i++) {
        $urlparams['page'] = $i;
        $items[] = sprintf('<span%s><a href="?%s">%s</a></span>'
            , ($intCurrentPage == $i) ? ' class="current"' : ''
            , http_build_query($urlparams)
            , $i + 1
        );
    }
    
    //表示中のページが最後ではない時
    if ($intCurrentPage < $intMaxpage) {
        $urlparams['page'] = $intCurrentPage + 1;
        $items[] = sprintf('<span><a href="?%s">%s</a></span>'
            , http_build_query($urlparams)
            , '次へ'
        );
    }

    //最後
    $urlparams['page'] = $intMaxpage - 1;
    $items[] = sprintf('<span><a href="?%s">%s</a></span>'
        , http_build_query($urlparams)
        , '最後'
    );

    return sprintf('<div class="pagination">%s</div>', implode(PHP_EOL, $items));
}

表示部分

<?php foreach($rows as $value) { ?>
            <?php if (count($rows) !== 0) { ?>
                <div class="layout">
                    <div class="i">
                        <img src="<?php print $img_dir . $value['img']; ?>" class="itemlist">
                    </div>
                    <div class="n">
                        <div class="nameprice">
                            <p class="list"><?php print $value['name']; ?></p>
                            <p class="list">価格¥<?php print $value['price']; ?></p>
                        </div>
                        <p class="list">数量<?php print $value['amount']; ?>個</p>
                        <p class="history">購入日<?php print $value['buy_datetime']; ?></p>
                    </div>
                </div>
            <?php } ?>
        <?php } ?>
        </div>
        <?php print pagination($count); ?>

https://qiita.com/ShibuyaKosuke/items/0c5c6df1fac218fbca38
こちらのサイトを参考にさせて頂きましたが、全体的にあまり理解できておりません。
ご教示頂ければ幸いです。

Answer 1

ざっくり直すだけなら、
$stmt->bindParam(':start', getCurrentPage(), PDO::PARAM_INT);
を

$start = getCurrentPage() * 10;
$stmt->bindParam(':start', $start, PDO::PARAM_INT);

みたいな感じすればOKなはず。

理由はLIMIT句の文法（参考）と今のコードのstartがどうなっているかを確認して見比べればわかると思います。

---

参考にしているサンプルの方では
$start = getCurrentPage() * $limit;
という感じでもう少し使いやすくなっているので、もう一度サンプルをじっくり読んで理解を試みることをお勧めします。

Comments

[h_r]

ご回答ありがとうございます。取り急ぎ、ご指摘頂いた内容に変更したところ上手くいきましたのでベストアンサーにさせて頂きます。理由がなぜかは、またゆっくり時間のあるときに確認します。本当にありがとうございました。