条件付きチェーンスコープを設定する最良の方法

Question

私のサーバサイドデータテーブルの機能を拡張しようとしています。私はいくつかの余分なフィルターを私のコントローラー/データテーブルに渡します。これを結果のフィルターに使用します。現在のところ、私のモデルでは、スコープを適用する前にパラメータが存在するかどうかをテストしていますが、フィルタのリストが大きくなるif/elseシナリオが多いので、これが最良の方法であるとは確信していません。これを「レールの道」としてどうすればいいですか？

if params[:store_id].present? && params[:status].present? 
    Order.store(params[:store_id]).status(params[:status]) 
elsif params[:store_id].present? && !params[:status].present? 
    Order.store(params[:store_id]) 
elsif !params[:store_id].present? && params[:status].present? 
    Order.status(params[:status]) 
else 
    Order.joins(:store).all 
end 

ANSWER：この作業コードに 組み合わせの答え：

query = Order.all 
query = query.store(params[:store_id]) if params[:store_id].present? 
query = query.status(params[:status]) if params[:status].present? 
query.includes(:store) 

Answer 1

関係が連鎖可能なので、それはあなたの検索クエリを「構築」することはしばしば便利です。これを行うための正確なパターンは大きく異なり、オーバーエンジニアリングには注意が必要ですが、私が取り組んだ大規模なRailsコードベースでは、普通のRubyオブジェクト（PORO）を使用してクエリを構築しています。あなたのケースでは、あなたはおそらくちょうどそうのようなあなたのロジックを簡素化して逃げることができます：

relation = Order.join(:store) 

if params[:store_id] 
    relation = relation.store(params[:store_id]) 
end 

if params[:status] 
    relation = relation.status(params[:status]) 
end 

@orders = relation.all 

Railsのも、あなたのニーズが特に複雑になる場合には、ways to "undo" logic that has been chained previously提供します。

Answer 2

あなたはこのようにそれを行うことができます。また

query = Order 
query = query.store(params[:store_id]) if params[:store_id].present? 
query = query.status(params[:status]) if params[:status].present? 
query = Order.joins(:store) if query == Order 

、あなたもすぐ内側の条件を含めることstatusとstoreスコープを再構築できます。

scope :by_status, -> status { where(status: status) if status.present? } 

その後、あなたの代わりにこれを行うことができます：

query = Order.store(params[:store_id]).by_status(params[:status]) 
query = Order.joins(:store) unless (params.keys & [:status, :store_id]).present?