Refactor: Plusieurs params dans find query

Question

Besoin d'aide pour refactoriser ce bloc if/else qui construit les conditions d'une requête de recherche.

if params[:status] && params[:carrier] 
    conditions = ["actual_delivery IS NOT NULL AND actual_delivery > scheduled_delivery AND status_id = ? AND carrier_id = ?", status.id, carrier.id] 
elsif params[:status] 
    conditions = ["actual_delivery IS NOT NULL AND actual_delivery > scheduled_delivery AND status_id = ?", status.id] 
elsif params[:carrier] 
    conditions = ["actual_delivery IS NOT NULL AND actual_delivery > scheduled_delivery AND carrier_id = ?", carrier.id] 
else 
    conditions = ["actual_delivery IS NOT NULL AND actual_delivery > scheduled_delivery"] 
end 

@packages = Package.find(:all, :conditions => conditions) 

Answer 1

Vous pouvez faire:

conditions = "actual_delivery IS NOT NULL AND actual_delivery > scheduled_delivery" 
conditions += " AND status_id = #{status.id}" if params[:status] 
conditions += " AND carrier_id = #{carrier.id}" if params[:carrier] 

@packages = Package.all(:conditions => [conditions]) 

Answer 2

Je recommande la création d'un champ dans votre modèle, de prendre soin de la première partie de votre requête qui est toujours le même dans cette action:

class Package < ActiveRecord::Base 
    named_scope :late_deliveries, :conditions => "actual_delivery IS NOT NULL AND actual_delivery > scheduled_delivery" 
end 

maintenant vous pouvez factoriser votre action comme ceci:

def index 
    conditions = {} 
    [:status, :carrer].each{|param| conditions[param] = params[param] if params[param]} 

    @packages = Package.late_deliveries.find(:conditions => conditions) 
end 

Si: transporteur et: état sont les deux seuls paramètres à cette action, il est encore plus simple:

def index 
    @packages = Package.late_deliveries.find(:conditions => params) 
end 

J'espère que cela aide!