Rails：将逻辑移出视图

Question

简而言之，我的视图中有一些 if/else 逻辑，我认为它们作为控制器方法或帮助器会更有效。也就是说，我不确定将其转化为方法的最佳方式是什么，或者放置它的最佳位置。

我正在我的 Rails 应用程序中构建一个用户消息传递系统。在我的应用程序中，个人资料有许多对话（两个个人资料之间），其中包含许多消息。我正在尝试构建基本上就像收件箱一样的对话索引。作为一名 Rails 初学者，我不太确定将 if/else 部分移出视图的最有效/最佳方式是什么，因为它在 .each 循环中运行。现在我的观点是这样的：

<% @conversations.each do |conversation| %>
<% if conversation.sender_id == current_user.profile.id %>
  <% recipient = Profile.find(conversation.recipient_id) %>
<% else %>
  <% recipient = Profile.find(conversation.sender_id) %>
<% end %>
<%= link_to recipient.first_name, conversation_messages_path(conversation)%>

供参考我的对话控制器索引操作：

def index
  @profiles = Profile.all
  @conversations = Conversation.involved(current_user.profile.id)
end

还有我的对话模式

class Conversation < ActiveRecord::Base
  belongs_to :sender, :foreign_key => :sender_id, class_name: 'Profile'
  belongs_to :recipient, :foreign_key => :recipient_id, class_name: 'Profile'
  has_many :messages, dependent: :destroy
  accepts_nested_attributes_for :messages

  validates_uniqueness_of :sender_id, :scope => :recipient_id

  scope :between, -> (sender_id,recipient_id) do
   where("(conversations.sender_id = ? AND conversations.recipient_id =?) OR (conversations.sender_id = ? AND conversations.recipient_id =?)", sender_id,recipient_id, recipient_id, sender_id)
  end

  scope :involved, -> (user) do
   where("(conversations.sender_id = ? OR conversations.recipient_id =?)", user, user)
  end
end

这是我的第一个 stackoverflow 问题，所以如果我没有遵循此问题的任何最佳做法，请告诉我！

Answer 1

欢迎来到 StackOverflow！

在这种情况下，您只是根据 conversation.sender == current_user 与否打印配置文件的名称。在您的 Conversation 模型上，您可以有一个接收用户的 :display_name 方法。

def display_name(profile)
  (sender == profile ? recipient : sender).name
end

这是演示方法，因此它可能更适合 Helper、Presenter、Decorator 或许多其他解决方案，但这应该可以解决问题。在你看来叫它：link_to conversation.display_name(current_user.profile), ...

Answer 2

就个人而言，我会说您的 if/else 逻辑适用于视图；最好避免在每个块中进行查询。本质上，您有所谓的 n+1 查询（@conversations 中有 n 个对象，并且您在数据库中查询每个对象的 Profile 对象，以及用于填充 @conversations 的原始查询，因此为 n+1）。

在控制器中，您可以像这样加载发件人和收件人：

scope :involved_with_profiles -> (user) do
  includes(:sender, :recipient).where(...same logic as your :involved scope...)
end

使用此范围而不是 :involved 来加载 @conversations。然后，您可以将 if/else 逻辑保留在视图中，但您可以这样做：

<% recipient = conversation.recipient %>

或

<% recipient = conversation.sender %>

因为这些已经从单个查询加载到@conversations。

Answer 3

它在哪里很好。

将它放在助手中不会给您带来任何好处，但可以在其他地方重用。