Laravel5.4のクエリビルダーでwhereやorWhereなどを使っての条件抽出について

Laravelのクエリビルダーについて質問させてください。

Laravel5.4のクエリビルダーでwhereやorWhereなどを使っての条件抽出について
うまくいかずに困っています。

記事を表示するサイトに検索の機能を作ろうと思い、
viewの検索窓から$keywordをコントローラーに送り
コントローラーで下記のような形でwhereやorWhereなどを使ってデータの抽出処理をしています。

PHP
1  public function topSearch(Request $request){
2  
3  $keywords = $request->input('keywords');
4  //省略	
5  
6//検索の前にドラフトと掲載前の記事を除外する	
7  $query = Board::query();
8    $query->where(function($query) use($now){
9          $query->where('status', '!=', 'draft')
10          ->where('published_at','<=', $now)
11    });
12
13
14//検索条件で抽出
15   $query->where('title','like','%'.$keywords.'%')->orWhere('description','like','%'.$keywords.'%');
16   $query->orWhere('title','like','%'.$keywords.'%')->orWhere('description','like','%'.$keywords.'%');
17
18
19    $query->orderBy('created_at', 'desc');
20    $boards = $query->paginate(20);
21
22//省略

この際に前段の

PHP
1          $query->where('status', '!=', 'draft')
2          ->where('published_at','<=', $now)

が効いておらず、$boardsの中にドラフトと掲載前の記事も含まれてしまします。
キーワードでの検索の処理の条件の方は効いており抽出されています。

下記のような記述でも結果は同じでした。

PHP
1$query = Board::query();
2$query->where('status', '!==', 'draft');
3$query->where('published_at','<=', $now);

暫定処理で表示したくないコンテンツはviewのbladeの方で

PHP
1@forelse($post_boards as $post_board)
2@if($post_board->status != 'draft')
3@if($post_board->published_at <= $now)
4//省略
5<li>
6  {!! mb_strimwidth( strip_tags( $board->description ), 0, 155, '...', 'UTF-8') !!}
7  <a href="{{ route('boards.show', ['id' => $board->id]) }}">[ReadMore]</a>
8</li>
9//省略
10@endif
11@endif
12@endforelse

というような処理で非表示にしているのですが、
$boardsの抽出のところでできれば制御したいと思っています。

whereやorWhereの使い方の基本的なところが理解できていないのかもしれませんが、
ヒントをいただければ幸いです。

よろしくお願いいたします。

行動規範の内容に同意します

回答2件

ベストアンサー

https://laravel.com/docs/5.5/queries
Parameter Groupingのところで書いているとおり、whereに関数を渡したらその内部をカッコで囲むっていう感じになるので、

php
1$query = Board::query();
2    $query->where(function($query) use($now){
3          $query->where('status', '!=', 'draft')
4          ->where('published_at','<=', $now)
5    });
6
7
8//検索条件で抽出
9   $query->where('title','like','%'.$keywords.'%')->orWhere('description','like','%'.$keywords.'%');
10   $query->orWhere('title','like','%'.$keywords.'%')->orWhere('description','like','%'.$keywords.'%');

だと
where (status != 'draft' and published_at <= '$nowの値')
and title like %{$keywords}%
or description like %{$keywords}%
or titlelike %{$keywords}%
or description like %{$keywords}%

となってしまうのではないでしょうか

php
1$query = Board::query();
2    $query->where(function($query) use($now){
3          $query->where('status', '!=', 'draft')
4          ->where('published_at','<=', $now)
5    });
6
7
8//検索条件で抽出
9$query->where(function($query) use($keywords){
10   $query->where('title','like','%'.$keywords.'%')->orWhere('description','like','%'.$keywords.'%');
11   $query->orWhere('title','like','%'.$keywords.'%')->orWhere('description','like','%'.$keywords.'%');
12});

こうかなっていう感じがしますね。

※QueryBuilderには

php
1$query->toSql();
2$query->getBindings();

っていう関数があるのでどういうSQL文になってるか出力してみるといいかもしれません。

投稿2017/12/14 15:47

rururu3

総合スコア5545

do_slice

2017/12/15 07:36

rururu3様ありがとうございます！教えていただいた通りで想定通りの動きをしました。勉強になりました。ベストアンサーとさせていただきました。

行動規範の内容に同意します

おそらくこのあたりが原因では？

PHP
1   $query->where('title','like','%'.$keywords.'%')->orWhere('description','like','%'.$keywords.'%');
2   $query->orWhere('title','like','%'.$keywords.'%')->orWhere('description','like','%'.$keywords.'%');

ここがやりたいことは「タイトルにキーワードを含む、または説明文にキーワードを含む」だと思うのですが、まず2回もtitleとdescriptionのorWhereが書いてあるのが多分無駄ですし、このorWhereがこの位置だとこれまでのwhereまたはこのorWhereになってしまいます。多分出来上がっているクエリはこんな感じです。

(ドラフトじゃない AND 公開日以降)
   AND タイトルにキーワードを含む
   OR 説明文にキーワードを含む
   OR タイトルにキーワードを含む
   OR 説明文にキーワードを含む

おそらくやりたいことはこうですよね？

(ドラフトじゃない AND 公開日以降)
   AND
   (タイトルにキーワードを含む OR 説明文にキーワードを含む)

だとしたらクエリの書き方はこうなると思います。

PHP
1$query = Board::query();
2$query->where(function($query) use($now){
3  $query->where('status', '!=', 'draft')
4    ->where('published_at','<=', $now);
5});
6
7
8//検索条件で抽出
9$query->where(function($query) {
10  $query->where('title','like','%'.$keywords.'%')
11    ->orWhere('description','like','%'.$keywords.'%');
12});