次のようなコードがあります
...
@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
...
これで、すべてのインスタンス変数がビューで使用されるようになりました。コードをリファクタリングしてコントローラーを細くしたいと思います。リファクタリングを行うための最良のアプローチは何ですか? このコードをモデルに移動するのは賢明でしょうか?