Combinaison de conditions dans find. (Rails)

Question

Maintenant j'entrer quelques données d'un formulaire et j'ai un code pour rechercher la base de données en entrant plusieurs paramètres comme conditions d'entrée. Maintenant, si l'un des paramètres est nul (c'est-à-dire que le champ n'est pas coché, j'ai besoin de remplacer ce paramètre par quelque chose disons * afin que la requête de recherche ne soit pas affectée. Comment ferais-je cela?

@report = Problem.find(:all, :conditions => ["problems.cause_id = ? and problems.location_id = ? and problems.device_id = ? and problems.priority_id = ?", Report.find_by_id(params[:id]).cause_id, Report.find_by_id(params[:id]).location_id, Report.find_by_id(params[:id]).device_id, Report.find_by_id(params[:id]).priority_id]) 

Answer 1

Il serait préférable de ne pas avoir cette condition du tout que d'utiliser *. Dans ce cas, c'est simple car tous vos opérateurs de comparaison sont "=". Cela signifie que vous pouvez utiliser la forme de hachage des conditions. Votre code est également très inefficace lorsque vous chargez le même objet de rapport 3 ou quatre fois. Votre question à propos de l'un des params étant nulle n'a pas de sens pour cette raison: vous utilisez simplement le même param encore et encore. De même, vous définissez une variable appelée @report comme étant un objet Problème qui prête à confusion.

@report = Report.find_by_id(params[:id]) 
conditions = {:cause_id => @report.cause_id, :location_id => @report.location_id, :device_id => @report.device_id, :priority_id => @report.priority_id} 
conditions.delete_if{|k,v| v.blank?} 
@problem = Problem.find(:all, :conditions => conditions) 

Answer 2

rep = Report.find_by_id(params[:id]) 
cause = rep.cause_id ? rep.cause_id : '*' 
location = rep.location_id ? rep.location_id : '*' 
device = rep.device_id ? rep.device_id : '*' 
priority = rep.priority_id ? rep.priority_id : '*' 
@report = Problem.find(:all, 
         :conditions => ["problems.cause_id = ? and 
             problems.location_id = ? and 
             problems.device_id = ? and 
             problems.priority_id = ?", 
             cause, location, 
             device, priority 
             ] 
         )