MySQLの行の追加について

イメージ説明
写真のとおりレコードを追加することができたのですが、
送信ボタンを押すと空のデータまで挿入されます。
なぜ空のデータが挿入されるからわかりません。

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<title>ＰＨＰ基礎</title>
</head>
<body>

<?php
$dsn='mysql:dbname=phpkiso;host=localhost';
$user='root';
$password='';
$dbh= new PDO($dsn,$user,$password);
$dbh->query('SET NAMES utf8');

$nickname=$_POST['nickname'];
$email=$_POST['email'];
$goiken=$_POST['goiken'];

$nickname=htmlspecialchars($nickname);
$email=htmlspecialchars($email);
$goiken=htmlspecialchars($goiken);

print $nickname;
print '様<br />';
print 'ご意見ありがとうございました。';
print 'いただいたご意見「';
print $goiken;
print '」<br />';
print $email;
print 'にメールを送りましたのでご確認下さい。';

$mail_sub='アンケート受け付けました';
$mail_body=$nickname."様\nアンケートご協力ありがとうございました。";
$mail_body=html_entity_decode($mail_body,ENT_QUOTES,"UTF-8");
$mail_head='From: kenjinakagawa87@gmail.com';
mb_language('japanese');
mb_internal_encoding("UTF-8");
mb_send_mail($email,$mail_sub,$mail_body,$mail_head);

$sql='INSERT INTO anketo (nickname,email,goiken) VALUES("'.$nickname.'","'.$email.'","'.$goiken.'")';
$stmt=$dbh->prepare($sql);
$stmt->execute();

$dbh=null;

?>
</body>
</html>

このコードから送信するのですが、どこが原因で空のデータまで挿入されるのでしょうか？

退会済みユーザー

2017/12/17 14:45

このコードだけでは二重登録は起こりません。この前のページのスクリプトや、他の要因があるかもしれません。

nakaken

2017/12/17 16:32

「いきなりはじめるPHP」という本の通りに操作しているのですが、前のスクリプトではデータベースに接続するコードは入力していないのですが、ローカル環境で構築しているのが原因ですか？XAMPPでMySQLを利用しているからなのか？わかりません。

回答2件

おそらくpostでデータがきちんと渡されていないのだと思います。
それとprepareが有効に処理されていません。
直にユーザーから受け取ったデータを書くことはNGです

$sql='INSERT INTO anketo (nickname,email,goiken) VALUES("'.$nickname.'","'.$email.'","'.$goiken.'")';

の次の行に

PHP
1print $sql;

してみてください

$nickname=htmlspecialchars($nickname);

$email=htmlspecialchars($email);
$goiken=htmlspecialchars($goiken);

DBに投入する前に生データを返還するのはただしくありません

また基本的な考え方としてユーザーが入力したメールアドレスに対して
メールをおくることは悪意のメールを送り付けるスパムの温床に
なりかねないので、あまりいい方法ではありません。
もしそうする場合でも、おなじメールアドレス宛に何度もメールが
送られないような仕掛けをつけて対応して下さい

投稿2017/12/18 00:49

yambejp

総合スコア109894

情報が足りないので

$nickname,$email,$goiken
についてそれぞれ「null」なのか「""」(空文字)なのかを調べる
（printで出力されているか、メールの中身に出力されているかなど）
ことと

$sql='INSERT INTO anketo (nickname,email,goiken) VALUES("てすと太郎","test@test","てすとです")';
みたいに変数を使わずinsertできるかをまず調べるのはいかがでしょう？

投稿2017/12/17 15:04

miyasaka

総合スコア271

回答へのコメント

nakaken

2017/12/17 16:36

回答ありがとうございます。すみません。初心者で＞$nickname,$email,$goiken についてそれぞれ「null」なのか「""」(空文字)なのかを調べる。ということがどういうことかわかりません。とりあえず、表示はされているので、空ではないかと思うのですが、＞$sql='INSERT INTO anketo (nickname,email,goiken) VALUES("てすと太郎","test@test","てすとです")'; と入力したら２行同じデータが挿入されました。

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう！