[PHP]掲示板でのコメント機能の不具合

コード
```### 前提・実現したいこと
掲示板の作成。

PHPで掲示板の作成をしています。
いろんな人がコメントを書き込めるようにしていて、一番目の人は正常に書き込めるのですが二番目の人が書き込むと一番目の人のコメントに上書きされてしまいます。

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

エラーメッセージ


### 該当のソースコード

PHP

```ここに言語を入力
<?php
$data_file = 'keijiban.txt';
$ext = file_exists($data_file);
$lines = $ext ? file($data_file) : array();
$errMsg = "";
date_default_timezone_set('Asia/Tokyo');

if(isset($_POST['submit'])){
        if(empty($_POST['name'])){
                $errMsg='名前を入力してください<br>';
        }
        elseif(empty($_POST['free'])){
                $errMsg .='記事を入力してください';
        }

        if(!$errMsg){
                function convert_str($str){
                        $str=htmlspecialchars($str);
                        $str=preg_replace("/\r\n/","<br>",$str);
                        $str=preg_replace("/\r|\n/","<br>",$str);
                        return $str;
                }

                $ln = explode(",", $lines[0]);
                $no = $ext ? sprintf("%03d", $ln[0]+1) : "001";
                $name = convert_str($_POST['name']);
                $free = convert_str($_POST['free']);
                $delkey = !empty($_POST['delkey']) ? convert_str($_POST['delkey']) : '#####';
                $time = date("Y/m/d H:i:s");

                $data = "$no,$name,$free,$delkey,$time\n";
                array_unshift($lines,$data);
        }
}
        if(isset($_POST['delbtn']) && $ext){
                for($i = 0; $i < count($lines); $i++){
                        $ln = explode(",", $lines[$i]);
                        if($ln[0] == $_POST['no'] && $ln[3] == $_POST['Rdkey']){
                                array_splice($lines,$i,1);
                                break;
                        }
                }
        }

        if(isset($_POST['submit']) || isset($_POST['delbtn'])){
                $fk = fopen($data_file, 'w');
                foreach($lines as $line)
                        fputs($fk, $line);
                fclose($fk);
        }
?>

<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="UTF-8">
<title>絵本の掲示板</title>
<link rel="stylesheet" href="keijiban.css">
</head>
<body>
<div id="wrapper">
<form method="post" action="keijiban.php">
<div class="center">
    <div id="title">絵本の掲示板</div>
    <p>
    No:<input type="text" name="no" size="5">
    削除キー:<input type="text" name="Rdkey" size="20">
    <input type="submit" name="delbtn" value="削除">
    </p>
</div>

<?php
        if($errMsg)
                echo"<div id='errMsg'>".$errMsg."</div>";
?>

<div id ="edit_area">
    <p>
    名前:<input type="text" name="name" size="26">
    削除キー:<input type="text" name="delkey" size="20">
    </p>
    <p>
    記事:<br><textarea name="free"></textarea>
    </p>
    <p class="center">
    <input type="submit" name="submit" value="書き込む">
    <input type="reset" value="取り消す">
    </p>
</div>

<?php
        foreach($lines as $line){
                $ln = explode(",", $line);
                echo"<div><p class='entry_ID'>[No.".$ln[0]."] 名前:".$ln[1]."&nbsp;";
                echo"書込み日付:".$ln[4]."</p>";
                echo"<p>".$ln[2]."</p>";
                echo"</div><hr>";
        }
?>

</form>
</div>
</body>
</html>


### 試したこと

上書きされてしまう、ということで配列の格納場所の移動ができていないのかと考え、確認してみましたが、私には間違いが発見できませんでした。

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

このプログラムはテラタームプロで作成しました。
ご回答よろしくお願いします！

退会済みユーザー

2020/01/20 08:00

function convert_str の定義箇所、それでいいのか？

m.ts10806

2020/01/20 08:01

コードはマークダウンのcode機能を利用してご提示ください

m.ts10806

2020/01/20 08:03

!$errMsg のときにしか定義されない関数… この手のものは「常に使える状態にしておく」のが通例かと思います。限定すればするほど考慮しなければならない点が多くなり、バグに繋がります。

kirinkun_

2020/01/20 08:07

マークダウンのコード機能使って修正してみましたが変わってなさそうですね… すみません！やり直してみます！

m.ts10806

2020/01/20 08:09

コード部分選択して「<code>」押すのが早いです。投稿前にプレビュー確認してください

退会済みユーザー

2020/01/20 08:11

コードの箇所の前と後に、「```」だけの行を置くだけの簡単なお仕事です。https://teratail.com/help#about-markdown

kirinkun_

2020/01/20 08:27

お待たせして申し訳ございません！修正させていただきました！

退会済みユーザー

2020/01/20 08:29

もしかして、「PHPの絵本第2版 Webアプリ作りが楽しくなる新しい9つの扉」とか読んでたりします？

kirinkun_

2020/01/20 08:30

はい！その本です！もしかしてダメでしたか...?

退会済みユーザー

2020/01/20 09:16

第2版であっても2017年4月のため、情報が古いです。例えば有名なCMSソフトであるWordPress最新版はPHP7.3以降が必須になっていたりなど、PHP7系に対応していない本の情報を鵜呑みにしてはいけないと考えます。PHPの関数を使う前に、PHPマニュアル https://www.php.net/manual/ja/ で引数や戻り値など念のため点検してください。あと、htmlspecialchars() についてはこちらもご参照ください： https://qiita.com/mpyw/items/19e6fed835ccdbcb0d6d

kirinkun_

2020/01/20 09:21

そうなんですね！わかりやすいご回答にリンクまでありがとうございます！！参考にさせていただきます！

m.ts10806

2020/01/20 09:22

本当にやろうと思ったらロジック最初から組み直した方がスッキリすると思います。 (あと、テキストファイルでデータ管理するやり方は実務で役に立ちません)

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

回答2件

試しに、ファイル名だけ変更してそのまま動かしてみたけど、
全くデータがない状況から、一人目の投稿をしたときに
$ln = explode(",", $lines[0]);でエラー「Notice: Undefined offset: 0 」になったりしたくらいで、
1行しかデータがない場合に$lines[0]じゃアクセスできないからっていう理由で、
少々微妙な気持ちになったり。

せめて、

php
1if (is_array($lines)) {
2}
3else {
4}

とかやってもいいかもしれない。

ちなみに、PHPのコーディングをやるときはたとえWindowsであっても、
UTF-8(BOMなし)エンコーディング、改行コードはLFで保存してる。
エンコーディングや改行コードの種類がわからないエディタを使っているようなら、
プログラミング向けのエディタを物色してほしい。
VisualStudioCodeでもいいだろうし。

まずは、keijiban.txtをまるっと削除して、
最初から再現テストをしてみてほしい。
（あるいは、削除する前にkeijiban.txtを開いて、余計な「,」を含んでいないかもチェック。）

あと、入力フォームに「,」が含まれると処理が破綻するので、
「,」を含んでいたらデータファイルが壊れてしまわないよう、
検証処理を加えるべき。
（よくネットで見られる簡易掲示板の、データ区切り文字は「,」を使わず「<>」だったりする。）

投稿2020/01/20 09:47

編集2020/01/20 10:35

退会済みユーザー

総合スコア0

テスト環境で検証した結果疑わしい情報があります。

keijiban.textファイルの作成、読み込み、または書き込みが許可されているでしょうか。keijiban.textの中身を確認してください。真っ白のままならばパーミッションエラーで書き込みができない設定のままになっているはずです。その場合はひたすら入力された文字をそのまま配列に受けて流す繰り返しになりますので、何度やっても上書きを繰り返します。もし、存在しない場合は、権限がないためにkeijiban.textのファイル作成を行えない可能性が極めて高いです。