这可以重构为单个数据库调用吗？

Question

我在 Events 模型中有一个事件列表。我需要做的是按城市/州对事件进行可视化分组。

从视觉上看，它看起来像这样：

这是我认为我正在做的事情的精简外观实现这一点：

<% @number.events.locations.each do |location| %>
  <li>
    <h4><%= "#{location.city}, #{location.state}" %></h4>
    <ol>
    <% @number.events.city_state(location).each do |event| %>
      <li><%= event.status %></li>
    <% end %>
    </ol>
  </li>
<% end %>

以及相应的范围：

scope :locations, :group => 'city, state'
scope :city_state, lambda {|location| {:conditions => ["city = ? AND state = ?", location.city, location.state] }}

但问题是我正在为每个位置进行额外的数据库调用。

那么我可以将其简化为单个查询吗？

Answer 1

使用预先加载从数据库中预取关联：

如果您一次提取所有事件，您可以将其更改为预加载事件的位置关联：

Event.all      # only loads Event objects from the database

#maybe try...

Event.includes(:locations)    # loads Event objects and their Locations

如果您需要某些其他对象的事件（例如您的@number 对象），您可能希望在控制器中执行类似的操作，例如：

@number = Number.where(:id => whatever).includes(:events => :locations)

现在，在此示例中，将从数据库中提取您的 @number 对象以及所有的事件和所有这些事件的地点。

有关更多信息，请参阅 Rails 指南：http://guides.rubyonrails .org/active_record_querying.html#eager-loading-associations

更新了我在评论中的答案：

根本不使用范围，您可以简单地使用 Ruby 为您完成这项工作，并将 ActiveRecord 和额外的数据库调用从循环中剔除。

Enumerable#group_by 将根据您提供的某些属性获取集合和分组。在这种情况下，您可以获取所有事件，按城市和州进行分组，然后迭代它返回的哈希值：

events = Event.all
events.group_by { |e| "#{e.city}, #{e.state}" }
# => {"Birmingham, AL" => [<event>, <event>, <event>], "Nashville, TN" => [<event>, <event>]}

这样做可能比使用范围不太漂亮，但它允许您进行一次数据库调用，并且就这样吧。

Answer 2

这是一个 O(n^2) 解决方案，假设您的 locations.each do |location| 循环按广告所示工作。应该将其降低至 2 分贝。

<% @number.events.locations.each do |location| %>
  <li>
    <h4><%= "#{location.city}, #{location.state}" -%></h4>
    <ol>
      <% @number.events.each do |event| %>
        <% if event.city == location.city and event.state == location.state %>
        <li><%= event.status -%></li>
        <% end %>
      <% end %>
    </ol>
  </li>
<% end %>

尚未测试，稍后再回来查看。