$_SESSION変数を使って入力フォームに表示したい【書き直し機能】

Question

入力ミスなどで，check.phpから書き直しでindex.phpに戻った場合，$_SESSION変数を利用して，ユーザの入力した内容を保持し，index.phpのフォームに表示したいと考えています。
これにより，入力ミスでない部分について再度入力する手間を省きたいと考えています。

今読んでいる「よくわかるPHPの教科書」という参考書では

//書き直し
if(isset($_REQUEST["action"]) && $_REQUEST["action"] === "rewrite") {
$_POST = $_SESSION["join"];
$error["rewrite"] = true;
}

このコードでそれが可能と書いてあったのですが、index.phpに書き直しで戻った際、フォームに何も入力されていないので困っています。

また、

if(isset($_REQUEST["action"]) && $_REQUEST["action"] === "rewrite") {
$_POST = $_SESSION["join"];
var_dump($_POST);
$error["rewrite"] = true;

で調べた際、

array(4) { ["name"]=> string(3) "aaa" ["email"]=> string(3) "aaa" ["password"]=> string(5) "aaaaa" ["image"]=> string(25) "20190923071935sample2.jpg" }

と表示されたのでセッションの受け渡しは上手くいっていると思います。

この場合、どこを修正したらよろしいでしょうか？

ご教授願います。

「index.php」

<?php
session_start();

if(!empty($_POST)) {
  //エラー項目の確認
  if($_POST["name"] === "") {
    $error["name"] = "blank";
  }
  if($_POST["email"] === "") {
    $error["email"] = "blank";
  }
  if(strlen($_POST["password"]) < 4) {
    $error["password"] = "length";
  }
  if($_POST["password"] === "") {
    $error["password"] = "blank";
  }
  $file = $_FILES["image"];
  if(!empty($file["name"])) {
    $ext = substr($file["name"], -3);
    if ($ext != "jpg" && $ext != "gif") {
      $error["image"] = "type";
    }
  }

  if(empty($error)) {
    //画像をアップロードする
    $image = date("YmdHis").$file["name"];
    move_uploaded_file($file["tmp_name"],"../member_picture/".$image);
    $_SESSION["join"] = $_POST;
    $_SESSION["join"]["image"] = $image;
    header("Location: check.php");
    exit();
  }
}
//書き直し
if(isset($_REQUEST["action"]) && $_REQUEST["action"] === "rewrite") {
  $_POST = $_SESSION["join"];
  $error["rewrite"] = true;
}

?>
<p>次のフォームに必要事項をご記入ください。</p>
<form action="" method="post" enctype="multipart/form-data">
  <dl>
    <dt>ニックネーム<span class="required">(必須)</span></dt>
    <dd>
      <input type="text" name="name" size="35" maxlength="255" value="<?php echo htmlspecialchars(filter_input(INPUT_POST,"name"), ENT_QUOTES); ?>">
      <?php if(isset($error["name"]) && $error["name"] ==="blank"): ?>
      <p class="error">*ニックネームを入力してください</p>
      <?php endif; ?>
    </dd>
    <dt>メールアドレス<span class="required">必須</span></dt>
    <dd>
      <input type="text" name="email" size="35" maxlength="255" value="<?php echo htmlspecialchars(filter_input(INPUT_POST,"email"), ENT_QUOTES); ?>">
      <?php if(isset($error["email"]) && $error["email"] === "blank"): ?>
      <p class="error">*メールアドレスを入力してください</p>
    <?php endif; ?>
    </dd>
    <dt>パスワード<span class="required">必須</span></dt>
    <dd>
      <input type="password" name="password" size="10" mexlength="20" value="<?php echo htmlspecialchars(filter_input(INPUT_POST,"password"), ENT_QUOTES); ?>">
      <?php if(isset($error["password"]) && $error["password"] === "blank"): ?>
      <p class="error">*パスワードを入力してください</p>
      <?php endif; ?>
      <?php if(isset($error["password"]) && $error["password"] === "length"): ?>
      <p class="error">*パスワードは4文字以上で入力してください</p>
      <?php endif; ?>
    </dd>
    <dt>写真など</dt>
    <dd>
      <input type="file" name="image" size="35">
      <?php if(isset($error["image"]) && $error["image"] === "type"): ?>
      <p class="error">*写真などは「.gif」または「.jpg」の画像を指定してください</p>
      <?php endif; ?>
      <?php if(!empty($error)): ?>
      <p class="error">*恐れいりますが、画像を改めて指定してください</p>
      <?php endif; ?>
    </dd>
  </dl>
  <div><input type="submit" value="入力内容を確認する"></div>
</form>

「check.php」

<?php
session_start();

if(!isset($_SESSION["join"])) {
  header("Location: index.php");
  exit();
}

?>
<p>記入した内容を確認して、「登録する」ボタンをクリックしてください</p>
<form action="" method="post">
  <dl>
    <dt>ニックネーム</dt>
    <dd>
    <?php echo htmlspecialchars($_SESSION["join"]["name"], ENT_QUOTES); ?>
    </dd>
    <dt>メールアドレス</dt>
    <?php echo htmlspecialchars($_SESSION["join"]["email"], ENT_QUOTES); ?>
    <dd>
    </dd>
    <dt>パスワード</dt>
    <dd>
    【表示されません】
    </dd>
    <dt>写真など</dt>
    <dd>
    <img src="../member_picture/<?php echo htmlspecialchars($_SESSION["join"]["image"], ENT_QUOTES); ?>" width="100" height="100" alt="">
    </dd>
  </dl>
  <div><a href="index.php?action=rewrite">&laquo;&nbsp;書き直す</a> | <input type="submit" value="登録する"></div>
</form>

ー書き直しのリンクを踏んだ際の画面ー

Comments

[m.ts10806]

$_file？

[pikumin777]

m.ts10806様、ご指摘ありがとうございます。

一箇所$fileでなく$_fileになってる所がありました。

修正したのですが、まだ問題は解決されてないです。

[m.ts10806]

「うまくいかない」では何が起きてるのか伝わらないので、具体的にこのコードで実現できるはずのことと、実際に起きている現象を記載してください。
あとできれば想定の箇所を通っているかデバッグして、その結果も追記してください

[pikumin777]

こちらの質問の趣旨としては
入力ミスなどで，check.phpから書き直しでindex.phpに戻った場合，$_SESSION変数を利用して，ユーザの入力した内容を保持し，index.phpのフォームに表示したいと考えています。
これにより，入力ミスでない部分について再度入力する手間を省きたいと考えています。

今読んでいる「よくわかるPHPの教科書」という参考書では
//書き直し
if(isset($_REQUEST["action"]) && $_REQUEST["action"] === "rewrite") {
$_POST = $_SESSION["join"];
$error["rewrite"] = true;
}
このコードでそれが可能と書いてあったのですが、index.phpに書き直しで戻った際、フォームに何も入力されていないので困っています。

[m.ts10806]

質問本文に追記願います
こちらに書かれてもデフォルト非表示ですので

Answer 1

質問主旨は全く理解していませんが、コードを見るとファイルアップロード部分が非常に危ういです。
ちゃんと安全な仕組みを学習した上で再実装してください。

Comments

[pikumin777]

こちらの質問の趣旨としては
入力ミスなどで，check.phpから書き直しでindex.phpに戻った場合，$_SESSION変数を利用して，ユーザの入力した内容を保持し，index.phpのフォームに表示したいと考えています。
これにより，入力ミスでない部分について再度入力する手間を省きたいと考えています。

[退会済みユーザー]

質問を理解するつもりは無いです。
脆弱なコードで公開するとサーバを乗っ取られるので指摘しただけです。
むしろ、脆弱な部分が修正されないまま動くほうがイヤです。

[pikumin777]

まだ、PHPを勉強し始めて数日なのでセキュリティの面まで考えるのは無理です。
これから勉強していく予定です。

[退会済みユーザー]

そうですね。まだファイルアップロード機能をつけるのは無理だと思います。
削除して、シンプルにしたもので質問してください。

参考）
phper によく参照される(?)記事
https://qiita.com/mpyw/items/939964377766a54d4682
https://qiita.com/mpyw/items/2c54d0ea95423bd88f60