导轨 - 将控制器逻辑转移到模型中

Question

我正在寻找清理我的控制器，因为它看起来很重且很多。不胜感激任何关于如何将这种类型的逻辑转移到我的模型中的帮助。感谢有这方面的帮助 - 下面的代码是我的索引操作：

case params[:find_by] 
    when 'topic' 
    nuggets = Nugget.where(['topic = ?', params[:topic_name]]) 
    @nuggets = nuggets.paginate(:page => params[:page],:per_page => 15) 
    @title = nuggets.first.topic 
    when 'audience' 
    nuggets = Nugget.where(['audience = ?', params[:audience_name]]) 
    @nuggets = nuggets.paginate(:page => params[:page], :per_page => 15) 
    @title = nuggets.first.audience 
    else 
    @nuggets = Nugget.paginate(:page => params[:page], :per_page => 15) 
end 

Answer 1

我不能完全肯定我会移动它到模型中。我可能只是将它移入控制器中的私有实用程序方法。

case params[:find_by] 
    when 'topic' 
    nuggets = Nugget.find_by_topic(params[:topic_name]) 
    @title = nuggets.first.topic 
    when 'audience' 
    nuggets = Nugget.find_by_audience(params[:audience_name]) 
    @title = nuggets.first.audience 
    else 
    nuggets = Nugget.all 
end 

@nuggets = Nugget.paginate(:page => params[:page], :per_page => 15) 

另一种选择是为不同的发现创建路由;不管它是否值得，嗯。你可能将find_by逻辑移动到模型中，或者使用send稍微干掉话题/观众差异，但是再一次，这看起来比它的价值更麻烦。尽管如此，我仍然有兴趣了解更多Rails-y人对这个问题的看法。

Answer 2

添加到您的模型

def self.topic(topic_name) 
    where(:topic => topic_name) 
end 

def self.audience(audience_name) 
    where(:audience => audience_name) 
end 

而在控制器与此

def index 
    if params[:find_by] 
     nuggets = Nugget.send(params[:find_by].to_sym)(params[:name]) 
     @nuggets = nuggets.paginate(:page => params[:page], :per_page => 15) 
    else 
     @nuggets = Nugget.paginate(:page => params[:page], :per_page => 15) 
    end 
    @title = @nuggets.first.send(params[:find_by].to_sym) 
end 

取代你只需要使用相同的参数键观众和主题