PHPにて掲示板を作成していますが、エラーの際に余計な文言が出力されてしまいます。

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

プログラミング初心者です。PHPで掲示板を作成しているのですが、
①nameとcommentを入力した際には、ブラウザにてどちらも出力（入力した時間も出力）→これはできています。
②nameかcommentのいずれかだけを入力した場合はエラー→こちらはできています。
③②でエラーが出た場合、投稿一覧には何も出力したくないのですが、「●: -2020-09-13 08:01:32」のように、出力されてしまいます。上記のような出力が出ないようにするためには、どのようにすれば良いでしょうか？ご教示お願い致しますm(__)m

該当のソースコード

<?php $filename = 'TokyoTyrantTable.csv'; $name=''; $comment=''; $name_max = 20; $comment_max = 100; $log = date('(-Y-m-d H:i:s)'); $fp = fopen($filename, 'a+b'); if ($_SERVER['REQUEST_METHOD'] === 'POST') { $name = htmlspecialchars($_POST['name']); $comment = htmlspecialchars($_POST['comment']); fputcsv($fp, [$name, $comment,$log]); rewind($fp); } while ($row = fgetcsv($fp)) { $rows[] = $row; } fclose($fp); ?> <!DOCTYPE html> <html lang="ja"> <head> <meta charset="UTF-8"> <link href="style.php" rel="stylesheet" type="text/css" media="all"> <title>掲示板</title> </head> <body> <h1>掲示板</h1> <section> <h2>新規投稿</h2> <?php if (mb_strlen($name) > $name_max){?> <p><?php print '名前は20文字以内で入力してください';?></p> <?php } ?> <?php if(mb_strlen($name) === 0){?> <p><?php print '名前を入力してください';?></p> <?php } ?> <?php if (mb_strlen($comment) > $comment_max){?> <p><?php print 'ひとことは100文字以内で入力してください';?></p> <?php } ?> <?php if(mb_strlen($comment) === 0){?> <p><?php print 'ひとことを入力してください';?></p> <?php } ?>

<form action="" method="post">
    <div class="name"><span class="label">お名前:</span><input type="text" name="name" value=""></div>
    <div class="honbun"><span class="label">本文:</span><textarea name="comment" cols="30" rows="3" maxlength="80" wrap="hard" placeholder="100字以内で入力してください。"></textarea></div>
    <input type="submit" value="投稿">
</form>

</section> <section class="toukou"> <h2>投稿一覧</h2> <?php if (!empty($rows)){?> <?php foreach ($rows as $row){ ?> <?php if((empty($name))&&(empty($comment))) { ?> <li><?php print $row[0].":"?> <?php print $row[1] ?> <?php print $row[2] ?></li> <?php } else{?> <?php break; } ?> <?php } ?> <?php } ?> </body> </html>

PHP

試したこと

試している箇所を抜粋いたします。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

行動規範の内容に同意します

回答3件

おかしなところがあります。

php
1$fp = fopen($filename, 'a+b');
2if ($_SERVER['REQUEST_METHOD'] === 'POST') {
3$name = htmlspecialchars($_POST['name']);
4$comment = htmlspecialchars($_POST['comment']);
5fputcsv($fp, [$name, $comment,$log]);
6rewind($fp);
7}

これ、POST受信したら$nameや$commentがどういう状態でも
fputcsv()にて出力しちゃいますが、それでいいのでしょうか？

fputcsv()には、
区切り文字の指定や、フィールドを囲む記号の指定や、特殊な文字をエスケープする処理の指定など、
便利な機能があるので、有効に使うべきです。

php
1<h2>投稿一覧</h2>
2<?php if (!empty($rows)){?>
3<?php foreach ($rows as $row){ ?>
4<?php if((empty($name))&&(empty($comment))) { ?>
5<li><?php print $row[0].":"?> <?php print $row[1] ?> <?php print $row[2] ?></li>
6<?php } else{?>
7<?php break; } ?>
8<?php } ?>
9<?php } ?>

$rowsから一つずつ$rowに引っ張り出して、
$rowと直接関係のない$nameと$commentで評価してから$rowを出力するけど、
この時点で$rowはたぶん一行丸々のカンマ区切り文字列のままになってて
$row[0]とか$row[1]とかでアクセスすると左端から０番目や１番目の文字を出力するような感じにならないでしょうか。

それと、直接の問題ではないですが、
一行ずつPHPの開始タグと終了タグを書くと、大変読みにくくミスの発見も遅れますし、動作も遅いようです。
アルゴリズムのおかしいところを目をつむり先の箇所を書き直すと、

php
1<h2>投稿一覧</h2>
2<?php 
3  if (!empty($rows)) {
4    foreach ($rows as $row) { 
5      if ((empty($name)) && (empty($comment))) { 
6?>
7<li><?php print $row[0].":"; ?> <?php print $row[1]; ?> <?php print $row[2]; ?></li>
8<?php 
9      }
10      else {
11        break; 
12      }
13    }
14  }
15?>

などとして、インデントレベルを適切にしてアルゴリズムがおかしいところがないかを追いやすくしたほうが良いかと。

あと、この先遭遇するであろう問題点も書き出しておきます。

●名前やコメントに「,」を含めてしまうと、CSVとしての文字区切りと混同してデータ処理がおかしくなる。（htmlspecialchars()は「,」の処理をしない。）
●コメントの入力でmaxlength=80としたら81文字以上99文字以下を受け付けられないのでは？重要な数字は定数を駆使して一箇所直すだけで全部反映されるようにする。
●データファイルにhtmlspecialchars()で加工したあとの文字列を記録すると、検索機能の実装に支障が出るので、ファイルへの記録は加工前の素の文字列で、htmlへの出力直前に加工して出力するのを徹底する。
●追記のみのようだけど、コメントの編集や削除機能がちゃんとできるのかどうか。

フォームで入力する名前やコメントに「,」が含まれるとまず破綻するので、
この手の機能を作るときはデータの区切り文字列として例えば「<>」を使ったりするけど、
フォームで入力する名前やコメントに「<>」が含まれたら結局同じなので、
フォーム入力できない文字（たとえばTABコード）にするか、
もしくはフォーム入力された文字列に「<>」が含まれていたら可逆性のある文字列に置き換える方法もあります。
以上蛇足。

投稿2020/09/14 02:04

編集2020/09/14 02:11