ruby-on-rails - Rails 控制器重构

Question

我有类似于以下的代码

...
@c = M.find(params[:id]).c.find(params[:c_id])
if @c.s.count > 0
  @s = @c.s.sort_by{|e| e[:order]}.first
  unless @s.p.nil?
    @img = @s.p.image_file.remote_url
  else
    @s.p = P.new
    @img = request.protocol + request.host_with_port + "/none.png"  
  end
  unless @s.a.nil?
    @voice = @s.a.audio_file.remote_url
  else
    @s.a = A.new
  end
else
 ...  
end
@c_v_url = ""
unless @c_v_url.nil?
  @c_v_url = @c.v_o.a_file.remote_url
else
  @c.v_o = A.new
end  
@c_m_url = ""
unless @c_m_url.nil?
  @c_m_url = @c.b_m.a_file.remote_url
else
  @c.b_m = A.new
end
...

现在所有实例变量都将在视图中使用，我想重构代码以使控制器变瘦。进行重构的最佳方法是什么？将此代码移动到模型中是否明智？

Answer 1

我真的看不出这段代码的用途，但它看起来像是显示图像、文件和音频链接的视图逻辑？

我会为每个视图创建一个辅助方法，例如：

 def s_image_url(s)
   unless s.p.nil?
     s.p.image_file.remote_url
   else
     request.protocol + request.host_with_port + "/none.png"  
   end
 end

有关查看助手的更多信息

Answer 2

我会使用演示者模式，这里有一些解释资源（还有很多）：

1. http://blog.jayfields.com/2007/03/rails-presenter-pattern.html
2. http://kpumuk.info/ruby-on-rails/simplifying-your-ruby-on-rails-code

短篇小说：您将检索模型的所有逻辑都放在演示者中。Presenter 易于测试且可扩展。在您的控制器动作中将只有一行代码来实例化演示者。