csvデータを下から数えて10行目から最後までを表

質問

<?php if (have_posts()): ?>
<?php
if(isset($_GET['s']) && empty($_GET['s'])) {
echo '検索キーワード未入力';
} else {
echo '“'.$_GET['s'] .'”の検索結果：'.$wp_query->found_posts .'件'; // 検索キーワードと該当件数を表示
// chmodで書き込み権限を付与
$filename = chmod("test.csv", 0775);

// PHPでcsvファイルを読み込みする
$f_path = "test.csv";
var_dump($f_path);

// 送信内容をShift_JISに変更
mb_convert_variables('Shift_JIS', 'UTF-8', $ShiftJIS);

// 書き方はおそらくあってる
$keyword_data = array(date('Y-m-d-G-i-s'), $_SERVER["REMOTE_ADDR"],$_GET['s'].PHP_EOL);

// ファイルを開くか、新しいテキストファイルを作って最後に追加書き込み
$fp = fopen('test.csv', 'a');

// ファイルに対する処理を行う前にファイルをロックして、作業中は他の人がそのファイルに作業を行えないようにする、排他的ロック
flock($fp, LOCK_EX);

// CSVファイルにフォームの送信内容を記入しています
fputcsv($fp, $keyword_data);

// ロック解除
flock($fp, LOCK_UN);

// CSVファイルをクローズして終わり
fclose($fp);
}
?>
<ul>
<?php while(have_posts()): the_post(); ?>
<li>
<a href="<?php the_permalink(); ?>"><?php echo get_the_title(); ?></a>
</li>
<?php endwhile; ?>
</ul>
<?php else: ?>
検索されたキーワードにマッチする記事はありませんでした
<?php endif; ?>

<?php
setlocale(LC_ALL, 'ja_JP.UTF-8');
$fp = fopen('test.csv', 'r');
flock($fp,LOCK_SH);
$rows = array();
while ($row = fgetcsv($fp)) array_push($rows, $row);
flock($fp,LOCK_UN);
fclose($fp);
foreach (array_slice($rows, -10) as $row) vprintf("%s %s\n", $row);
var_dump($row);
?>

var_dump($row);で確認したところ
2021-09-07-4-27-36 126.156.166.227 2021-09-07-4-27-52 126.156.166.227 2021-09-07-4-31-03 126.156.166.227 2021-09-07-4-50-36 126.156.166.227 2021-09-07-4-52-03 126.156.166.227 2021-09-07-4-52-15 126.156.166.227 2021-09-07-4-52-26 126.156.166.227 2021-09-07-5-01-27 126.156.166.227 2021-09-07-5-14-25 126.156.166.227 2021-09-08-4-48-06 126.133.202.39 array(3) { [0]=> string(18) "2021-09-08-4-48-06" [1]=> string(14) "126.133.202.39" [2]=> string(40) "ううううううううううううう " }
と出てます

2021-09-03-13-28-52,126.255.28.185,"ううう
"
2021-09-03-13-29-46,126.255.28.185,"ううう
"
2021-09-03-13-29-49,126.255.28.185,"ううう
"
2021-09-06-14-43-33,126.204.198.239,"ううううううううううううう
"
2021-09-06-14-45-37,126.204.198.239,"ううううううううううううう
"
2021-09-07-3-09-52,126.156.166.227,"ううううううううううううう
"
2021-09-07-3-10-34,126.156.166.227,"ううううううううううううう
"
2021-09-07-3-15-56,126.156.166.227,"ううううううううううううう
"
2021-09-07-3-17-27,126.156.166.227,"ううううううううううううう
"
2021-09-07-3-24-07,126.156.166.227,"ううううううううううううう
"
2021-09-07-3-24-29,126.156.166.227,"ううううううううううううう
"
2021-09-07-3-25-04,126.156.166.227,"ううううううううううううう
"
2021-09-07-3-25-19,126.156.166.227,"ううううううううううううう
"
2021-09-07-3-25-33,126.156.166.227,"ううう
"
2021-09-07-3-26-02,126.156.166.227,"ううう
"
2021-09-07-3-26-40,126.156.166.227,"ううう
"
2021-09-07-3-27-08,126.156.166.227,"ううう
"
2021-09-07-3-27-27,126.156.166.227,"ううう
"
csvにはこのように表示されていて、下の10件最新のものを表示したいのですが、現状はIPと日時のみ表示されます。

回答No.1

asciiz · ベストアンサー

最後の表示部分のこちら、

＞ while ($row = fgetcsv($fp)) array_push($rows, $row);

fgetcsv() は、1行のcsvデータをカンマで分割した、配列を返します。
それを$row に代入しているのがちょっと妙なところ。

「下から10行を表示する」ということを考えたとき、カンマで分割してもらう必要はありません。
fgetcsv() → fgets() にして1行単位で読んでみてはどうですか?

補足

<?php if (have_posts()): ?>
<?php
if(!isset($_GET['s']) || empty($_GET['s'])) {
echo '検索キーワード未入力';
} else {
echo '“'.$_GET['s'] .'”の検索結果：'.$wp_query->found_posts .'件'; // 検索キーワードと該当件数を表示
// chmodで書き込み権限を付与
$filename = chmod("test.csv", 0775);

// PHPでcsvファイルを読み込みする
$f_path = "test.csv";
var_dump($f_path);

// 送信内容をShift_JISに変更
mb_convert_variables('Shift_JIS', 'UTF-8', $ShiftJIS);

// 書き方はおそらくあってる
$keyword_data = array(date('Y-m-d-G-i-s'), $_SERVER["REMOTE_ADDR"],$_GET['s'].PHP_EOL);

// ファイルを開くか、新しいテキストファイルを作って最後に追加書き込み
$fp = fopen('test.csv', 'a');

// ファイルに対する処理を行う前にファイルをロックして、作業中は他の人がそのファイルに作業を行えないようにする、排他的ロック
flock($fp, LOCK_EX);

// CSVファイルにフォームの送信内容を記入しています
fputcsv($fp, $keyword_data);

// ロック解除
flock($fp, LOCK_UN);

// CSVファイルをクローズして終わり
fclose($fp);
}
?>
<ul>
<?php while(have_posts()): the_post(); ?>
<li>
<a href="<?php the_permalink(); ?>"><?php echo get_the_title(); ?></a>
</li>
<?php endwhile; ?>
</ul>
<?php else: ?>
検索されたキーワードにマッチする記事はありませんでした
<?php endif; ?>

<?php
setlocale(LC_ALL, 'ja_JP.UTF-8');
$fp = fopen('test.csv', 'r');
flock($fp,LOCK_SH);
$rows = array();
while ($row = fgetcsv($fp)) array_push($rows, $row);
flock($fp,LOCK_UN);
fclose($fp);
foreach(array_slice($rows, -10) as $row)
vprintf("%s %s\n", array($row[0], $row[2]));
?>

こちらのコードで出力されました

回答No.2

AsarKingChang

あんまり深く考えず。。
https://www.php.net/manual/ja/function.file.php
最初から配列に突っ込む関数があります。
なので、
$data= file (ファイル名);
で読み込み、何行あるかを調べる。
$count=count($data);
if ($count<10) { // 元々１０行以下なら全部とする
// 全行を表示
}else{
// 上のチェックで、１０行以上ある事が保証される為。
for ($index=$count-10;$index<$count;$index++) {
後ろから１０行前から、最後までをダンプ
　　$data[$index]が「行」ですので、
　　カンマ区切りでも、スペース区切りでも・・・
$csv = explode(',', trim($data[$index]) );
テストなら、print_r($csv)などで、ダンプを！
　}
}

なので、ロックとかしなくても、１命令で
完結させるほうが、いいかと。

ただし！！

それだと、CSVが１０万行あったらどうするの？ってな
問題がでるでしょうが。。
今回は、短いものだと仮定しての回答です。
また、エスケープも考慮なしです。

補足

>$keyword_data = array(date('Y-m-d-G-i-s'), $_SERVER["REMOTE_ADDR"],$_GET['s'].PHP_EOL);

日付、IPアドレス、投稿内容の3つのカラムで格納したのを無視して、
$rows = array_slice($rows, -10);
> vprintf("%s %s\n", $row);
としていたので、格納したデータと表示したいデータが結びついていないのが原因でした。
また3カラムで格納したのに、2カラムで表示しようとして空データが返ってきていたようです。