Rails，如何避免对关联中的总数（计数、大小、counter_cache）进行“N + 1”查询？

Question

我有这些型号：

class Children < ActiveRecord::Base
    has_many :tickets
    has_many :movies, through: :tickets
end


class Movie < ActiveRecord::Base
    has_many :tickets
    has_many :childrens, through: :tickets
    belongs_to :cinema
end


class Ticket < ActiveRecord::Base
    belongs_to :movie, counter_cache: true
    belongs_to :children
end


class Cinema < ActiveRecord::Base
    has_many :movies, dependent: :destroy
    has_many :childrens, through: :movies
end

我现在需要的是在“电影院”的页面中，我想为电影院的电影打印孩子们的总和（计数，大小？），所以我写了这个：

• 在cinemas_controller.rb中：

@childrens = @cinema.childrens.uniq

• 在电影院/show.html.erb：

<% @childrens.each do |children| %><%= children.movies.size %><% end %>

但显然我有子弹 gem 提醒我 Counter_cache 并且我不知道将这个 counter_cache 放在哪里，因为电影的 id 不同。

而且，如果没有 counter_cache，我所拥有的也不是我想要的，因为我想计算那家电影院有多少孩子从那家电影院的票中拿走他们。

怎么做？

更新

如果在我看来我使用此代码：

<% @childrens.each do |children| %>
  <%= children.movies.where(cinema_id: @cinema.id).size %>
<% end %>

gem bullet 什么都没说，一切正常。

但是我有一个问题：这种查询数据库的方式因为视图中的代码比较重？

Answer 1

这可能会对你有所帮助。

@childrens_count = @cinema.childrens.joins(:movies).group("movies.children_id").count.to_a

Answer 2

您可以使用includes 提前加载所有关联。例如：

@childrens = @cinema.childrens.includes(:movies).uniq

这将在控制器中加载所有孩子的电影，防止视图需要访问循环中的数据库。

Answer 3

您可能同意，属于孩子的电影数量等于他们购买的门票数量。 这就是为什么您可以只缓存门票数量并将其显示在电影院#show 上。 你甚至可以创建一个方法让它更清晰。

class Children < ActiveRecord::Base
  has_many :tickets
  has_many :movies, through: :tickets

  def movies_count
    tickets.size
  end
end

class Ticket < ActiveRecord::Base
  belongs_to :movie, counter_cache: true
  belongs_to :children, counter_cache: true
end

class Movie < ActiveRecord::Base
  belongs_to :cinema
  has_many :tickets
  has_many :childrens, through: :tickets
end

class Cinema < ActiveRecord::Base
  has_many :movies, dependent: :destroy
  has_many :childrens, through: :movies
end

然后：

<% @childrens.each do |children| %><%= children.tickets.size %><% end %>

或者

<% @childrens.each do |children| %><%= children.movies_count %><% end %>

但如果你想显示每部电影的票数，你肯定需要考虑以下几点：

@movies = @cinema.movies

然后： <% @movies.each do |movie| %><%= movie.tickets.size %><% end %> 由于您有belongs_to :movie, counter_cache: true，tickets.size 不会进行计数查询。 并且不要忘记添加tickets_count 列。 More about counter_cache...

附：请注意，根据惯例，我们将模型命名为 Child，将关联命名为 Children。

Answer 4

我前段时间写了一个小 ActiveRecord 插件，但还没有机会发布 gem，所以我只是创建了一个 gist：

https://gist.github.com/apauly/38f3e88d8f35b6bcf323

例子：

# The following code will run only two queries - no matter how many childrens there are:
#   1. Fetch the childrens
#   2. Single query to fetch all movie counts
@cinema.childrens.preload_counts(:movies).each do |cinema|
  puts cinema.movies.count
end

---

再解释一下：

已经有类似的解决方案（例如https://github.com/smathieu/preload_counts），但我不喜欢他们的接口/DSL。我一直在寻找类似于活动记录 preload (http://apidock.com/rails/ActiveRecord/QueryMethods/preload) 方法的东西（在语法上），这就是我创建自己的解决方案的原因。

为了避免“正常”的 N+1 查询问题，我总是使用 preload 而不是 joins，因为它运行一个单独的查询并且不会修改我的原始查询，如果查询本身是已经相当复杂了。

Answer 5

其实比剩下的解决方案简单很多

你可以使用lazy loading:

在您的控制器中：

def index
  # or you just add your where conditions here
  @childrens = Children.includes(:movies).all 
 end

在你看来index.hml.erb：

<% @childrens.each do |children| %>
  <%= children.movies.size %>
<% end %>

如果您使用size，上面的代码不会进行任何额外查询，但如果您使用count，您将面临select count(*) n + 1 个查询

Answer 6

在你的情况下，你可以使用这样的东西：

class Ticket < ActiveRecord::Base
    belongs_to :movie, counter_cache: true
    belongs_to :children
end
class Movie < ActiveRecord::Base
    has_many :tickets
    has_many :childrens, through: :tickets
    belongs_to :cinema
end
class Children < ActiveRecord::Base
    has_many :tickets
    has_many :movies, through: :tickets
end
class Cinema < ActiveRecord::Base
    has_many :movies, dependent: :destroy
    has_many :childrens, through: :movies
end


@cinema = Cinema.find(params[:id])
@childrens = Children.eager_load(:tickets, :movies).where(movies: {cinema_id: @cinema.id}, tickets: {cinema_id: @cinema.id})


<% @childrens.each do |children| %>
  <%= children.movies.count %>
<% end %>

Answer 7

您使用counter_cache 的方法是正确的。

但要充分利用它，我们以 children.movi​​es 为例，您需要先将 tickets_count 列添加到 children 表中。

执行rails g migration addTicketsCountToChildren tickets_count:integer，

然后rake db:migrate

现在每张票创建都会自动将其所有者（孩子）中的票数增加 1。

那么你可以使用

<% @childrens.each do |children| %>
  <%= children.movies.size %>
<% end %>

没有收到任何警告。

如果你想按电影统计孩子，你需要将childrens_count添加到movie表中：

rails g migration addChildrensCountToMovies childrens_count:integer

然后rake db:migrate

参考：

http://yerb.net/blog/2014/03/13/three-easy-steps-to-using-counter-caches-in-rails/

如果有任何问题，请随时询问。

Answer 8

基于sarav answer，如果您有很多事情（请求）要计算，您可以做：

在控制器中：

@childrens_count = @cinema.childrens.joins(:movies).group("childrens.id").count.to_h

在视图中：

<% @childrens.each do |children| %>
   <%= @childrens_count[children.id] %>
<% end %>

如果你训练计算关联记录，这将阻止大量 sql 请求