ページング機能について

質問

ページング機能について

以前こちらで質問させて頂き、素晴らしい回答者様がオブジェクト指向でサンプルを作っていただき涙がでる思いでした。
その後オブジェクト指向についてサイトを色々見てクラスなどで管理して呼び出すという素晴らしい方法と出会いました。

しかし私はドットインストールで学んでいる初心者なため、また壁に当たってしまいました。
http://dotinstall.com/lessons/paging_php_v2/8510

前回の質問で頂いたものでオブジェクト指向も一緒に勉強してやると思ったのですが、知識が詰め込み＆始めたばかりで完全に詰んでしまいました。

オブジェクト指向はSQLで呼び出すものをクラスに作り実体化させ表示するとは思うのですが、書き方が悪いのかどうもうまくいかず、せっかく頂いたサンプルソースもグチャグチャになり再ダウンロード状態です。

ドットインストールの上記のページング機能と件数表示、これに関しましてオブジェクト指向の場合はどのようにするか教えて頂ける方がいないかと質問させて頂きました。

自分で調べろと言われそうなご質問ですが、色々なサイトで自分なりにやってみたのですが、完全にパンクしてしまったので申し訳ございませんがよろしくお願いします。

回答No.1

noname#244856 · ベストアンサー

何か所か修正を行ったので、最新版をDLしてください。catchブロックの中が以前より簡単になっていると思います。
https://github.com/Certainist/sns_php

動画見ましたが、これページング処理としては典型的にダメな方法なんですよね。SQL_CALC_FOUND_ROWSオプションと、FOUND_ROWS関数を利用するのが正解です。
http://blog.goo.ne.jp/shinx1265/e/7e173ab99d4fe541c75a9ea4bfbbdf5e

こんな感じでクラスにメソッドを実装してみてはどうでしょうか。
http://pastebin.com/89pjMn62

（OKWave内では全角スペースインデントしているので注意）

public function getComments($page) {

　　// 必ず1以上の整数になるようにする
　　$page = max(1, (int)$page);

　　// 開始オフセット
　　$offset = COMMENTS_PER_PAGE * ($page - 1);

　　// SQL実行
　　$stmt = $this->pdo->query(sprintf(
　　　　implode(' ', array(
　　　　　　'SELECT CALC_FOUND_ROWS *',
　　　　　　'FROM `comments`',
　　　　　　'LIMIT %d, %d',
　　　　)),
　　　　$offset,
　　　　COMMENTS_PER_PAGE
　　));

　　// コメントを取得
　　$comments = $stmt->fetchAll();

　　// 現在件数を取得
　　// (PDOのコンストラクタでMYSQL_ATTR_USE_BUFFERED_QUERYを
　　// 有効化しているときだけSELECTに対してrowCountメソッドが使える)
　　$current_count = $stmt->rowCount();

　　// 総件数を取得
　　$whole_count = (int)$this->pdo->query('SELECT FOUND_ROWS()')->fetchColumn();

　　// メッセージ作成
　　$message = sprintf('全%d件中、%d件から%d件まで表示しています',
　　　　$whole_count,
　　　　$offset + 1,
　　　　$offset + $current_count
　　);

　　// 情報を連想配列で返す
　　return array(
　　　　'comments' => $comments,
　　　　'message' => $message,
　　);

}

ちなみに私がスクリプト内で使用しているfilter_struct_utf8関数を使ってページパラメータをGETで受け取るならば

extract(filter_struct_utf8(INPUT_GET, array(
　　'page' => '',
)));

でいいと思います。どうせメソッド側でどんな値が来ても無理やり1以上の整数に変換するのでこの程度で構いません。むしろこの関数に頼らなくても、「配列を整数にキャスト」するのは「配列を文字列にキャスト」するのと違ってNOTICEエラーが発生しないので、このような場合においては最低限未定義かどうかのチェックを行うように

$page = isset($_GET['page']) ? $_GET['page'] : '';

としても十分です。

お礼

まさかまたご回答頂けるとは光栄です。しかも最新版にしていただいて申し訳ないです＞＜

初心者で学ぶのにドットインストールをおすすめされたので参考にしていたのですがページングのところは別の方法がいいなど、一人では中々気づけないので非常に嬉しいアドバイスです。

回答も説明込みでしていただき、早く一人で解決できるようになり、回答者側へなりたいと思います。

しばらくは助けてくださいばかりになりますが、お時間がありましたらお付き合いいただければ光栄です。

最新版をダウンロードさせて頂き勉強させて頂きます。

ありがとうございましたm(_ _)m