会員データをうまくテーブルに挿入され、会員登録完成させたいです。

前提・実現したいこと

phpの初心者です。

Windowsを使用し、開発環境Eclipse、サーバーはXMAPPを使用しています。

現在、たにぐちまこと著「よくわかるPHPの教科書PHP5.5対応版」で学習を進めております。

PRACTICE2 「Twitter風ひとこと掲示板を作る」の中で、ユーザー登録の際に、上手くユーザーのデータがデータベースにINSERTできなく、ボタンを押下すると、同じページへ遷移しました。

また、
ユーザー登録ページ初期表示について、
本に書かれた通りで記入すると、入力欄に変な文字列が代入されておりましたので、少し改造しました。
改造後、エラー項目があっても、エラー文が表示されませんでした。

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

<会員登録画面> index.php
改造前のソースはコメントアウトの部分。
エラーメッセージは、「Undefined index：未定義のインデックス」と表示されました。

<会員登録確認画面> check.php

該当のソースコード

データベース名：mini_bbs
テーブル名：members

テーブル構造：	定義	AI
id	INT(11)	○
name	VARCHER(255)
email	VARCHER(255)
password	VARCHER(100)
picture	VARCHER(255)
created	DATETIME
modified	TIMESTAMP

DB接続

php
1<?php
2try {
3  $db = new PDO('mysql:dbname=mini_bbs;host=localhost;charset=utf8', 'root', '');
4} catch (PDOException $e) {
5  echo 'DB接続エラー: ' . $e->getMessage();
6}
7 ?>

会員登録画面<index.php>

php
1<!DOCTYPE html>
2<html lang="ja">
3<head>
4<meta charset="UTF-8">
5<title>会員登録処理</title>
6</head>
7<body>
8    <h1>会員登録</h1>
9    <?php
10    require('../dbconnect.php');
11
12    session_start();
13
14    //フォーム画面の処理　初期化
15    $_POST=array();
16    $_SESSION = array();
17
18    if(!empty($_POST)) {
19      //エラー項目の確認
20      if($_POST['name'] == '') {
21        $error['name'] = 'blank';
22      }
23      if($_POST['email'] == '') {
24        $error['email'] = 'blank';
25      }
26      if(strlen($_POST['password']) < 4) {
27        $error['password'] = 'length';
28      }
29      if($_POST['password'] == '') {
30        $error['password'] = 'blank';
31      }
32      $fileName = $_FILES['image'] ['name'];
33      if(!empty($fileName)) {
34        $ext = substr($fileName, -3);
35        if($ext != 'jpg' && $ext != 'gif') {
36          $error['image'] = 'type';
37        }
38      }
39
40      //重複アカウントのチェック
41      if(empty($error)) {
42        $member = $db->prepare('SELECT COUNT(*) AS cnt FROM members WHERE email=?');
43        $member->execute(array($_POST['email']));
44        $record = $member->fetch();
45        if($record['cnt'] > 0) {
46          $error['email'] = 'duplicate';
47        }
48      }
49
50      if(empty($error)) {
51        //画像をアップロードする
52        $image = date('YmdHis') . $_FILES['image']['name'];
53        move_uploaded_file($_FILES['image']['tmp_name'], '../member_picture/' . $image);
54
55        $_SESSION['join'] = $_POST;
56        $_SESSION['join']['image'] = $image;
57        header('Location: check.php');
58        exit();
59      }
60    }
61     ?>
62      <p>次のフォームに必要事項をご記入ください。</p>
63      <form action="" method="post" enctype="multipart/form-data">
64        <dl>
65          <dt>ニックネーム<span class="required">必須</span></dt>
66          <dd><input type="text" name="name" size="35" maxlength="255" value="<?php echo htmlspecialchars($_POST['name'], ENT_QUOTES,'UTF-8'); ?>" />
67          <?php if(!empty($sen1)):?>
68            <?php if($error['name'] == 'blank'): ?>
69              <p class="error">* ニックネームを入力してください</p>
70            <?php endif; ?>
71          <?php endif; ?>
72          </dd>
73          <dt>メールアドレス<span class="required">必須</span></dt>
74          <dd><input type="text" name="email" size="35" maxlength="255" value="<?php echo htmlspecialchars($_POST['email'], ENT_QUOTES,'UTF-8'); ?>" />
75          <?php if(!empty($sen1)):?>
76            <?php if($error['email'] == 'blank'): ?>
77              <p class="error">* メールアドレスを入力してください</p>
78            <?php endif; ?>
79            <?php if($error['email'] == 'duplicate'): ?>
80            <p class="error">* 指定されたメールアドレスはすでに登録されています</p>
81            <?php endif; ?>
82          <?php endif; ?>
83          </dd>
84          <dt>パスワード<span class="required">必須</span></dt>
85          <dd><input type="password" name="password" size="10" maxlength="20" value="<?php echo htmlspecialchars($_POST['password'], ENT_QUOTES,'UTF-8'); ?>" />
86          <?php if(!empty($sen1)):?>
87            <?php if($error['password'] == 'blank'): ?>
88              <p class="error">* パスワードを入力してください</p>
89            <?php endif; ?>
90            <?php if($error['password'] == 'length'): ?>
91              <p class="error">* パスワードは4文字以上で入力してください</p>
92            <?php endif; ?>
93          <?php endif; ?>
94          </dd>
95          <dt>写真など</dt>
96          <dd><input type="file" name="image" size="35" />
97            <?php if(!empty($image)):?>
98              <?php if($error['image'] == 'type'): ?>
99                <p class="error">* 写真などは「.gif」または「.jpg」の画像を指定してください</p>
100              <?php endif; ?>
101              <?php if(!empty($error)): ?>
102                <p class="error">* 恐れ入りますが、画像を改めて指定してください</p>
103              <?php endif; ?>
104            <?php endif; ?>
105          </dd>
106        </dl>
107        <div><input name="sen1" type="submit" value="入力内容を確認する" /></div>
108      </form>
109</body>
110</html>

会員確認画面 <check.php>

php
1<?php
2session_start();
3require_once ('../dbconnect.php');
4
5if(!isset($_SESSION['join'])) {
6	header('Location: index.php');
7	exit();
8}
9
10if(!empty($POST)){
11	//登録処理をする
12	$statement = $db->prepare('INSERT INTO members SET name=?, email=?, password=?, picture=?, created=NOW()');
13    $ret = $statement->execute(array(
14    		$_SESSION['join']['name']
15    	   ,$_SESSION['join']['email']
16    	   ,shal($_SESSION['join']['password'])
17    	   ,$_SESSION['join']['image']
18
19    ));
20	unset($_SESSION['join']);
21
22	header('Location: thank.php');
23	exit();
24}
25?>
26<!DOCTYPE html>
27<html lang="ja">
28<head>
29<meta charset="UTF-8">
30<title>会員登録処理</title>
31</head>
32<body>
33<h1>会員登録確認</h1>
34<form action="" method="post">
35  <input type="hidden" name="action" value="submit">
36  <dl>
37    <dt>ニックネーム</dt>
38    <dd>
39      <?php echo htmlspecialchars($_SESSION['join']['name'], ENT_QUOTES, 'UTF-8'); ?>
40    </dd>
41    <dt>メールアドレス</dt>
42    <dd>
43      <?php echo htmlspecialchars($_SESSION['join']['email'], ENT_QUOTES, 'UTF-8'); ?>
44    </dd>
45    <dt>パスワード</dt>
46    <dd>
47      【表示されません】
48    </dd>
49    <dt>写真など</dt>
50    <dd>
51      <img src="../member_picture/<?php echo $_SESSION['join']['image']; ?>" width="100" height="100" alt="">
52    </dd>
53  </dl>
54  <div>
55    <a href="index.php?action=rewrite">&laquo;&nbsp;書き直す</a>｜<input type="submit" value="登録する">
56  </div>
57</form>

会員登録完了画面<thanks.php>

php
1<!DOCTYPE html>
2<html lang="ja">
3<head>
4<meta charset="UTF-8">
5<title>会員登録処理</title>
6</head>
7<body>
8  <h1>会員登録完了</h1>
9  <p>ユーザー登録が完了しました</p>
10  <p>
11    <a href="../">ログインする</a>
12  </p>
13</body>
14</html>

試したこと

★会員登録画面について、
①コメントアウトの直下に書いたコードに変更しました。
例：
修正前>

修正後>

<dd><input type="text" name="email" size="35" maxlength="255" value="" />

　②エラー文は送信ボタンを押下前にエラー文が表示されましたので、
それに対して、判定文を追加しました。(ボタンをクリックされるかの判定を追加)

<?php if(!empty($sen1)):?>

<?php if($error['email'] == 'blank'): ?>
  <p class="error">* メールアドレスを入力してください</p>
<?php endif; ?>
<?php if($error['email'] == 'duplicate'): ?>
  <p class="error">* 指定されたメールアドレスはすでに登録されています</p>
<?php endif; ?>

<?php endif; ?>

★会員確認画面について、
特に試したことはございません。

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

環境情報について、以下の通りです。
・Windows10 64bit
・Eclipse バージョン4.4 (PHP バージョン5.5)
・XAMPP：バージョン3.2.1

上記のコードの記述も全て確認しましたが、DBへの接続もできており、ミスしている箇所が見つけられません。

先人のお力をお借りしたく、質問させていただきます。

何卒宜しくお願い申し上げます。

m.ts10806

2020/05/14 13:35

＞エラーメッセージは、「Undefined index：未定義のインデックス」と表示されました。エラーは全文コピペで提示してください。質問者による解釈が入ってしまっては正しい情報は伝わらず、アドバイスに繋がりません。行数なども出ているはずです。

m.ts10806

2020/05/14 13:36

それにUndefined indexはよく出るものです。原因もハッキリしています。調べれば分かることも多いのでは

dashu_sena

2020/05/14 18:42

失礼いたしました。エラー発生につきまして、修正前のソースとかかわりますので、改めて提示したソースを修正させていただきます。また、エラー文につきまして、以下の通りでございます。(⇒ニックネーム必須の入力欄value) Notice: Undefined index: name in C:\pleiades\xampp\htdocs\xampp\workspace\mailForm\join\index.php on line 66 Call Stack #TimeMemoryFunctionLocation 10.0223150696{main}( )..\index.php:0 " /> また、画面に表示された入力欄につきまして、以下の文字列が代入されました。 <br /><font size='1'><table class='xdebug-error xe-notice' dir='ltr' border='1' cellspacing='0' cellpadding='1'><tr><th align='left' bgcolor='#f57900' colspan= 少し見づらいかもしれませんが、内容のご確認を何卒よろしくお願い致します。

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

回答1件

ベストアンサー

ざっと見て気になるところだけ列挙します。

$_POST=array();

割りと早い段階でやってしまっています。
$_POSTや$_GETなどの自動で定義される変数を上書きしてはいけません。

ということで

if(!empty($_POST)) {

ここに絶対に入らないようになってますしね。

$POST

別の変数です。

INSERT INTO members SET

INSERT INTO table SET [column=value] は確かMySQLでしか使えない表現です。
多くのDBではINSERT INTO table ([columns]) VALUES ([values]) で、MySQLでも使えます。
なるべく多くで使える表現を覚えた方が良いかと思います。

※ただ実際はフレームワークやその仕組みでデータのやりとりをするので直にSQLを書く機会は減りました。あまり構文を気にしなくても良いのですけど

画像アップロード

確認画面で戻った場合、アップロードした使われない画像がそのまま残ります。
この対策はされたほうが良いのでは？
※閉じたときは仕方ないかもしれませんが

「確認用」と「登録用」は別のディレクトリした方が良いと思います。

よくわかるPHPの教科書PHP5.5対応版

書籍を変えましょう。いろんな理由がありますが、PHP5.5はサポートが切れて久しいですし、
今回のような実装があるコードは参考にしてはいけません。

プログラミングをしていく上で、様々なことを覚えていくと思いますが、
初期の初期で覚えておかないと初心者から抜け出せないのが下記3点です。

エラーをよく読むこと
デバッグをすること
公式ドキュメントで機能を確認すること（PHPの場合はPHPマニュアル。

今回の問題点の1つである$_POSTについてもちゃんとあります）

あとできれば、デバッグとも関係するのですけど、構文チェック機能のあるエディタは必須です。
コード整形機能や機能のサジェストもあるものが望ましいですね。
「IDE」と呼ばれる統合環境がより望ましいです。（eclipseなどが代表的です。動作は重たいですが）

投稿2020/05/14 21:21

編集2020/05/14 21:24

m.ts10806

総合スコア80850

dashu_sena

2020/05/15 05:49 編集

ご回答ありがとうございました。ご指摘していただいた部分を少し修正しました。 ① <会員登録画面> 　$_POST=array();　⇒　初期化するのはやめました。 ②<会員確認画面>　変数記述間違え $POST ⇒　$_POST ③insert文について、書き換えましたが、値を代入するには以下のような書き方できないですか。 > INSERT INTO members SET ⇒　INSERT INTO members (name,email,..,modified) values (?,?,..,NOW()); ご指摘していただいた部分を修正後、改めて動作確認を行いました。今回は別の問題で登録画面から確認画面へ遷移せず、そのまま完了画面へ飛ばされました。デバックチェックで確認した結果は、check.php の $_POST変数の値が空っぽだそうです。なにかコード記述漏れがあれば、ご指摘していただければ、幸いです。