laravelで複数検索機能を作りたい

Question

ユーザーテーブル

id	name	prefecture

商品テーブル
|id|product_name|price|expiration|user_id|
|:--|:--:|--:|
||||

ある商品を、1,出品したユーザーが住んでいる都道府県、2,商品の賞味期限、3,商品の価格3つの条件で一覧画面から検索できる機能を作っています。if文で各々requestに値が入っていたら条件に合った値を取得しています。しかし都道府県で検索した時、レコードがnullだった時にエラーが返ってきてしまうのを、もしnullならそのまま次の処理に走らせたいのですが、いまいちうまくいきません。また、if文の中で毎回retuenでviewに値を返さないと表示されないのですが、もっといい方法があれば教えていただきたいです。また、もし2つの条件、3つの条件で同時に検索された時、それに合う値を表示するには、また別に処理を書かないといけないのでしょうか。初歩的なことではありますがアドバイスをいただきたいです。

ProductController.php

public function index(Request $request) {
   // Log::debug(print_r($order, true));

//検索された値を取得
    $pref = $request->input('prefecture');
    $price_from = $request->input('price_from');
    $price_until = $request->input('price_until');
    $expiration = $request->input('expiration');
    $prefs = config('prefectures');
    $users = User::where('prefecture', $pref)->first();
    //単一のプロパティはgetではなくfirstとります。

//都道府県での検索
if (isset($pref) ) {
    $users = User::where('prefecture', $pref)->first();
    $products = $users->products()->paginate(10);
            
    return view('products.index', [
        'products' => $products,
        'prefs' => $prefs,
        ]);
}


//賞味期限での検索
    if (isset($expiration)  && $expiration === '切れていない'){
    //今日の日づけを取得    
    $today = Carbon::today();
    //今日の日づけより後の日付けのproductを取得
    $products = Product::whereDate('expiration_date','>', $today )
    ->paginate(10); 
    return view('products.index', [
        'products' => $products,
        'prefs' => $prefs,
        ]);
    }elseif(isset($expiration ) && $expiration === '切れている' ) {
    $today = Carbon::today();
    $products = Product::whereDate('expiration_date','<',  $today )
    ->paginate(10); 
    return view('products.index', [
        'products' => $products,
        'prefs' => $prefs,
        //'order' =>$order
        ]);
    }else {
    $products = Product::latest()->paginate(10);
    }
    //価格での検索
    if (isset($price_from) && isset($price_until)) {
        $products = Product::whereBetween('price', [$price_from,$price_until])
        ->paginate(10); 
        }else {
        $products= Product::latest()->paginate(10);
        }

    return view('products.index', [
        'products' => $products,
        'prefs' => $prefs,
        ]);
    }

index.blade.php

<section class="l-inner-wrapper__s u-bgcolor__accent" id="Salon">
  <div class="l-inner__s ">
    <div class="c-inner--title"></div>
     <div class="l-inner--body">
       <h2 class="p-concept--title">詳細検索</h2>
         <div class="p-login">
           <form method="GET" action=""class="c-form p-form__login">
            <ul class="p-form--list">
              <li class="p-form--item">
               <input type="number" placeholder="価格" class="c-text" name="price_from" value="{{ old('price_from') }}">
             </li>
              <li class="p-form--item">
                 円〜
             </li>
             <li class="p-form--item">
               <input type="number" placeholder="" class="c-text" name="price_until" value="{{ old('price_until')}}">
             </li>
             <li class="p-form--item">
                 円
             </li>
            </ul>
             <ul class="p-form--list">
              <li class="">
               <li class=" c-text u-border__none">
                 <select name="prefecture" id="" class="">
                  <option value="">コンビニ都道府県</option>
                   @foreach($prefs as $pref )
                    <option value="{{ $pref }}"
                    @if(old('prefecture') === $pref) selected @endif
                    >{{$pref}}</option>
                   @endforeach
                </select>
            </li>
            </li>
            <li class="">
              <li class=" c-text u-border__none">
              <select name="expiration" id="">
                <option value="" >賞味期限</option>
                <option value="切れていない"
                @if(old('expiration')==='切れていない') selected  @endif
                >切れていない
              </option>
                <option value="切れている"
                @if(old('expiration')==='切れている') selected  @endif
                >切れている
              </option>
              </select>
            </li>
            </li>
          </ul>
          <div class="">
           <input type="submit" value="検索する" class="c-btn c-btn--search">
         </div>
      </form>
    </div>
  </div>
  </div>
  </section>

Comments

[hina]

質問内容とは本質的ではないんですが、ユーザーと商品の間にリレーションがあるよう(おそらくproducts belongs to users, users hasMany products)ですが、提示されているテーブルには外部キーを持つカラムが見当たりません……

[ORyohei]

user_idのカラムをつけ忘れていました。実際には存在します。アドバイスありがとうございます。これから試して見ます。

Answer 1

都道府県で検索した時、レコードがnullだった時にエラーが返ってきてしまう

そもそも質問のコードには不可解な点が複数あります。

$users = User::where('prefecture', $pref)->first();

この文は最初のif文内とその前で2回呼ばれているようですが、おそらく後者は意図しているものではないものかと思います。
また、$usersとなっているようにも複数の出品者を取りたいものかと思いますがここで取得されるユーザーは単体またはnullです。「単一のプロパティはgetではなくfirstとります。」とわざわざコメントで書かれているとおりなのですが、ここでほしいのはおそらくは単一ではないでしょう。そして、nullであればこのあとのメソッドチェーンでこけるのは当然です。

しかしまあ、おそらくやりたいことは「条件に一致するユーザーの商品を抽出」でしょうのでこうはしません。では代わりにどうするかというと、リレーションのクエリを使います。whereHasが使えます。

たとえば

    $users = User::where('prefecture', $pref)->first();
    $products = $users->products()->paginate(10);

をリレーションクエリを用いるように置き換えると

Product::whereHas('user', function($query) use ($pref) {
    $query->where('prefecture', $pref);
});

のように書くことができます。

---

複数の検索条件を指定した場合にすべての条件を指定したい場合はどうすればいいでしょうか、クエリをつなげていけばいいでしょう。上の変更はそれへの第一歩ともいえます。詳細なコードは省略しますが、イメージだけ。

$query = new Product();

if (条件1) {
  $query = $query->where(...);
}

if (条件2) {
  $query = $query->where(...);
}
...

$products = $query->paginate(10);
return view(...);

Comments

[ORyohei]

無事解決することができました！コードも綺麗になりクエリーの知識も整理することが出来ました。本当にありがとうございます。