如何（大量）减少 Rails 应用程序中 SQL 查询的数量？

在我的 Rails 应用程序中我有users其中可以有很多invoices反过来又可以有很多payments.

现在在dashboard我想总结一下所有的观点payments a user已收到，按年、季度或月订购。这payments也分为gross, net, and tax.

user.rb:

class User < ActiveRecord::Base

  has_many  :invoices
  has_many  :payments

  def years
    (first_year..current_year).to_a.reverse
  end

  def year_ranges
    years.map { |y| Date.new(y,1,1)..Date.new(y,-1,-1) }
  end

  def quarter_ranges
    ...
  end

  def month_ranges
    ...
  end

  def revenue_between(range, kind)
    payments_with_invoice ||= payments.includes(:invoice => :items).all
    payments_with_invoice.select { |x| range.cover? x.date }.sum(&:"#{kind}_amount") 
  end

end

发票.rb:

class Invoice < ActiveRecord::Base

  belongs_to :user
  has_many :items
  has_many :payments

  def total
    items.sum(&:total)
  end

  def subtotal
    items.sum(&:subtotal)
  end

  def total_tax
    items.sum(&:total_tax)
  end

end

付款.rb:

class Payment < ActiveRecord::Base

  belongs_to :user
  belongs_to :invoice  

  def percent_of_invoice_total
    (100 / (invoice.total / amount.to_d)).abs.round(2)
  end

  def net_amount
    invoice.subtotal * percent_of_invoice_total / 100
  end  

  def taxable_amount
    invoice.total_tax * percent_of_invoice_total / 100
  end

  def gross_amount
    invoice.total * percent_of_invoice_total / 100
  end

end

仪表板控制器:

class DashboardsController < ApplicationController

  def index    
    if %w[year quarter month].include?(params[:by])   
      range = params[:by]
    else
      range = "year"
    end
    @ranges = @user.send("#{range}_ranges")
  end

end

索引.html.erb:

<% @ranges.each do |range| %>

  <%= render :partial => 'range', :object => range %>

<% end %>

_range.html.erb:

<%= @user.revenue_between(range, :gross) %>
<%= @user.revenue_between(range, :taxable) %>
<%= @user.revenue_between(range, :net) %>

现在的问题是这种方法有效，但也会产生大量 SQL 查询。在一个典型的dashboard我得到的视图100+SQL 查询。添加前.includes(:invoice)还有更多的询问。

我认为主要问题之一是每张发票的subtotal, total_tax and total不存储在数据库中的任何位置，而是根据每个请求进行计算。

有人能告诉我如何加快这里的速度吗？我不太熟悉 SQL 和 ActiveRecord 的内部工作原理，所以这可能是这里的问题。

谢谢你的帮助。

每当revenue_between被调用，它获取payments在给定的时间范围和相关的invoices and items来自数据库。由于时间范围有很多重叠（月、季度、年），因此会一遍又一遍地获取相同的记录。

我认为最好一次获取用户的所有付款，然后在 Ruby 中过滤和汇总它们。

要实施，请更改revenue_between方法如下：

def revenue_between(range, kind) 
   #store the all the payments as instance variable to avoid duplicate queries
   @payments_with_invoice ||= payments.includes(:invoice => :items).all
   @payments_with_invoice.select{|x| range.cover? x.created_at}.sum(&:"#{kind}_amount") 
end

这将急切地加载所有付款以及关联的发票和项目。

还要更改invoice求和方法，以便它使用急切加载items

class Invoice < ActiveRecord::Base

   def total
     items.map(&:total).sum
   end

   def subtotal
     items.map(&:subtotal).sum
   end

   def total_tax
     items.map(&:total_tax).sum
   end

end