Ruby on Rails-从视图到模型重构ActiveRecord查询

Question

I am aware of the advice "fat models / skinny controllers" and "never put logic in the view"; however, it would help me to learn from an example. In the following, what is the best way to rewrite the code so that the query is not in the view?

Model

 class Product < ActiveRecord::Base
   belongs_to :order
 end

 class Order < ActiveRecord::Base
   has_many :products
 end

Controller

 @orders = Order.all

View

 <% @orders.each do |o| %>
 <%= Product.where("order_id = ?", o.id).count %>
 <% end %>

Answer 1

It depends on exactly what you want to display, but the straightforward option is to take advantage of the associations you've specified:

<% @orders.each do |o| %>
  <%= o.products.count %>
<% end %>

Then in your controller, you can use eager loading to optimize your SQL calls.

@orders = Order.all(:include => :products)