在数组中构建 Active Record 条件 - 私有方法“scan”称为错误

Question

我正在尝试使用数组动态构建一组条件，如第一个答案中所建议的： 模型中的一个或多个参数使用 Ruby on Rails 查找条件。然而，我似乎做错了一些事情，我不确定我正在尝试的事情是否从根本上不合理，或者我只是搞砸了我的语法。

我在这里简化为一个条件来尝试说明问题，因为我尝试在对我正在处理的 5 种不同条件样式进行分层之前沿着这些思路构建一个简单的概念证明。

这有效：

excluded.push 12
excluded.push 30
@allsites = Site.all(:conditions => ["id not in (?)", excluded])

这会导致私有方法“扫描”，称为错误：

conditionsSet = []
excluded.push 12
excluded.push 30
conditionsSet << ["id not in (?)", excluded]
@allsites = Site.all(:conditions => conditionsSet)

感谢您的任何建议。我不确定将其作为我在顶部指出的相关问题/答案的后续项目是否正确。因为我有一个问题没有答案。如果有更好的方法来发布与现有帖子相关的内容，请告诉我。

Answer 1

试试这个：

Rails 2.3

class Site < ActiveRecord::Base

  def self.build_conditions(ids, name=nil, state=nil)
     cond = []
     cond << send(:sanitize_sql_array, ["id NOT IN (?)", ids]) unless ids.empty?
     cond << send(:sanitize_sql_array, ["name = ? ", name]) unless name
     cond << send(:sanitize_sql_array, ["state = ? ", state]) unless state
     cond.join(" and ")
  end    
end

现在在你的控制器中的某个地方：

Site.all(:conditions => Site.build_conditions([1,2])) 
Site.all(:conditions => Site.build_conditions(nil, "ABC"))

Rails 3

class Site < ActiveRecord::Base          
  def self.exclude_ids_by_name_and_state(ids, name=nil, state=nil)
    result = scoped
    result = result.where("id NOT IN (?)", ids) if ids.present?
    result = result.where(:name => name) if name.present?
    result = result.where(:state => state) if state.present?
    result
  end    
end

现在在你的控制器中的某个地方：

Site.exclude_ids_by_name_and_state([1,2])).all 
Site.exclude_ids_by_name_and_state(nil, "ABC").all

Answer 2

您需要：

conditionsSet += ["id not in (?)", excepted]

而不是：

conditionsSet << ["id not in (?)", excepted]

+= 将两个数组添加在一起（将其视为将两个数组合并为一个数组），而 << 将新元素推入数组。因此，当使用 << 和 :conditions 时，您会得到： [["id not in (?)", excepted]]想要一个数组，其中第一个元素是字符串（而不是数组）。

Answer 3

尝试 SmartTuple，它是专门为此类情况设计的。

def self.build_conditions(options = {})
  [
    SmartTuple.new(" AND "),
    (["id NOT IN (?)", ids] if options[:ids].present?),
    ({:name => options[:name]} if options[:name].present?),
    ({:state => options[:state]} if options[:state].present?),
  ].sum.compile
end

...

Site.all(:conditions => Site.build_conditions(:ids => {1,2]))
Site.all(:conditions => Site.build_conditions(:name => "abc", :state => "disabled")

对我来说，最好使用 options 哈希而不是有序参数。随着项目的发展，可能会出现更多的情况，而您将不知道哪些情况会出现。哈希看起来和行为都更清晰，而且您可以轻松验证它以避免软件错误。