PHPとMySQLを使ったフォームを作っていますが、うまくできないこと

質問

PHPとMySQLを使ったフォームを作っていますが、うまくできないことがあり書き込みさせていただきます。
わかる方、ご助力願えれば幸いです。

フォームから、送信されたデータを受け取って、MySQLに登録する際に、フォーム内に空欄があると、うまくMySQLのデータベースに追加ができなかったり、変更ができなかったりします。

コードを簡単に書くと…

<?php

//フォームからデータを取得
$name = $_POST['name'];
$address = $_POST['address'];
$age = $_POST['age'];

//データベース接続処理
$db = mysql_connect("localhost","user","password");
mysql_select_db("db_name",$db);
$rs = mysql_query("select * from tablename order by my_id desc",$db);

//データベースへの登録
$rs = mysql_query("INSERT INTO tablename(name,address,age)VALUES('$name','$address','$age')",$db);

?>

このような感じになります。
フォームには、名前、住所、年齢を入力しているわけですが、年齢が必須でないため、空欄として送られてくることもあります。
年齢に数値が入っている場合は、うまく登録されますが、空欄であった場合、特にエラーが出るわけでもなく、単に「登録されない」という状況がおきます。

データベースのテーブルのフィールドでは、ageは　int(11)でNULLはYes、defaultはNULLになっております。

自分なりに調べてみたつもりですが、知りたい情報に出合うことができませんでした。

よろしくお願い致します。

回答No.2

yambejp · ベストアンサー

たとえば、default値があるのですから、入力時点ではじくこともできます。

<?
$name="test_name";
$address="test_address";
//$age="test_age";
$list=array("name"=>$name,"address"=>$address,"age"=>$age);
foreach($list as $key=>$val){
if($val=="") unset($list[$key]);
}
$keys="`".implode("`,`",array_keys($list))."`";
$vals="'".implode("','",$list)."'";
$sql="INSERT INTO tablename({$keys})VALUES({$vals})";
print $sql;
?>

お礼

お礼が遅くなりました、申し訳ありません。

そうですよね。

PHPを初めて、毎回「自分の発想力次第だな」と思っているのに、
なかなか、それを実践できないので反省です。

大変参考になりました。

現在、アドバイスいただきました内容を基に、順調に動作しております。

本当にありがとうございました。

回答No.3

noname#243182

上手くいかない場合のコードでは NULL が文字列として扱われていませんか。

エラーを捕捉するコードを書くことが大切です。
http://jp.php.net/manual/ja/function.mysql-errno.php
http://jp.php.net/manual/ja/function.mysql-error.php

マニュアルは全て読んでください。必要なのは気合と根性と十分な睡眠です。

参考URL：

http://jp.php.net/manual/ja/ref.mysql.php

お礼

ありがとうございます。

int mysql_errno てのがあるんですね。

いつも、うまく動けば「これでいいや」って思っていましたが、おかしいな、と感じたら、原因をはっきりさせておいた方がいいですよね。

ありがとうございました。
参考にさせていただきます。

回答No.1

yambejp

データのバリデート（チェック）はしていないのでしょうか？
ユーザーから流れてくるデータは汚染されている前提でかならずチェックし想定外の
データはすべてはじいてください。

仮に今のままだとしても$ageに値提示がなくても'$age'は''となり、投入すると「0」として
認識されると思いますが・・・明確にロジック的には$ageが数値として有効であれば'$age'、そうでないならNULLを指示してみるか、(name,address,age)のageを指定しないような
書式をつかうのが妥当でしょう

お礼

ありがとうございます。

データのチェックは行っています。
今回の質問をわかりやすくするために省略いたしました。

その後、自分なりに回避策としてデータベース登録時の処理を

//データベースへの登録
if($age == ""){
$rs = mysql_query("INSERT INTO tablename(name,address,age)VALUES('$name','$address',NULL)",$db);
} else{
$rs = mysql_query("INSERT INTO tablename(name,address,age)VALUES('$name','$address','$age')",$db);
}

とすることで解決できました。
が、この方法では、もう少し大きなフォームで、入力が必須でない項目が複数できてしまった時に、その数だけの分岐を作ることになりそうなので、もう少しほかの方法を模索したいと思っています。

ちなみに、上記のコードを以下のように書いた場合はうまくできませんでした。

//データベースへの登録
if($age == ""){
$age = NULL;//$ageに何も含まれていなければNULLを代入
}
$rs = mysql_query("INSERT INTO tablename(name,address,age)VALUES('$name','$address','$age')",$db);

アドバイスありがとうございました。