【PHP】お問い合わせフォームに入力されたデータをデータベースに格納するプログラムについて

Question

・前回、myAdminを使用して作成したテーブルをPHPファイルとリンクについて質問させていただきました。
mysql関数が非推奨とのご指摘も頂きましたがまずこれを使って完成させたく思ってます。
・データベースの接続には成功したのです、SQLにつながりません。これはmyadminで作ったものが正しくないのか、
ソースコードが間違っているのか、あるいはやはり関数が非推奨だからなのか

ご存知の方が居られましたらご教授ください

<?php
//フォームから格納されたかを判定
if(isset($_POST['name']) && isset($_POST['phone']) && isset($_POST['mail']) && isset($_POST['message'])){

	$hostname = "localhost";	//ホスト名
	$userid = "ユーザ名";			//データベースユーザ名
	$passwd = "";			//接続パスワード
	$dbname = "データベース名";				//データベース名
	$DB = mysql_connect($hostname,$userid,$passwd,$dbname);    //db接続に必要な情報を変数に入れる

	//データベースに接続
	if(!$DB){
		echo "接続失敗<br />";
		exit;
	}else{
		echo "接続成功<br />";
	}

	//データベース選択
	$db_selected = mysql_select_db($dbname, $DB);
	if (!$db_selected){
		echo "db選択失敗<br />";
		exit;
	}else{
		echo "db選択成功<br />";
	}

	//mysqlの文字コード設定
	mysql_set_charset('utf8');

	//実行するクエリを変数に入れる
	$sql = "INSERT INTO contact_tbl(name,phone,mail,message,) VALUES ('" . $_POST['name'] . "','" . $_POST['phone'] ."','" . $_POST['mail'] . "','" . $_POST['message'] . "')";

	//クエリを実行して、フォームから入力されたデータをデータベースに挿入
	$result_reg = mysql_query($sql);
	if(!$result_reg){
		echo "SQL失敗<br />";
		exit;
	}else{
		echo "SQL成功<br />";
	}
	//データベース接続を切断
	mysql_close($DB);
	exit;
}

・画面に表示されるエラーメッセージ

接続成功
db選択成功
Notice: Undefined index: name in C:\xampp\htdocs\form.php on line 32
Notice: Undefined index: phone in C:\xampp\htdocs\form.php on line 32
Notice: Undefined index: mail in C:\xampp\htdocs\form.php on line 32
Notice: Undefined index: message in C:\xampp\htdocs\form.php on line 32
SQL失敗

online32→$sql = "INSERT INTO contact_tbl(･･･)の行

Answer 1

Noticeは配列にそんなインデックスは存在しませんということを言っています。
したがって、今回実行しているSQLは以下のようになります。

INSERT INTO contact_tbl(name,phone,mail,message,) VALUES ('','','','')

maisumakunさんが言っているようにカンマが余計なため、SQLが失敗するのだと思います。
SQL失敗と出力している箇所に以下を追記してみてください。

echo mysql_error();

上記コードでSQLの実行に失敗する理由が表示されますので、それを参考に直してみてください。
なお、今回SQLの実行に失敗することと関数が非推奨であることは関係ないと思われます。

また、maisumakunさんがおっしゃているように、POSTなど、外部から来たデータをそのままMySQLに投げるということは、絶対にやってはいけません。mysqlライブラリでプリペアドステートメントは利用できないので以下のようにすると少しはましです。

$sql = "INSERT INTO contact_tbl(name,phone,mail,message) VALUES ('" . mysql_real_escape_string($_POST['name']) . "','" . mysql_real_escape_string($_POST['phone']) ."','" . mysql_real_escape_string($_POST['mail']) . "','" . mysql_real_escape_string($_POST['message']) . "')";

Comments

[bh_C]

ご回答頂きましてありがとうございます！
SQLの’’とテキストの””がごっちゃになっており、ちゃんと書けてなかったのがひとつ。
そして32行目は確認画面ファイルと連動しているためPOSTではなくSESSIONに変えました。（そもそも$_POSTは何処にあるのか？という問題でした）
コードを書いてくださりありがとうございます。今後の参考にさせて頂きます！

Answer 2

ざっと見たところですが、INSERT INTO contact_tbl(name,phone,mail,message,) の最後にあるコンマは余計ですね。

あと、POSTなど、外部から来たデータをそのままMySQLに投げるということは、絶対にやってはいけません（どんなSQL文でも実行できてしまいますので、DBを破壊、任意のデータを抽出される危険もあります）。エスケープをかける、もしくは（mysqlライブラリでは非対応ですが）プリペアードステートメントを使うことを行ってください。

Comments

[bh_C]

ご回答頂きましてありがとうございます！
気づきませんでした…！

そしてエスケープ、プリペアードステートメントについて教えてくださってありがとうございます。
現在はローカルホストでただプログラムを書いて実行できるかどうか試している段階ですが、今後はそういったことにも注意していきたいと思います