DBを使用しないで条件にマッチしたもののみページング

Question

以前ＤＢを使用しないページングについてお教えいただいた者です。少し改良して降順ソートを出来るようになったのですが、リスト表示の際にある条件に一致したもののみを表示する場合のやり方が分かりません。。。現在は以下のようになっています。 ■sample.csv no0001,あああああ,20050101,ON no0002,いいいいい,20050102,OFF 略） no0020,ととととと,20050120,OFF ■index.php （800字オーバーしてしまうので、省略しています。） //ファイルを配列に格納 $rec = file("sample.csv"); //レコード数を取得して、最後尾の行番号を取得 $rec_number = sizeof($rec) -1; //ページ範囲を出力 for ( $i = $rec_number-$page*$max; $i > $rec_number-$page*$max-$max; $i -- ) { $data = explode(",","$rec[$i]"); if ($data[3]==("ON") && $data[2]<=date("Ymd")) { print $data[0].("
"); print $data[1].("
"); print $data[2].("

"); } } print "次のページへ "; ?> CSVの各行の[3]が「ON」となっていれば表示し、「OFF」となっていれば表示しないようにし、また、[2]が今日以降の日付であれば表示しないようにしたいのですが、 for文の中に上記のようなif文を使ってしまうとオカシナことになってしまいます。（no0020が抜けただけで1ページ目に9件しか表示されません。本当は no0020が抜けたので、no0010までの10件を表示させたいのです。）根本的な部分が間違っているような気がするのですが、どのような考え方で作れば良いのか分かりません。また皆様のお力をお貸しください。宜しくお願いします。

anmochi · Accepted Answer

> 次のページへ渡していないからだと思うのですが、

> これはPOSTなりGETで参照渡しをするのでしょうか？
　今ページ番号って渡してる？　おそらくそれが最終的に$pageになるのだと思うのだが。
　今、「有効な行を数えて」10行分きちんと出す事ができた。ここでちょっと考えて欲しい。2ページ目に表示するのは「CSVの11行目から」だろうか？
　否、「CSVを先頭から全部舐めていって、有効な行を10行分（←1ページ目の分）捨てた後に、最初に出てきた有効な行」だ。これが、前回の回答で私が「頭出し」と言った部分だ。
　という訳で、次の関数を作りましょう。クラスにしても良いだろう。

class MyRecords {
var recnum; // int
var recmax; // int
var records; // array
int initCsvData(array recs); // file関数で読み込んだ行データを引数に、現在行などを初期化。
array getNextLine(); // 次の有効な行を返す。
}

このような下請けルーチンを作ることで、メインルーチンはこんなに簡単になる。

$recs = new MyRecords();
$recs->initCsvData($rec); // CSVデータをセット
for($i = 0; $i < $page * $max; $i++) { // 「ページ数×ページあたりの行数」分だけ
$recs->getNextLine(); // 捨てる。
}
for($i = 0; $i < $max; $i++) { // 「ページあたりの行数」分だけ
if(!$arr = $recs->getNextLine()) break; // もうデータが無ければループ終了(最後のページとなる）
// ～～行をHTMLに吐き出す～～
}

とても見通しが良くなったっしょ。こげな感じでどうでっしゃろ。

riaf · Answer

continueしてやるのはどうでしょうか？ for ( $i = $rec_number-$page*$max; $i > $rec_number-$page*$max-$max; $i -- ) { $data = explode(",","$rec[$i]"); if ($data[3]==("ON") && $data[2]<=date("Ymd")) { print $data[0].("
"); print $data[1].("
"); print $data[2].("

"); }else{ continue; } }

anmochi · Answer

for ( $i = $rec_number-$page*$max; $i > $rec_number-$page*$max-$max; $i -- ) { ここが問題だね。中をよく見てみると、「出力しようがしまいが$max回ループする」事になる。という訳で、「$max行出力するまでループ」という方法に変えてあげましょう。そうだなぁ。あまりごちゃごちゃ変えるのもあれなので、こんなのはどうだろう。 $outputline = 0; $i = $rec_number-$page*$max; // ここもおかしいよね。$rec[$i]を先頭から1行ずつ舐めていって、条件に合致する行数を数えていって頭だしをしないといけない。 while($outputline < max || $i <= $rec_number) { $data = explode(",", $rec[$i++]); if ($data[3]==("ON") && $data[2]<=date("Ymd")) { print $data[0].("
"); print $data[1].("
"); print $data[2].("

"); outputline++; // 行を出力したので1行増やす。これが$maxまで到達すると1ページ分。 } }

DBを使用しないで条件にマッチしたもののみページング

> 次のページへ渡していないからだと思うのですが、

continueしてやるのはどうでしょうか？

for ( $i = $rec_number-$page*$max; $i > $rec_number-$page*$max-$max; $i -- ) {

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング

for ( $i = $rec_number-$page$max; $i > $rec_number-$page$max-$max; $i -- ) {