phpで作る簡易的な掲示板の二重送信について

Question

完了画面でリロードすると内容が重複して投稿されてしまう。

回避方法としてはheader()関数でリダイレクトさせる方法です。

ページの構成としては下記の様な形です。
index.php・・・入力フォームを記述しているファイル
output.php・・・ファイルの書き込み処理と投稿一覧を表示しているファイル

■index.php

<form action="output.php" method="post">
  <input type="text" name="message">
  <input type="submit" name="datepost" value="投稿">
</form>

■output.php

<?php
$file = 'board.txt';
if ($_SERVER['REQUEST_METHOD'] === 'POST') {
  if (file_exists($file)) {
    $board = json_decode(file_get_contents($file));
  }
  $board[] = $_REQUEST['message'];

  file_put_contents($file, json_encode($board));

  foreach ($board as $message) {
    echo $message.'<hr>';
  }
  header('Location:http://192.168.33.10:8000/lessonform/output.php');
}
?>

上記コードで解決になりません。
この状態で「$_SERVER['REQUEST_METHOD']」するとPOSTのままになっております。
header関数の場所を変更してもうまくいきませんでした。

よろしければアドバイスなど頂けると。。。

Answer 1

header() 関数はそれより前に出力があると機能しません。
本当に場所を変えてもダメでしたか？
わかりやすくするために、redirect させる URL を自分自身ではなく、変えて試してみては？

Answer 2

一般的には「CSRF対策」としてトークンを発行してチェックします。

• とっても簡単なCSRF対策

Answer 3

if ($_SERVER['REQUEST_METHOD'] === 'POST') {

にisset($_REQUEST['message'])で入ってるかどうかチェックするのはだめですか

Comments

[rururu3]

あと、
echo $message.'<hr>';
があるので
header('Location:http://192.168.33.10:8000/lessonform/output.php');
はワーニングでリダイレクトしないですね。

[SENNA0510]

早々の返答ありがとうございます！

ですが、、、うまくいかず、、、
色々な記事を見てるのですが、、、

[SENNA0510]

ワーニングでリダイレクトしないというのは
ではecho $message.'<hr>';の部分を何か別の方法で記述すれば
うまくいきそうですかね！

[SENNA0510]

確かにecho $message.'<hr>';をカットしたらGETにかわりました！

立て続けに質問で申し訳ないですが
echo $message.'<hr>';
がワーニングになる理由は何でしょうか。。。

[rururu3]

http://php.net/manual/ja/function.header.php
header出力においてはHTTP仕様に基づくので、文章送ったあとでheader出力するのはNGです。
(出力順などでうまくいくときもあるんですが、基本的にNGと覚えておけば大丈夫です)

[SENNA0510]

以下のコード内にある<hr>をカットすればリロードしても平気にはなりましたが
<hr>や<br>などhtmlタグは読み込まれているようです、、、

<?php
$file = 'board.txt';

if (file_exists($file)) {
$board = json_decode(file_get_contents($file));
}
$board[] = $_REQUEST['message'];
file_put_contents($file, json_encode($board));

if ($_SERVER['REQUEST_METHOD'] === 'POST') {

header('Location:http://192.168.33.10:8000/php/lesson/userOutput.php');

} else {

foreach ($board as $message) {
echo $message.'<hr>';
}

}
?>