循环遍历 SQL 结果集的更好方法

Question

我只是想知道是否有更好的方法来对存储在实例变量中的另一个 SQL 结果集执行 SQL 查询。

我的代码如下所示：

def index
  id = current_user.id.to_s

  #First SQL query
  @messages = Message.where("user_id = ? OR recipient_id = ?", id, id).group('recipient_id')
end

在我看来

<% @messages.each do |m| %>
<!-- Second SQL query -->
<% r = Tutor.find(m.recipient_id).user %>
  <%= "#{r.first_name + ' ' + r.last_name}" %><br />
  <%= m.subject %><br />
  <%= m.body %><br />
  <%= m.user_id %><br />
  <%= m.recipient_id %><br />
<% end %>

我正在循环@messages，然后对每个对象执行 SQL 查询，我想知道是否有更好（更智能）的方法来执行此操作？ 谢谢！

Answer 1

您需要在 Message 和 Tutor 之间建立 ActiveRecord 关联。我需要查看您的模型以准确说明如何将它们关联起来，但您基本上是在自己进行关联工作，而不是将其卸载到 AR/SQL。

一旦建立了适当的关系，您就会看到类似的情况。

在您的控制器中：

def index
  id = current_user.id
  @messages = Message.includes(:tutor).where("user_id = ? OR recipient_id = ?", id, id).group(:recipient_id)
end

在你看来：

<% @messages.each do |m| %>
  <%= m.tutor.user.to_s %><br />
  <%= m.subject %><br />
  <%= m.body %><br />
  <%= m.user_id %><br />
  <%= m.recipient_id %><br />
<% end %>

在您的用户模型中：

class User < ActiveRecord::Base
  def to_s
    "#{first_name} #{last_name}"
  end
end