1

ある教材を元にphp,MySQLを勉強しています。

フォーム情報が正しく入力されると「1was inserted」と表示され、
データベースに情報が保存されるはずですが、正しく処理されません。

※「-1was inserted」と「-」がついてしまいます。
※データベースにも保存されません。

何が原因か教えて頂けますでしょうか?


<html>
<head>
</head>

<body>

<form method="post" action="insert.php">
Name: <input type="text" name="name" /><br />
Email: <input type="text" name="email" /><br />
Password: <input type="password" name="password" /><br />
<input type="submit" value="register" /><br />
</form>

</body>
</html>

<?php

$name = $_POST["name"];
$email = $_POST["email"];
$password = $_POST["password"];

if($name && $email && $password){
    mysql_connect("localhost","root","") or die("we could not connect!");
    mysql_select_db("testsite");
    mysql_query("INSERT INTO users(name,email,password) VALUES ($name,$email,$password)");
    $registered = mysql_affected_rows();
    echo "$registered was inserted";
} else {
    echo "you have to complete the form";
}

mysql_close();

?>
1
  • もしもよろしければ、どのような教材を使われているのか教えていただけないでしょうか?
    – ockeghem
    Commented 2015年5月9日 3:27

1 件の回答 1

4

mysql_affected_rows は最終クエリが失敗すると -1 を返します。

https://stackoverflow.com/questions/6722809/mysql-affected-rows-returns-1

理由ですが、$name$email$password はおそらく文字列です。"..." の中で使用するとリテラルとして扱われます。例えば $nametest$email[email protected]$passwordmy-password と入っていた場合、クエリは

INSERT INTO users(name,email,password) VALUES (test,[email protected],my-password)

となります。実際に期待しているのは

INSERT INTO users(name,email,password) VALUES ('test','[email protected]','my-password')

です。ですので

mysql_query("INSERT INTO users(name,email,password) VALUES ('$name','$email','$password')");

として下さい...と書く人もいますが、これが一番やってはいけない事です。

mysql_query(sprintf("INSERT INTO users(name,email,password) VALUES ('%s','%s','%s')",
    mysql_real_escape_string($name),
    mysql_real_escape_string($email),
    mysql_real_escape_string($password)));

といった具合に mysql_real_escape_string を使用してエスケープするか、PDO 等を使って prepared statements にするべきです。

http://php.net/manual/ja/pdo.prepared-statements.php

なぜやってはいけないかについては徳丸さんのサイトを見ると勉強になると思います。

http://blog.tokumaru.org/

1
  • ご丁寧にありがとうございます! 大変勉強になりました。。。 Commented 2015年5月8日 9:36

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。