Q&A
toSqlなどでクエリは確認されましたか？
また、該当のソースコードでの結果はどうなるのでしょうか？
「発生している問題」「試したこと」で、同一条件で結果が違うように見えるので。
前提・実現したいこと
初めての投稿なので足りないことだらけになってしまうと思いますが...。
現在検索機能を作っていて、配列のリクエストが来た場合、
code1,code2,code3の3つのフィールドにAND検索を行いたい。
Sample Table
| sample_id | code1 | code2 | code3 | is_deleted |
|---|---|---|---|---|
| 1 | 700 | 500 | 600 | false |
| 2 | 600 | 100 | null | false |
| 3 | 200 | 500 | null | false |
| 4 | 800 | 500 | 600 | false |
上記のテーブルがあった場合、
リクエストが$codes = [100,200,300]としたら
検索結果として"sample_id = 2,3"のレコードを取得したい。
発生している問題
code1,code2,code3というフィールドにwhereInをかけているのですが、
条件に引っかからない場合スルーしてほしいのにひっかかり、
全件で返ってきてしまう(is_deletedのfalseが原因？)
該当のソースコード
php
1 $sample_query = Sample::query(); 2 $display = self::displayResultData(); 3 $mansion_query->where(self::SAMPLE_TABLE . '.is_deleted', false) 4 ->whereIn(self::SAMPLE_TABLE . '.code1', $codes) 5 ->whereIn(self::SAMPLE_TABLE . '.code2', $codes) 6 ->whereIn(self::SAMPLE_TABLE . '.code3', $codes) 7 ->select($display); 8 9 $result = $sample_query->get(); 10
※$displayはselectで出したいフィールド名が書いてあります。
試したこと
whereの文をコメントアウトし、WhereIn(self::SAMPLE_TABLE . '.code1', $codes, 'OR')
にするとほしい結果が取れるが、deleteしたレコード取得はしたくないため、
whereの文を消せない。
補足情報（FW/ツールのバージョンなど）
Laravel / Mysql
回答3件
0
ベストアンサー
まずはこうしてください。
投稿2020/01/30 13:01
総合スコア114839
create table tbl(id int,num enum('1','2','3') not null,code int null,unique key(id,num));
insert into tbl values
(1,'1',9),
(1,'2',0),
(1,'3',9),
(2,'1',9),
(2,'2',1),
(2,'3',null),
(3,'1',9),
(3,'2',0),
(3,'3',null);
select id
,case when sum((code is null)*(num=1))=1 then null else sum(code*(num=1)) end as code1
,case when sum((code is null)*(num=2))=1 then null else sum(code*(num=2)) end as code2
,case when sum((code is null)*(num=3))=1 then null else sum(code*(num=3)) end as code3
from tbl
group by id;
で、codeが(1,2,3)のどれかの場合
select distinct id from tbl where code in (1,2,3);
こちらのコードをsqlにて実行し新しくテーブル作成しましたが検索対象フィールドが足りないため、
(code1,code2,code3でなくcodeだけである)
私の求めている結果とは異なってしまいます。。。
> 検索対象フィールドが足りないため
ちょっと意味不明
ちゃんとコピペして実行していただきましたか？
code1～3が正しく表示されますよね？
>私の求めている結果とは異なってしまいます。。。
当初命題にあったとおり、「2」が返ってくるとおもいます
途中で質問内容書き換えたと思いますが、それでも
ほしい結果は得られると思いますが？
>ちょっと意味不明
ちゃんとコピペして実行していただきましたか？
code1～3が正しく表示されますよね？
phpmyadminで実行していたため他画面に遷移すると、
フィールドにid,num,codeと出力されていたため勘違いしていました...
今実行して遷移せずにそのままの画面にて
2コメ目のコードを実行したらid=2が取れました!
しかし私の条件が足りなくこのコードに追加で、
is_deleted=falseである条件も追加しなくてはなりません...。
テーブル構造もただいま修正いたしました。
>is_deleted=falseである条件も追加
そうなると正規化の問題なのでマスターテーブルが必要になります
サンプルでいうと
sample_id,is_deleted
1,false
2,false
3,false
4,false
というsample_idを主キー（もしくはユニークキー）にした
テーブルで管理し、codeは別テーブルで管理をします
なぜ、こんな面倒なことをするかというと
code１-3を、同等の検索条件で検索する方針が示されているからです
効率的な検索をするには結果を想定したテーブルの設計が必要になります
逆に冗長なテーブルをつくればそれなりの検索しかできないということです
やはりテーブル設計が悪いですよね....
この場合だと一度CONCAT_WSを使用してcodesという新しいフィールドを作り、
カンマ区切りにして、そのあとforeachなど回して検索かける方法しかないですよね...。
いえ、効率が悪くていいなら
code1 in (1,2,3) or code2 in(1,2,3) or code3 in(1,2,3)
のような書き方で対処できます
DB設計者に言ってはいるんですけどどうしてもtable増やしたくないの一点張りでして...
たしかにその場合だとほしいものが取得できるのですが、
is_deleted = falseの条件を付けたとたん全件取得になってしまいます。
orとandを誤解しているのでは？
where
(code1 in (1,2,3) or code2 in(1,2,3) or code3 in(1,2,3))
and is_deleted = false
「どうしてもtable増やしたくない」は一番最初に条件提示することです
もしそうなら、無駄な提案はしないですみました
yambejpさんのおかげで気づけました。
私の書き方が間違っていてcode1にもorを付けていたため全件になった感じです。
クエリビルダに慣れすぎてしまっているため、sql文をしっかり勉強しようと思います。
次回の質問から気を付けたいと思います
アドバイスに感謝いたします。成長につながります。
0
解決後のソースコード
php
1 $sample_query = Sample::query(); 2 $display = self::displayResultData(); 3 $mansion_query->where(self::SAMPLE_TABLE . '.is_deleted', false) 4 ->whereIn(self::SAMPLE_TABLE . '.code1', $codes) 5 ->whereIn(self::SAMPLE_TABLE . '.code2', $codes, 'OR') // 'OR'の追記 6 ->whereIn(self::SAMPLE_TABLE . '.code3', $codes, 'OR') // 'OR'の追記 7 ->select($display); 8 9 $result = $sample_query->get(); 10
一番最初のwhereIn(self::SAMPLE_TABLE . '.code1', $codes)に対しても'OR'を付けたため、
思った動きになりませんでした。
投稿2020/01/31 04:06
総合スコア12
0
設計が微妙
code1/code2/code3
は第三正規化されていません。
mansionsテーブルとmantion_codesテーブルに分けで
$mantion_ids = MansionCode::whereIn('code', $codes)->get(['mansion_id'])->pluck('mansion_id') Mansion::whereIn('id', mantion_ids)->get()
とした方が効率的です
投稿2020/01/30 09:00
総合スコア5036
あなたの回答
tips
プレビュー
