初回読み込み時の変数エラー

Question

初回読込時のUnderfind Indexエラーが出てしまう

送信元|送信先：hoge.php

全体をif(isset($_REQUEST['fname']){}で囲って、else側に全体のソースコードをいれたら初回の読み込みエラーは無くなったが、ファイルを選んでもアップロードされなくなってしまう。

      <form action="hoge.php" method="post" enctype="multipart/form-data">
        <input type="file" name="fname">
        <input type="submit" value="アップロード" onclick="submitChk();">
      </form>

      $tempfile = $_FILES['fname']['tmp_name'];
      $filename = 'img/bg/' . $_FILES['fname']['name'];
      if (is_uploaded_file($tempfile)) {
          if ( move_uploaded_file($tempfile , $filename )) {
        echo $filename . "をアップロードしました。";
          } else {
              echo "ファイルをアップロードできません。";
          }
      } else {
          echo "ファイルが選択されていません。";
      }
      

Answer 1

くどいかもしれないけど、書き直してみた：

<?php
    if ((isset($_SERVER['REQUEST_METHOD'])) && ($_SERVER['REQUEST_METHOD'] === 'POST')) {
        if (isset($_FILES['fname'])) {
            $tempfile = $_FILES['fname']['tmp_name'];
            $filename = 'img/bg/' . $_FILES['fname']['name'];  // $_FILES['fname']['name'] は細工されると面倒だからサーバー側で自動生成したほうがいいかも
            if (is_uploaded_file($tempfile)) {
                if ( move_uploaded_file($tempfile , $filename )) {
                    echo $filename . "をアップロードしました。";
                } 
                else {
                    echo 'アップロードされたファイルの処理に失敗しました。';
                }
            }
            else {
                echo 'ファイルアップロードがありません。';  // ここに来ることってあるのかなぁ
            }
        }
        else {
            echo 'ファイルアップロードがありません。';
        }
    }

Comments

[kingkamehameha]

最初のif文のコードは、POSTされた時に同じ型(ここでいうと同じform？)の時を判定しているという意味合いで合っていますでしょうか？

[退会済みユーザー]

POST受信しているかどうか、を判定しています。
ファイルアップロードは基本的にPOSTによって行われます：
PHP: POST メソッドによるアップロード - Manual <https://www.php.net/manual/ja/features.file-upload.post-method.php>
そのため、くどいけど、POSTだよねってところから丁寧に検証しています。

[kingkamehameha]

理解しました。有難う御座います。
ちなみに成功した際に以下のようなSQLを入れると、毎度2回実行されてしまいます。
select文で取得するなんかは、決めた回数通り出来ているのですが。。。
$sql ="INSERT INTO sainageimg (filename) values ('$filename')";
$stmt = $pdo -> query($sql);
$stmt->execute();

[退会済みユーザー]

query()じゃなくてprepare()じゃないかな。PHPマニュアルで関数やメソッドの仕様を確認する習慣をつけましょう。

[kingkamehameha]

いつも同じ関数を使っていましたので、queryとprepareの違い学習したいと思います。

Answer 2

issetより$_SERVERのREQUEST_METHODがPOSTかどうかで判定された方が良いかと思います。

Answer 3

S_FILESで定義されていない要素にアクセスしているのでは？

Comments

[kingkamehameha]

恐らくそうですよね。