SQL文をスマートに記述したい

Question

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・・・のところをもっと簡潔に記述する方法はありますか？
他にもお気づきの点があればご教示いただきたいです。

Comments

[Orlofsky]

コードは https://teratail.com/help/question-tips#questionTips3-7 の [コード] に修正してください。

[Orlofsky]

質問にCREATE TABLE文も載せてください。

[m.ts10806]

SQLだけでいきたいのでしたらPHPなどプログラミング言語要素は排除してSQL文だけを提示したほうが良いでしょう。
それとも「プログラミング言語側でスマートに記述したい」ということでしょうか。
いずれにしてもコードはマークダウンのcode機能を用いてご提示ください。

[m.ts10806]

細かいですがtag3が2回出てきてます。

[m.ts10806]

「character」という名称のテーブルは予約語です。使わない方が良いでしょう。
https://dev.mysql.com/doc/refman/5.6/ja/reserved-words.html
確かに`character` とすれば通るのは通りますが、あまりよろしくはありません。

[nogimareske]

皆様数々のご指摘ありがとうございます。
これから修正いたします。

[gentaro]

修正するならタイトルも直したほうが良い。

[gentaro]

「SQL分」→「SQL文」の事だったんだけど。

[nogimareske]

gentaro様
すいません、誤字に気づいていませんでした。
今訂正しました。

Answer 1

スマートに処理するにはテーブル構造を変えて、tag2～5を正規化して別テーブルに管理します
そもそもLIKEで%word%を処理するのはインデックスが効かないのでスマートではありません。
キーワード検索はスペース区切りにして全文検索にするなどパフォーマンスをあげる工夫が必要です

Comments

[nogimareske]

そもそものテーブル設計が失敗だったんですね。。。
正規化やインデックスについて調べてみます。
参考になるご回答ありがとうございます。

Answer 2

下記はやったらダメな例

SELECT code,name,about FROM `character`
 WHERE concat(tag1,tag2,tag3,tag4,tag5) LIKE '%word%'
 ORDER BY code DESC LIMIT 0,10

なぜなら下記のようになっていた時

tag1	tag2	tag3
りんご	ごりら	らっぱ

「ごご」や「らら」で引っかかるから。
セパレータ入れてもあまり意味ないですし、
既に出ているようにテーブル構成を見直したほうが良いです。
テーブル構成みなせないなら1つずつorでつなげる今のやり方が最も確実です。

Comments

[yambejp]

適当なセパレータを設定してやればconcatが一番有効ですけどね
concat(tag1,'|',tag2,'|',tag3,'|',tag4,'|',tag5)
※「|」が使われないという前提

[m.ts10806]

フリー入力だと容赦なく何でも入ってきそうなので「ダメな例」としました。

[nogimareske]

characterは予約語だったんですね。。。知らずに使ってしまいました。
テーブル構成は今から見直せるので見直すことにします。
正規化やインデックスについて調べて設計を考え直そうと思います。
丁寧にいろいろなことをご指摘いただきありがとうございました。