Q&A
コードは https://teratail.com/help/question-tips#questionTips3-7 の [コード] に修正してください。
SQLをもっとスマートにか記述したいです。
データベースはMysqlを使っており、データベースにはtag1,tag2,tag3,tag4,tag5というカラムがあり、その中のいずれかに一致する値があればデータを引っ張ってくるようにしています。
【コード】
sql = "SELECT code,name,about FROM character WHERE tag1 LIKE word OR tag2 LIKE word OR tag3 LIKE word OR tag4 LIKE word OR tag5 LIKE word OR tag3 LIKE word ORDER BY code DESC LIMIT 1,10";
whereの後のtag1 LIKE word OR tag2 LIKE word OR・・・のところをもっと簡潔に記述する方法はありますか？
他にもお気づきの点があればご教示いただきたいです。
質問にCREATE TABLE文も載せてください。
SQLだけでいきたいのでしたらPHPなどプログラミング言語要素は排除してSQL文だけを提示したほうが良いでしょう。
それとも「プログラミング言語側でスマートに記述したい」ということでしょうか。
いずれにしてもコードはマークダウンのcode機能を用いてご提示ください。
細かいですがtag3が2回出てきてます。
「character」という名称のテーブルは予約語です。使わない方が良いでしょう。
https://dev.mysql.com/doc/refman/5.6/ja/reserved-words.html
確かに`character` とすれば通るのは通りますが、あまりよろしくはありません。
皆様数々のご指摘ありがとうございます。
これから修正いたします。
修正するならタイトルも直したほうが良い。
「SQL分」→「SQL文」の事だったんだけど。
gentaro様
すいません、誤字に気づいていませんでした。
今訂正しました。
回答2件
0
スマートに処理するにはテーブル構造を変えて、tag2～5を正規化して別テーブルに管理します
そもそもLIKEで%word%を処理するのはインデックスが効かないのでスマートではありません。
キーワード検索はスペース区切りにして全文検索にするなどパフォーマンスをあげる工夫が必要です
投稿2019/09/09 08:14
総合スコア114769
0
ベストアンサー
下記はやったらダメな例
sql
1SELECT code,name,about FROM `character` 2 WHERE concat(tag1,tag2,tag3,tag4,tag5) LIKE '%word%' 3 ORDER BY code DESC LIMIT 0,10
なぜなら下記のようになっていた時
| tag1 | tag2 | tag3 |
|---|---|---|
| りんご | ごりら | らっぱ |
「ごご」や「らら」で引っかかるから。
セパレータ入れてもあまり意味ないですし、
既に出ているようにテーブル構成を見直したほうが良いです。
テーブル構成みなせないなら1つずつorでつなげる今のやり方が最も確実です。
投稿2019/09/09 08:19
総合スコア80850
適当なセパレータを設定してやればconcatが一番有効ですけどね
concat(tag1,'|',tag2,'|',tag3,'|',tag4,'|',tag5)
※「|」が使われないという前提
フリー入力だと容赦なく何でも入ってきそうなので「ダメな例」としました。
characterは予約語だったんですね。。。知らずに使ってしまいました。
テーブル構成は今から見直せるので見直すことにします。
正規化やインデックスについて調べて設計を考え直そうと思います。
丁寧にいろいろなことをご指摘いただきありがとうございました。
あなたの回答
tips
プレビュー
