複数検索時の条件分岐について

Question

環境：php5、mysql5

条件検索はできるのですが検索状況の違いによる条件分岐について
どなたがアドバイスをお願いします。
$sql内でWHERE以降を「 if ～ else ～」などを使って変更するか
WHILE以降を変更する気がするのですがやり方がよくわかりません。
よろしくお願いします。

以下設定

テーブル　：tbl_item [int]
フィールド1：item_id [int]
フィールド2：cate_id [int] ←他テーブル(名前に変換)
フィールド3：item_name [varchar]
フィールド4：price [decimal]
フィールド5：stock_id [int]　　　←他テーブル

条件1：cate_id(プルダウン)を検索条件に加えている時
$sql = "SELECT tbl_item.*, tbl_stock.item_stock FROM tbl_item
LEFT JOIN tbl_stock ON tbl_item.item_id = tbl_stock.item_id
WHERE
tbl_item.cate_id = '" . mysql_real_escape_string( $_GET['cate_id'] ) . "' AND
tbl_item.item_name LIKE '%" . mysql_real_escape_string( $_GET['item_name'] ) . "%' AND
price BETWEEN '" . mysql_real_escape_string( $_GET['price_s'] ) . "' AND '" . mysql_real_escape_string( $_GET['price_e'] ) . "'
ORDER BY tbl_item.item_id desk LIMIT ${_GET['page']}, ${_GET['rows']} ";
$result = mysql_query( $sql ) ;
$num_rows = mysql_num_rows( $result );

条件2：cate_id(プルダウン)を検索条件に加えていない時
$sql = "SELECT tbl_item.*, tbl_stock.item_stock FROM tbl_item
LEFT JOIN tbl_stock ON tbl_item.item_id = tbl_stock.item_id
WHERE
tbl_item.item_name LIKE '%" . mysql_real_escape_string( $_GET['item_name'] ) . "%' AND
price BETWEEN '" . mysql_real_escape_string( $_GET['price_s'] ) . "' AND '" . mysql_real_escape_string( $_GET['price_e'] ) . "'
ORDER BY tbl_item.item_id desk LIMIT ${_GET['page']}, ${_GET['rows']} ";
$result = mysql_query( $sql ) ;
$num_rows = mysql_num_rows( $result );

・
・
アイテム数 <?= $num_rows ?> 種類
・
・
<? WHILE ( $item = mysql_fetch_array( $result ) ) { ?>
・
・
・
<? $n ++; } ?>

shimix · Accepted Answer

>>最初のSQL文とあとからのSQL文を両方printで表示させて >この調べ方がよく解らないです。ということなので、この部分だけ。 $sqlという変数への代入を書いていますよね？そのあとで$sqlをmysql_query()に渡す代わりに　　print "
SQL_String = " . $sql; などとして表示させます。それから別の$sqlへの代入も転記して同じようにprintします。最後にexitを追加して、ここまでで処理を終了させます。で、ブラウザに表示されたSQL文をじっくり比較すれば「どこか違う」ハズです（まったく同じなら違う結果にはならないので）。その違いを元のソースを追いながら考えてください。（すでに言及されていますが）表示されたSQL文を修正しながらphpMyAdminなどで実行させて「こういう記述にすればOK」となった時点でソースに反映させるのが一番いいです。

shimix · Answer

＃連投失礼（汗差し出がましいようですが・・ >ブラウザ上で真っ白になっているので文法エラーだと思います。通常エラーがあれば、そのエラー内容が表示されるのがphpです。本番環境ではセキュリティ上の理由でエラー表示を抑制したりしますが、少なくともデバッグ時にはエラー表示を抑制しないのがスジです。（参考）　　http://www.php.net/manual/ja/function.error-reporting.php 適宜printなどを挿入して、どこまで動いているのかを確認してください。たとえば　　$num_rows = mysql_num_rows( $result ); の次行に　　print "
Num_Rows = " . $num_rows; といった感じでいいと思います。

wp_ · Answer

>マニュアルを何十回も何百回も読みなさいとは言わないで下さい。
>悲しくなります・・・。
キツイ事申しますが、「PHPはマニュアル以上のドキュメントはない」です。
教えてくれる人以上に親切なドキュメントです。見れば答えが載ってるわけですから。
人に聞く前に「既にある答え」を知るのは最低限のマナーです。
悲しくなることを承知で申しますが「あと42回読みましょう」
// 42回の所以は42でぐぐりませう。


・・・と書いてみましたが、
これまでの経緯から察するに質問者様の場合はマニュアル読むよりも
原因の究明を行うセンスを磨くと良いと思います。

SQLとの連携の場合は
・SQLクエリをphpで吐き出す前に「正常なSQL」を先に組み立ててdebug
・phpからSQLクエリをechoし、それをDBに直接叩いて正常なクエリか確認
・条件分岐する場合はクエリそのものを変える
・動的パラメータの場合はSQL内にPHP変数を埋め込む
・インジェクション対策
この辺の順序に注力すると開発効率が上がると思いますよ。
質問文では三つ目のものが該当しますが、それ以前にやるべきことが二つあります。
振り分けは「正常に吐き出されるべきSQL文を完璧に構築」してからでも遅くありません。

// 当方的には「SQLクエリ自体がオブジェクト」という理念で
// SQLそのものをクラス化してます。
// 所詮はカラム名、from句、where句、order句などを動的に生成しているだけなので
// setter(動的パラメータ設置)とgetter（SQL取得）さえキッチリ作ってしまえば
// 単一のSQL文を作るのには楽だったりします。
// 実力が上がればサブクエリやLEFT JOINなど実装したりすることも出来ますしね。

直接的な回答にはなっていませんが、参考まで。

naktak · Answer

> ただ、「SELECT～FROM～」、「LEFT JOIN～ON～」、「ORDER BY～」は皆
> さん同じですね。
私は質問にあったSQL文字列をコピペしただけですので、
SQLの保証はしていません。
個人的にLEFT JOINは使わない派です＾＾；
直感的に分かりづらいので。

> 残念ながら、なぜか今回のにも挑戦したのですが上手く動作しません。
何を以って『上手く動作しない』のでしょうか？
$result = mysql_query( $sql ) ;を実行後にMySQL側のSQL文法エラーが
発生していない事は確認しましたか？
また、正しくSQLが発行されたとしても、それが意図するレコードを取得してくれていないのですか？
PHPは関係なく、意図するレコードを取得してくれるSQLは2パターン出来上がっていますか？

> この最後には『;』をつけなくても良いのですか？
いりません。（確か、あっても動作しますが）
そこら辺を気にしないとSQLインジェクションという脆弱性に絡んできます。
（MySQLはバージョンによってはSQLの文法エラーになるか、回避されるか
だった気もしますが）

> >if (条件1：cate_id(プルダウン)を検索条件に加えている時) {
> のif()の中には、この場合どのように書いたら良いのでしょうか？
0 選択なし
1 選択A
2 選択B
というプルダウンであり、要素の名前がcate_idだった時、
if ($_GET['cate_id'] == 1 or $_GET['cate_id'] == 2) {
などとなるはずです。
ここはMySQLのお話とは関係なしです。どう制御したいか、というお話になります。

shimix · Answer

>この最後には『;』をつけなくても良いのですか？

つけなくても良いです（というかつけてはいけません）。

「検索できなくなる」という件ですが、最初のSQL文とあとからのSQL文を両方printで表示させて違いを見れば（ブラウザの表示では見つけにくいのでブラウザのソース表示から切り出してエディタなどに貼り付けて）MySQLに投げているSQLが必ず違っているハズです。

naktak · Answer

違いがあるのって
tbl_item.cate_id = '" . mysql_real_escape_string( $_GET['cate_id'] ) . "' AND 
ここだけですよね？

$sql = "SELECT tbl_item.*, tbl_stock.item_stock FROM tbl_item
LEFT JOIN tbl_stock ON tbl_item.item_id = tbl_stock.item_id
 ";
$where = "WHERE ";
if (条件1：cate_id(プルダウン)を検索条件に加えている時) {
$where = "tbl_item.cate_id = '" . mysql_real_escape_string( $_GET['cate_id'] ) . "' AND "
}
$where .= "tbl_item.item_name LIKE '%" . mysql_real_escape_string( $_GET['item_name'] ) . "%' AND
price BETWEEN '" . mysql_real_escape_string( $_GET['price_s'] ) . "' AND '" . mysql_real_escape_string( $_GET['price_e'] ) . "'
ORDER BY tbl_item.item_id desk LIMIT ${_GET['page']}, ${_GET['rows']} "
$sql .= $where;

echo $sql;
$result = mysql_query( $sql ) ;
$num_rows = mysql_num_rows( $result );

これで十分だと思いますけど・・・。
生成されたSQL文をechoしておいて、思いどおりの結果が得られないなら、
echoされたSQL文をMySQL側で直接流して、ともかくSQLが正しくなる状態がわかってから、
PHPでの『SQL生成制御処理』に手を加えれて、echoで確認すればいいと思います。
つまり、PHPに書くSQLは『動作が保証されているSQL』である前提があれば、
問題の早期発見も出来るはずです。

あと、変数を文字列に設定するとき、命令を利用している所は
{}を利用していないのに、ただの変数の時は{}を利用していて
分かりづらいので、一切{}を使わない方が良いですよ。

shimix · Answer

>何も検索されなくなってしまいました。

一応これだけにはレスを（汗

最終的なSQL文を確認してみてください。多分「where」という文字列が抜けていませんか？私は$whereを連結する前のリテラルに入れてることが多いのですが、説明を省略し過ぎましたかね・・。でもSQL文をprintすれば気付きそうな（＠言い訳モード）

やはりこう書くべきだったか・・

　　$sql = 'select * from ～～ where ' . $where . ' order by ～～';

yambejp · Answer

方法を３つほど紹介します。とりあえずエスケープはあらかじめやっておきます $_GET['cate_id'] =mysql_real_escape_string($_GET['cate_id']); $_GET['item_name'] =mysql_real_escape_string($_GET['item_name']); $_GET['price_s'] =mysql_real_escape_string($_GET['price_s']); $_GET['price_e'] =mysql_real_escape_string($_GET['price_e']); そこで・・・(1)姑息ですが、汎用性があるもの $sql = <<"; print_r($sql1); print ""; //コメントをはずしたいものには第２引数を配列でわたす $sql2=cutComment($sql,Array("cate_id")); print "

";
print_r( $sql2);
print "

"; ここまでくればもうちょい工夫するとsqlは外部ファイルにして管理できます。

shimix · Answer

基本的にSQL文のwhereを書き替えますね。データベースに処理させてから取り出すほうが確実に速いですから（というかそうなるように設計すべき）。

　　$where = '(1)';　　// 必ず設定される条件があればそれを記述してもOK

としておいて、条件が追加のたびに

　　$where .= 'and (～追加条件～)';

　などとandで追記しておいてから、

　　$sql = 'select * from ～～' . $where . ' order by ～～';

　などとしています。whileのLoopなどで自前でチェックするのは「どうしてもそれしか方法がない」ときだけですね。CASE WHENなどを多用してSQL文が長くなっても実行するデータベースエンジンは気にしませんから。

複数検索時の条件分岐について

>>最初のSQL文とあとからのSQL文を両方printで表示させて

この回答への補足

＃連投失礼（汗

この回答への補足

>マニュアルを何十回も何百回も読みなさいとは言わないで下さい。

この回答への補足

> ただ、「SELECT～FROM～」、「LEFT JOIN～ON～」、「ORDER BY～」は皆

この回答への補足

>この最後には『;』をつけなくても良いのですか？

この回答への補足

違いがあるのって

この回答への補足

>何も検索されなくなってしまいました。

この回答への補足

方法を３つほど紹介します。

この回答への補足

基本的にSQL文のwhereを書き替えますね。

この回答への補足

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

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

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

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