PHPでアイコン画像変更機能をつけようとしました

前提・実現したいこと

PHPで作った簡易掲示板に後からアイコン画像を変更できるシステムを作っています。

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

試してみたところ、ページが遷移せず、エラーメッセージもでません。
エラーメッセージ
なし

該当のソースコード

<?php
session_start();

if(!empty($_POST)) {
  print('ポストが送信されました');
  if ($_POST['reimage'] === ''){
    $error['icon'] = 'blank';
  } 
  $fileName = $_FILES['reimage']['name'];
  if(!empty($fileName)){
    // 画像がアップロードされていれば
    $ext = substr($fileName, -3);
    if($ext !== 'jpg' && $ext !== 'gif' && $ext !== 'png' ){
      $error['icon'] = 'type';
    }
  }
  if(empty($error)) {
    print('エラーの中身は空でした。');
    //$errorの内容が空であるかを確認する
    $image = date('YmdHis') . $_FILES['reimage']['name'];
    move_uploaded_file($_FILES['reimage']['tmp_name'], 'member_picture/' . $image);
    $_SESSION['icon'] = $_POST;
    // ポストで送信した内容をセッションに保存する
    $_SESSION['icon']['reimage'] = $image;
    var_dump($image);
    header('Location: iconcheck.php');
    exit();
  }
}
if(!empty($error)) {
  header('Location: changeicon.php');
  exit();
}
if ($_REQUEST['action'] === 'rewrite' && isset($_SESSION['icon'])) {
  $_POST = $_SESSION['icon'];
}
?>

<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="UTF=8">
<meta name="viewport" content="width=device-width, initial-scale=1,0">
<meta http-equiv="X-UA=Compatible" content="ie=edge">
<title>アイコンの変更</title>

<link rel="stylesheet" href="style.css" />
</head>

<body>
  <div id="wrap">
    <div id="head">
      <h1>アイコンの変更画面</h1>
        <div id="content">
        <form action="" method="post" enctype="multipart/form-data"><!-- ファイルを付け加える時にはこのenctypeの部分は必須項目 --> 
        <dl>
        <dt>写真など</dt>
		<dd>
        	  <input type="file" name="reimage" size="35" value="test"  />
              <?php if($error['icon'] === 'type'):?>
			  <p class="error">[.jpg]  [.png]  [.gif]の画像を設定してください</p>
	          <?php endif; ?>
              <?php if($error['icon'] === 'blank'):?>
			  <p class="error">画像を選択してください</p>
		      <?php endif; ?>
			  <?php if(!empty($error)): ?>
			  <p class="error">恐れ入りますが画像を改めて選択ください</p>
			  <?php endif; ?>
          </dd>
          </dl>
            <div><input type="submit" value="画像を確認する"/></div>
        </form>
   </div>
  </div>
</body>

【iconcheck.php】
<?php 
session_start();
require('dbconnect.php');

if(!isset($_SESSION['icon'])) {
  // sessionに値が保存されていなかった場合にこのこの後の処理をする
  // ブラウザから直接このページを呼び出した場合は戻ってもらう
  header('Location: changeicon.php');
  exit();
}
if(!empty($_POST)){
	$statement2 = $db->prepare('UPDATE members SET picture=?, created=NOW()');
	echo $statement2->execute(array(
    $_SESSION['icon']['reimage']
	));
	unset($_SESSION['icon']);

	header('Location: index.php');
	exit;
}
?>
<!DOCTYPE html>
<html lang="ja">
<head>
	<meta charset="UTF-8">
	<meta name="viewport" content="width=device-width, initial-scale=1.0">
	<meta http-equiv="X-UA-Compatible" content="ie=edge">
	<title>画像変更チェック</title>

  <form action="" method="post">
  <iuput type="hidden" name="action" value="submit"/>
  <dl>
  <dd>
    <?php if($_SESSION['icon']['reimage'] !== ''): ?>
	<img src="/member_picture/<?php print(htmlspecialchars($_SESSION['icon']['reimage'], ENT_QUOTES)); ?>">
    <?php endif; ?>
  </dd>
  </dl>
  <div><a href="changeicon.php?action=rewrite">&laquo;&nbsp;選び直す</a> | <input type="submit" value="登録する" /></div>
  </form>

試したこと

udemyで質問したのですが対応範囲外と言われました。
またそのときに少しだけヒントをもらい、var_dump($image)を入れてみたら？とアドバイスをいただいたので試してみましたが変化はありませんでした。

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

退会済みユーザー

2020/01/13 06:16

https://teratail.com/tour

m.ts10806

2020/01/14 06:48

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

Smith.A

2020/01/14 15:03

ご指摘ありがとうございました。

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

回答1件

ベストアンサー

ページが遷移せず、エラーメッセージもでません。

いえ、エラーは出るはずです。

header()

header() 関数は、通常の HTML タグまたは PHP からの出力にかかわらず、すべての実際の出力の前にコールする必要があることです。

こんな感じのエラーが出ます（正確にはWarning）
Warning: Cannot modify header information - headers already sent by (output started at …

学習段階、開発段階では常にエラー表示はONにしておくべきでしょう。

またデバッグについてはvar_dump()だけではなく「ここを通っている」という確認もする必要があります。
とりあえずheader()部分はコメントアウトしておくとして
echo __LINE__.PHP_EOL;　を各分岐の前後に設置して、どういう結果が出るか確かめてください。
これにより「処理がどこを通っているか」が確認できます。

想定の手順で画面を操作して想定の場所を通っていなければ、
その分岐で使われている変数を出力し、想定の情報が来ているか確かめます。

まずは、ここまでやってみてください。

投稿2020/01/14 15:09

m.ts10806

総合スコア80850

Smith.A

2020/01/15 00:40

やってみると最初のif文に入っておらず、echoで$_POSTを出力すると、Arrayと表示されました。これはどうゆうことなのでしょうか。

m.ts10806

2020/01/15 00:50

$_POSTは配列だからです。(PHPマニュアル参照) echoが正しく情報を出力できるのは基本的に文字列、数値情報のみです(PHPマニュアル参照) Udemy使ったことないですがあまりちゃんと基本を教えてもらってないようですね…。有料なのにデバッグ方法もちゃんと教えてないのはエンジニア育てる気がないと思って良さそうです。ひとまず、下記三点は早いうちに覚えましょう。･PHPマニュアルで機能の仕様を確認すること･エラーを読むこと･デバッグをすること

m.ts10806

2020/01/15 00:52

その他含めて下記の質問でしっかり回答書いてますのでご確認を。 https://teratail.com/questions/234484

Smith.A

2020/01/15 00:59

ありがとうございます。まだ始めたばかりなので基本の知識が定着していないまま質問をしてしまってすみません。確認してみます。ありがとうございました。

m.ts10806

2020/01/15 01:07

いえ、謝る必要はないかなと思います。むしろudemyが本来ちゃんと教えるべきことを教えず、これくらいのことを範囲外として対応していないことの方が問題ですね。プログラミング言語のほとんどはオープンソースでやろうと思えば独学で全て無料でできるものでもあります。 udemyよりも未経験就職でojtしてもらったほうがより実践的なスキルが身に付くとは思います(年齢や状況による。アウトソーシングが多くなる可能性もある)

Smith.A

2020/01/15 01:55

ありがとうございました。参照していただいたサイトなどを見て勉強したいと思います。

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