4

我正在开发一些时间跟踪应用程序，但遇到了一个我不知道如何解决的问题。我有一个Task模型和一个Client模型。每个任务都属于一个客户端。

class Task < ActiveRecord::Base
  belongs_to :client
  attr_accessible :client_id, :description, :start, :end

  scope :yesterday, -> {
    where('start > ?', Date.yesterday.to_time).where('start < ?', Date.today.to_time)
  }
end

class Client < ActiveRecord::Base
  attr_accessible :name
  has_many :tasks
end

现在，我正在显示任务完成日期范围内的任务列表，并按完成时间排序。我想显示相同的列表，但按客户端分组，并按客户端名称排序。这是我想做的事情：

<div id="yesterday_summary">
  <% @yesterday_clients.each do |client| %>
    <h2><%= client.name %></h2>
    <ul>
      <% client.tasks.each do |task| %>
        <li><%= task.description %></li>
      <% end %>
    </ul>
  <% end %>
</div>

在我的控制器中，我目前有：

@tasks_yesterday = Task.yesterday
@yesterday_clients = group_tasks_by_client @tasks_yesterday

在该group_tasks_by_client方法中，我有一些非常丑陋的代码，目前甚至无法正常工作：

  def group_tasks_by_client(tasks)
    clients = []
    tasks.collect(&:client).each do |client|
      clients << {client.id => client} unless clients.has_key? client.id
    end
    clients_with_tasks = []
    clients.each do |client|
      c = Struct.new(:name, :tasks)
      cl = c.new(client.name, [])
      tasks.each do |task|
        cl.tasks << task if task.client_id = client.id
      end
      clients_with_tasks << cl
    end
    clients_with_tasks
  end

我确信有一个干净、简单、rails 的方式来做到这一点，但我不知道怎么做。如何才能做到这一点？

4

1 回答 1

8

您可以让数据库为您执行此操作，如下所示：

@yesterdays_clients = Client.includes(:tasks).merge(Task.yesterday).order(:name)

除了更清洁之外，它更高效，因为它可以一次性获取所有客户和任务。原始代码需要进行 N+1 次查询，因为没有预先加载。

顺便说一句，您也可以使您的范围更简单：

scope :yesterday, -> { where(:start => (Date.yesterday.to_time...Date.today.to_time)) }
于 2013-03-06T17:48:40.540 回答