ひとこと掲示板でコメントなしのときの時間を表示させない

ひとこと掲示板で、コメントなしの状態で送信ボタンを押した後に、名前とコメントを
入力した後に送信ボタンを押すと前の時間が表示されしまいます。

名前とコメントに記入されていないときには、保存先に行かないようにすればいいと思うのですが、
方法が分かりません。

初心者なのを加味していただけながら、ご教示していただきたいです。

よろしくお願いします。

PHP
1<?php
2$name = '';
3$comment = '';
4$log = date('Y m/d H:i:s') . "\n";
5$file = './file.txt';
6$data = [];
7
8if($_SERVER['REQUEST_METHOD'] ==='POST') {
9    if(isset($_POST['name']) === true) {
10        $name = $_POST['name'];
11    if(isset($_POST['comment']) === true) {
12        $comment = $_POST['comment'];
13        if(($fp = fopen($file, 'a')) !== false) {
14          if (fwrite($fp, $name. $comment) === FALSE) {
15            print '書き込み失敗';
16        }  
17            
18        }
19
20            if(mb_strlen($comment) >= 100) {
21                print '１００文字以内にして下さい';
22            }else if (empty ($comment) === true){
23                print 'コメントを入力してください' . "\n";    
24            }else {
25                print '';
26            }
27            if(mb_strlen($name) >= 20) {
28                print '２０文字以内にして下さい';
29            }else if (empty($name) === true){
30                print 'お名前を入力してください' . "\n";    
31            }else {
32                print '';
33            }
34            fclose($fp);
35    }    
36    }
37}
38if (is_readable($file) === TRUE) {
39    if (($fp = fopen($file, 'r')) !== FALSE) {
40        while (($tmp = fgets($fp)) !== FALSE) {
41            $data[] = htmlspecialchars($tmp, ENT_QUOTES, 'UTF-8');
42        }
43        fclose($fp);
44    }
45}    
46?>
47<!DOCTYPE html>
48<html lang="ja">
49<head>
50   <meta charset="UTF-8">
51   <title></title>
52</head>
53<body>
54    
55        <form method="post" action="./bbs.php">
56            名前:<input type="text" name="name" value="">
57            コメント:<input  type="text" name="comment" value=""> 
58            <input type="submit" value="送信">
59        </form>        
60    <p><?php if($name === '' || $comment === ''){ 
61            print '';
62            } else {
63                foreach ($data as $read){ 
64            print $read . "<br>";
65            }
66        } ?>
67    </p>
68
69    
70</body>
71
72</html>

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

回答1件

ベストアンサー

名前とコメントに記入されていないときには、保存先に行かないように

文章化できているので、問題はないと思いますよ。
今の処理を追ってみるとおかしいところは確実にあります。
（インデント揃ってないとか気になるところはありますけど）

ゆっくりロジックを整理すると見えてくるはず。

fwrite()したあとに入力のチェックが行われています。
これではフォーム送信されたら未入力があっても書き込まれてしまいます。
先に入力のチェックをしましょう。

ちなみにisset()はあくまで変数の存在チェックであり、「変数が存在し、中身が空文字である場合」はtrueです。

php
1<?php
2var_dump(isset($a)); //false
3
4$b = "";
5var_dump(isset($b)); //true
6
7$c = null;
8var_dump(isset($c)); //false
9
10$d = [];
11var_dump(isset($d)); //true
12

エラーもその場で出力するのではなく、配列にため込んでおいて、その配列が空かどうかで書き込むかどうか決めては如何でしょう。

php
1<?php
2$error = [];
3if(mb_strlen($comment) >= 100) {
4    $error[] = '１００文字以内にして下さい';
5}else if (empty ($comment) === true){
6    $error[] = 'コメントを入力してください' . "\n";    
7}
8if(mb_strlen($name) >= 20) {
9    $error[] = '２０文字以内にして下さい';
10}else if (empty($name) === true){
11    $error[] = 'お名前を入力してください' . "\n";    
12}
13
14if(count($error) === 0){
15    // 書き込み処理
16}else{
17    // エラー出力
18}