PHPでMySQLを使った検索のプログラム2

Question

まだ解決できないので、大変恐縮ですがご教授お願いします。どうかよろしくお願いいたします。数人にご教授いただいき、チェックボックスの選択には一つのkid（kodawari_keyの略）に対して２複数の値をもっているホテルを表示させるにはデータベースのtableを複数に分けて紐付けする必要があるとアドバイスをいただき、教えて頂いたとおりtableをホテル用とチェックボックス用とチェックボックスのid用とに分けて、SQL文で紐付けしたつもりだったのですが、チェックボックスをチェックしてから検索ボタンを押すと Fatal error: Call to a member function execute() on a non-object in C:\xampp\htdocs\koredake\xxxxx\ｘｘｘｘｘｘｘ.php on line 97 のようなエラーが出てしまいます。このことから紐付けのSQL文が間違っているのではと思うのですが、正解がわかりません。どなたか教えていただけたらと思っております。 //MySQLの部分　教えていただいた部分で大変恐縮です。 //ホテルの基本情報 CREATE TABLE t_hotels(id int not null primary key,name varchar(100) not null,price_min int not null,price_max int not null,address varchar(100) not null); INSERT INTO t_hotels VALUES(1,'HOTEL A',5000,10000,'栃木県・・・'), (2,'HOTEL B',5000,12000,'栃木県・・・'), //こだわり情報 CREATE TABLE t_kodawari_key(id int not null primary key,name varchar(20)); INSERT INTO t_kodawari_key VALUES(1,'温泉'),(2,'ランチ'),(3,' ディナー'); //ホテルごとのこだわり CREATE TABLE t_hotel_kodawari(hid int not null,kid int not null,unique key(hid,kid)); INSERT INTO t_hotel_kodawari VALUES(1,1),(1,2),(1,3),(2,1),(3,2),(3,3),(4,3); //温泉かランチにこだわりがあるところ SELECT hid,t3.name,t3.price_min,t3.price_max,GROUP_CONCAT(t2.name) as kodawari,t3.address //温泉かランチかディナーのうち２つ以上にこだわりがあるところ SELECT hid,t3.name,t3.price_min,t3.price_max,GROUP_CONCAT(t2.name) as kodawari,t3.address //PHP部分　前半省略

ビジネスホテルの条件検索

false ) ); $sql = "select * from t_hotels where 1 = 1 "; $condition = array(); //この部分が特に自信が無いです。 if( !empty( $_POST["kid"] )){ $sql = $sql . " left outer join kid on t_hotels.hid = kid.hid"; } if( !empty( $_REQUEST["price_min"] ) ){ $sql = $sql . " and price >= :price_min "; $condition[":price_min"] = $_REQUEST["price_min"]; } if( !empty( $_REQUEST["price_max"] ) ){ $sql = $sql . " and price <= :price_max "; $condition[":price_max"] = $_REQUEST["price_max"]; } if( !empty( $_REQUEST["address"] ) ){ $sql = $sql . " and ( address like :address ) "; $condition[":address"] = "%{$_REQUEST["address"]}%"; } $statement = $pdo->prepare( $sql ); $statement->execute( $condition ); $results = $statement->fetchAll(); ?>

検索結果
	ホテル名	宿泊料金	住所
.png" />		\

yambejp · Accepted Answer

＞＞あまり最初から詰め込みすぎずに、徐々に条件を付けていった方がいいですよ

＞ということはもっと単純な条件式で大丈夫なのでしょうか？

え～と、そういうことではなく、たくさん条件を最初から書くと
どの条件節でエラーになっているかわかりづらくなるということです。
今回の質問であれば「こだわり」の部分が問題だったのですから
ほかの上限や下限料金、住所などはばっさりコメントアウトしてまえば
エラーの切り分けができるので、どこを直せばいいか絞り込めます。
デバグをするときにはなるべく簡潔な処理まで切り詰めた方がいいということです。
（まぁ複合バグもあるのでケースバイケースですが・・・）

＞LEFT OUTER JOINを変更して

ちなみに、こまかく見てないのでなんともいえませんが
今回の例だとLEFT JOINした場合はWHEREやHAVINGの処理をかませないと
絞り込みになっていないかもしれません

＞何もcheckboxや他の項目をチェックしなかったら、全てのホテルが表示されてしまします。
＞本当はチェックしなかったら、表示されないようにしたいのですが。

通常の絞り込みのロジックはこう

WHERE 1 AND 条件1 AND 条件2 ・・・
AND検索なのですべての条件に当てはまるものが絞り込まれます。
条件が何もない場合なにも表示しないなら
フラグ管理して、条件句に条件を付加していくときにフラグを立てるようにし
フラグが0の際には条件句の最後に「AND 0」を足せばいいでしょう。

$flag=0;
$sql="SELECT * FROM tbl WHERE 1 ";

if(条件1){
$sql.="AND hoge='hogehoge' ";
flag=1;
}
if(条件2){
$sql.="AND fuga BETWEEN 'xxx' AND 'yyy' ";
flag=1;
}
if(条件3){
$sql.="AND piyo LIKE 'zzz%' ";
flag=1;
}

if($flag==0){
$sql.="AND 0 ";
}

yambejp · Answer

＞$sql = $sql . " left outer join kid on t_hotels.hid = kid.hid";

とありますが・・・
「kid」というテーブルがあるようにみえませんが？

あまり最初から詰め込みすぎずに、徐々に条件を付けていった方がいいですよ

PHPでMySQLを使った検索のプログラム2

＞＞あまり最初から詰め込みすぎずに、徐々に条件を付けていった方がいいですよ

この回答への補足

＞$sql = $sql . " left outer join kid on t_hotels.hid = kid.hid";

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング