[Laravel]GETによる検索条件絞り込みのURL生成方法がわかりません

Question

Laravelで検索結果一覧の並び替え、絞り込みを実装しました。

商品一覧を表示するページで、GETパラメータは３つです。

・ order 並び順 (価格の安い順、価格の高い順、発売日 など)
・ kind「商品の種類」
・ used「中古かどうか」

(例) 商品の種類が3で、かつ中古の商品を、価格の安い順に並びて表示

?kind=3&used=true&order=price_low

※ 実際はキーワード検索もあるので keyword というパラメータも存在
※ ページネーションしているので page パラメータも存在

発生している問題
(例)の状態から「価格の高い順」をクリックすると以下のようにorderが重複してしまう。
?order=price_low&kind=3&used=true&order=price_high

つまり、クリックされた条件のGETパラメータを解除できていません。

検索条件が一つの場合は以下のようにするだけです。

<a href="{{ url('/?order=price_low') }}">価格の安い順</a>

しかしパラメータが複数ある場合、選択済みの条件を保ったままパラメータを付与する必要があります。

求める動き

order=price_low を表示している状態で「価格の高い順」のリンク生成する場合
↓
order=price_low (とpage)をリセットした上で、他のパラメータは引き継ぎ、
最後に order=price_high を付与

「クリックされた条件のパラメータ」と「page」のみをリセットすることができません。

コントローラ

public function search(Request $request)
{
     // 存在し得る全パラメータ
     $params = $request->only('order', 'kind', 'used', 'keyword', 'page');

     // $remains 引き継ぐパラメータ  
     // 以下が問題　「クリックされた条件」を除外できない      
     // 除外するのは「クリックされた条件」と「page」のみ
     $remains = $request->only('order', 'kind', 'used', 'keyword');
     
     // 商品を絞り込み検索
     $goods = Good::search($params)->paginate(30);

     return view('list', compact('goods', 'params', 'remains'));
}

ビュー

価格の安い順 | 価格の高い順 | 発売日

↑「価格の安い順」の表示部分だけ以下に示しています。

// 現在「価格の安い順」を表示している場合はリンクにしない
@if( isset($params['order']) && $params['order'] === 'price_low') )
      <span>価格の安い順</span>

//現在「価格の安い順」ではない場合
@else 
      // 引き継ぐパラメータ 無
      @if(empty($remains))
         <a href="{{ url('/?order=price_low') }}">価格の安い順</a>
      // 引き継ぐパラメータ 有 
      @else
         <a href="{{ url('/?'.http_build_query($remains).'&order=price_low') }}">価格の安い順</a>
      @endif
@endif

URLの生成方法として、クリックした条件を解除して新たにパラメータを付与 という方法が良くないのでしょうか？

どのような仕組みでURLを生成すべきなのでしょうか。

ご教示いただけると幸いです。どうそよろしくお願いいたします。

Answer 1

ルート定義で、urlの/にtopなどの名前をつけて、routeヘルパーを使うと、第二引数でルートパラメータでない場合は、クエリストリングになります。

route("top",["order"=>"price_low"])

Laravel route url with query string

記法がすっきりするので、これで必要な配列変数を整理すれば、問題解決しやすいかと思います。

Comments

[akomit]

url() より route() を使用した方がスマートですね。

route() の第二引数配列は、ルートの {} で指定したパラメータ以外であれば、GETパラメータになるのですね。

ありがとうございました。

Answer 2

(例)の状態から「価格の高い順」をクリックすると以下のようにorderが重複してしまう。
?order=price_low&kind=3&used=true&order=price_high

http_build_queryに渡す引数の配列自体に正しいorderの値を入れた方が簡潔（後に文字列で連結するよりも）だと思うので、マージした配列を渡してはどうですか？

<a href="{{ url('/?'.http_build_query(array_merge($remains, ['order' => 'price_low']))) }}">価格の安い順</a>

これであれば、$remainsの中身自体は変えずに、orderを上書き（なければ追加）できます。

※array_mergeは、配列のキーが「数字」の時と「文字列」の時で異なる挙動をしますので、マニュアルをよく確認してから使ってくださいね。。！

Comments

[akomit]

実現できました！ありがとうございます。

array_merge は同じキー文字列がある場合、後に指定された値で上書きされる仕様なのでピッタリですね！

$remains は $request->only() で存在しうる全てのパラメータを指定することで、リクエストされたパラメータのみを取得することができますね。

そして array_merge すれば、クリックされたパラメータが無ければ普通に追加し、有れば上書きされますね。

ありがとうございました。

Answer 3

http_build_query($remains)

ここで重複しているだけなので、パラメーター自体存在して困るものでもないなら
普通に?order=xxx&kind=xxx&used=xxx&keyword=xxxと書いてxxxの部分にそれぞれ変数を当て込むか、直接price_lowなどを書く

Comments

[akomit]

確かに存在しうる全てのパラメータを用意して、無いものは空文字を当てれば問題なさそうですね。

ありがとうございます。