質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.59%

PHP簡易掲示板にて最新投稿がechoで表示されない。

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,708

score 13

前提・実現したいこと

HTML,PHPを使用し簡易掲示板を作成しています。
bbs.phpで入力フォームに書いたものが、bbs.txtに保存され、さらに入力フォームの下に投稿したものが順番に表示される(古いものが上で新しいものは下にくる)という掲示板です。

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

bbs.txt内では最新の投稿が表示されているのですが、なぜかブラウザ上(bbs.php上?)では最新の投稿が表示されません。
参考画像: https://gyazo.com/d62d40eb705315823654aaceef77aa23

つまり、最初の「投稿1」を送信した時点では、フォームの下には何も表示されていない状態になります。

該当のソースコード

<!DOCTYPE html>
<html>
<head>
    <meta charset="utf-8">
</head>

<body>
    <form action="bbs.php" method="post">
    User Name: <input type="text" name="userName"><br>
    Comment: <input type="text" name="comment"><br>
    <input type="submit">
    </form>

</body>
</html>
    <?php
    $fileName = "bbs.txt";
    $file = file($fileName, FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES);

    foreach ( $file as $value ) {  
        $line = explode("<>", $value);  
        echo $value."<br />\n";
    } 

    $num = sizeof($file) + 1;
    $userName = $_POST["userName"];
    $comment = $_POST["comment"];
    $time = date("Y/m/d");
    $write = $num. "<>". $userName. "<>". $comment. "<>". $time. "\n";

    $fp = fopen($fileName, "a");
    flock ($fp, LOCK_EX);
    fwrite($fp, $write);
    flock ($fp, LOCK_UN);
    fclose($fp);
    ?>

試したこと

理由が全くわからず、調べても該当するものを見つけることができなかったため、コードの順番が悪いのかと思いforeachを一番下に書いてみたり、echoではなくprintで表示してみたり、echo $valueではなくecho $writeと書いてみたり色々悪あがきをしてみたのですが改善しませんでした。

どなたかご教示宜しくお願いいたします。

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

MAMPを使いローカル環境で開発しています。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    退会済みユーザー

    2019/02/21 20:06

    余談だけど、投稿に「<>」とか、改行コードとか含んだら、破綻すると思う。

    キャンセル

  • mo__fumi

    2019/02/21 23:39

    ご指摘ありがとうございます!こちらこういう課題でして、書き方は課題に沿って書きましたので、またメンターに聞いてみます^^;

    キャンセル

回答 2

checkベストアンサー

+3

ファイルに書き込む前にechoしているのが原因なので順番を変えます。

<?php
    $fileName = "bbs.txt";
    $file = file($fileName, FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES);

    $num = sizeof($file) + 1;
    $userName = $_POST["userName"];
    $comment = $_POST["comment"];
    $time = date("Y/m/d");
    $write = $num. "<>". $userName. "<>". $comment. "<>". $time. "\n";

    $fp = fopen($fileName, "a");
    flock ($fp, LOCK_EX);
    fwrite($fp, $write);
    flock ($fp, LOCK_UN);
    fclose($fp);

    // $fileを再度読み込んでからecho
    $file = file($fileName, FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES);
    foreach ( $file as $value ) {  
        $line = explode("<>", $value);  
        echo $value."<br />\n";
    } 


上記では手を加えていませんが、$_POSTで送信されてきた値をそのまま保存したり別の処理に使用したりするのは危険なので、必ずバリデーションや整形処理を行ってからされることをお勧めします。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/21 20:00

    ご回答ありがとうございます!
    $fileを再度読み込むという部分は目からウロコでした。JSを書いている際に同じものは何回も書かないほうがいいと教えられてきたのでついその癖で思い込みを・・・時と場合によるのですね。
    また更なる改善点もご教示いただき感謝いたします。現状私の知識ではあまり理解できませんでした・・・しかしこの掲示板はまだまだ実装途中ですので仰って頂いたことも調べつつ進めていきます、ありがとうございました^^

    キャンセル

+2

書き込んだ直後の表示バグと思いますが、
読み込んでから書き込んでるのが問題です

具体的にコメント記載しておきます
解決策は、書き込もうとしてる内容もechoすることかな

    <?php
    // ここでbbs.txtを読み込んでるので今投稿された内容は読み込まない
    $fileName = "bbs.txt";
    $file = file($fileName, FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES);

    foreach ( $file as $value ) {  
        $line = explode("<>", $value);  
        echo $value."<br />\n";
    } 

    // ここでbbs.txtに反映してる(ので投稿しようとしてるのはechoされない)
    $num = sizeof($file) + 1;
    $userName = $_POST["userName"];
    $comment = $_POST["comment"];
    $time = date("Y/m/d");
    $write = $num. "<>". $userName. "<>". $comment. "<>". $time. "\n";

    $fp = fopen($fileName, "a");
    flock ($fp, LOCK_EX);
    fwrite($fp, $write);
    flock ($fp, LOCK_UN);
    fclose($fp);

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/21 19:56

    ご回答ありがとうございます!
    なるほど、やはり`foreach`は一番下に書くべきだったのですね。
    確かに、全て読み込んでからechoしないと表示されませんよね・・・今後気をつけます。
    私の知識不足で「書き込もうとしてる内容もechoする」の書き方がわからなかったのですが、もう一人の回答者cerfwebさんが実装してくださったものをみて理解できました^^;
    しかし、ヒントという形で示してくださりありがとうございました。もっと勉強します。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.59%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る