3

这是我所拥有的

目前,我正在这样做:

current_account.send(object).search(params[:search]).user_id_equals_any(users).visibility_is_any(visibilities)

但那不是很灵活?如果我想有条件地不拥有这些范围之一怎么办?

我认为具有条件添加元素的范围数组将是一个很好的解决方案

我将使用 .send() 调用的参数数组

scopes = []    
scopes = << [:user_id_equals_any, users] if filter_users
    scopes = << [:visibility_is_any, visibilities] if filter_visibility

所以,我可以有一些未知的数字范围。

要执行范围,我需要调用.send(:scope_name, scope_param)

但是我如何为我的scopes阵列中的任意数量的范围做到这一点?

作为一个循环,我认为它会像

result = current_account.send(object).search(params[:search])
scopes.each do |scope|
    result.send(scope[0], scope[1])
end
return result

循环可以简化为

scopes.each {|s| result.send(s[0], s[1]) }

但是有没有办法在一行中附加不同的发送调用?

4

2 回答 2

3

您可以使用inject

scopes = []
# Note that in your example, the assignment is invalid syntax
scopes << [:user_id_equals_any, users] if filter_users
scopes << [:visibility_is_any, visibilities] if filter_visibility

# initial result
result = current_account.send(object).search(params[:search])

# applying the scopes
result = scopes.inject(result) do |result, (method, param)|
  result.send(method, param)
end
return result

在每个循环中,前一个块执行的最后一个返回值被传递给块的第一个参数,在这种情况下是result. 第一个循环获取初始结果(即inject调用方括号中的参数。返回最后一个循环的返回值。

有关更多详细信息,请查看文档

于 2012-06-15T19:38:25.007 回答
1

随意构建自己的抽象,以便您的代码反映您对问题的看法:

class Object
  def send_if(condition, *args, &block)
    condition ? send(*args, &block) : self
  end
end

current_account.send(object).search(params[:search]).
  send_if(filter_user, :user_id_equals_any, users).
  send_if(filter_visibility, :visibility_is_any, visibilities)
于 2012-06-15T20:34:30.360 回答