这段代码感觉不对。我试图让我的控制器变瘦,我觉得控制器动作中的逻辑太多了。
组织此代码的更好方法是什么?
def search
where_obj = {:status => 1}
if params[:city].present?
where_obj.merge! :city => params[:city]
end
if params[:county].present?
where_obj.merge! :county => params[:county]
end
## THERE WILL BE MANY MORE IF STATEMENTS HERE DUE TO GROWING SEARCH FORM
@person = Person.where(where_obj)
end
随着搜索表单的增长,此控制器操作也会增长。我怎样才能让我的控制器变瘦?