Rails内の複数のクエリ条件 - 存在する場合。

Question

ここで検索しているときに、私はいくつかの同様の質問を見つけましたが、私は私が見つけた解決策にしない限り、追加しようとしたとき、物事が壊れ始めた...

は、ここで私が持っているものです動作する：

コントローラ：

@metrics = Metric.where("current_ratio > ?", @screen.current_ratio_min) unless @screen.current_ratio_min.nil? 

私はエラーを取得する

@metrics = Metric.where("current_ratio > ?", @screen.current_ratio_min) unless @screen.current_ratio_min.nil? 
    .where("current_ratio < ?", @screen.current_ratio_max) unless @screen.current_ratio_max.nil? 

、（私は多くを追加する必要があるの）別の.where行を追加すると：

undefined method `where' for false:FalseClass 

これは、最初のクエリが終了しない限り、これは仮定しています。個々の状態だけでなく、どのように私を応募すればよいですか？それが実際問題であれば、\

ありがとうございます！

Answer 1

@metrics = Metric.all 
@metrics = @metrics.where('current_ratio > ?', @screen.current_ration_min) if @screen.current_ratio_min.present? 
@metrics = @metrics.where('other_value > ?', @screen.other_value) if @screen.other_value.present? 

これは私がプログラムでSQLインジェクションのために危険なことができWHERE句の文字列を構築することなく、考えることができる最高の方法です。

必要な条件を追加してください。注目すべきは、もし何か？あなたの代わりに何かをしないでください。

また、Metric.allは理想的ではないかもしれませんが、すべてのレコードを開始するために必要なものは何でも構いません。

Answer 2

このようなものはどうですか？

if !@screen.current_ratio_min.nil? && !@screen.current_ratio_max.nil? 
    @metrics = Metric.where("current_ratio > ?", @screen.current_ratio_min).where("current_ratio < ?", @screen.current_ratio_max) 
elsif @screen.current_ratio_min.nil? && !@screen.current_ratio_max.nil? 
    @metrics = Metric.where("current_ratio < ?", @screen.current_ratio_max) 
elsif !@screen.current_ratio_min.nil? && @screen.current_ratio_max.nil? 
    @metrics = Metric.where("current_ratio > ?", @screen.current_ratio_min) 
else 
    @metrics = Metric.all 
end 

Answer 3

metric.rbでscope

を使用し、次のコード

if @screen.current_ratio_min and @screen.current_ratio_max 
    @metrics = Metric.where("current_ratio > ? and current_ratio < ?", @screen.current_ratio_min, @screen.current_ratio_max) 
else 
unless @screen.current_ratio_min.blank? 
    @metrics = Metric.where("current_ratio > ?", @screen.current_ratio_min) 
else 
    unless @screen.current_ratio_max.blank? 
    @metrics = Metric.where("current_ratio < ?", @screen.current_ratio_max) 
    else 
    @metrics = Metric.all 
    end 
end 
end 

Answer 4

あなたはクリーンなコードが必要な場合にお試しください

scope :current_ratio_min, lambda {|current_ratio_min| 
    current_ratio_min.present? ? where('current_ratio > ?', current_ration_min) : where()} 
    scope :current_ratio_max, lambda {|current_ratio_max| 
    current_ratio_max.present? ? where('current_ratio > ?', current_ratio_max) : where()} 

あなたのクエリ：

@metrics = Metric.current_ratio_min(@screen.current_ratio_min).current_ratio_max(@screen.current_ratio_max)` 

Answer 5

conditions = [] 
conditions.add_condition!(["current_ratio > ?", @screen.current_ratio_min]) unless @screen.current_ratio_min.nil? 
conditions.add_condition!(["current_ratio < ?", @screen.current_ratio_max]) unless @screen.current_ratio_max.nil? 
@metrics = Metric.where(conditions) 

これは、AND/ORの組み合わせ

で複数の条件を構築する上で参考になりますが、次のようにあなたはActiveRecordのためのあなたの条件をビルドする場所や条件を見つけることができます

class Array 
    def add_condition!(condition, conjunction = 'AND') 
    if String === condition 
    add_condition!([condition]) 
    elsif Hash === condition 
    add_condition!([condition.keys.map { |attr| "#{attr}=?" }.join(' AND ')] + condition.values) 
    elsif Array === condition 
    unless condition.empty? 
     self[0] = "(#{self[0]}) #{conjunction} (#{condition.shift})" unless empty? 
     self.push(*condition) 
    end 
    else 
    raise "don't know how to handle this condition type" 
    end 
    self 
end 
end 

あなたArrayクラスに次のメソッドを書きます