Rails：这段代码属于哪里？

Question

我现在有在我的控制器如下：

@items = Item.scoped 
@items = @items.where('price >= ?', params[:price_min]) if params[:price_min] 
@items = @items.where('price <= ?', params[:price_max]) if params[:price_max] 
@items = @items.where(:size => params[:size]) if params[:size] 
@items = @items.where(:colour => params[:colour]) if params[:colour] 
# ... 
@items = @items.paginate(:page => params[:page], :per_page => 10) 

这是此代码正确的地方，还是应该真正属于我的模型与控制器一个方法调用？即

@items = Item.apply_filters(params) 

我试图尽可能地坚持约定。

非常感谢。

Answer 1

你是对的，这一切都属于你的模型。这与我前几天查看的某个人的代码非常相似。 MetaWhere创业板可能很适合您的项目。

https://codereview.stackexchange.com/questions/500/how-to-filter-search-a-model-with-multiple-params-in-hash/501#501

Answer 2

另外，如果您的项目总是会先限定，你可以说：default_scope。在你的模型：

default_scope order("item_number"), where('price >= ?', 100) 

（我不能完全肯定我得到了所有的语法正确的，但它是类似的东西。）

named_scope也可以帮助你。