在视图之外重构代码

Question

我有以下视图代码，它显示事件，以及通知用户今天、昨天或过去一周是否发生以下事件的标题。

一切都工作正常，但我更愿意将其重构到视图之外，我不能将其放入助手中，因为它需要在循环迭代之间维护变量状态。知道如何整理这个吗？谢谢

<% displayed_week_already = false %>
<% displayed_yesterday_already = false %>
<% displayed_today_already = false %>

<% @events.each do |event| %>

  <%= event.title &>

  <% if (Time.now - 1.week) > event.created_at && !displayed_week_already %>
    <% displayed_week_already = true %>
    Events in past week
  <% elsif (Time.now - 1.day) > event.created_at && (Time.now - 2.day) < event.created_at && !displayed_yesterday_already %>
    <% displayed_yesterday_already = true %>
    Events yesterday
  <% elsif (Time.now - 1.day) < event.created_at && !displayed_today_already %>
    <% displayed_today_already = true %>
    Events in past day
  <% end %>

<% end %>

---

预期输出为：

过去一天的事件
约翰创建了一篇新文章
Dan 对第 1 条的评论
苏删除了文章 2

昨天的事件
Sue 创建了一篇新文章

过去一周的活动
卡罗尔升职
丹发表了一篇新文章

Answer 1

我可能会做类似以下的事情。我省略了模型本身定义的实现方法。

现有视图

<%= render "events", :title => "Events in past day",  :events => from_today(@events) %>
<%= render "events", :title => "Events yesterday",    :events => from_yesterday(@events) %>
<%= render "events", :title => "Events in past week", :events => from_this_week(@events) %>

_events.html.erb

<% if events.present? %>
  <%= title %>
  <% events.each do |event| %>
    <%= event.title %>
  <% end %>
<% end %>

app/helpers/event_display_helper.rb

module EventDisplayHelper
  def from_today(events)
    events.select{|x| x.happened_today?}
  end

  def from_yesterday(events)
    events.select{|x| x.happened_yesterday?}
  end

  def from_this_week(events)
    events.select{|x| x.happened_this_week_before_yesterday?}
  end
end

Answer 2

您可以使用命名范围轻松地在模型中执行此操作（我假设使用 Rails 3，因为这是我最了解的）。您需要自己定义它们，但它们非常简单。

@last_week = @event.last_week
@yesterday = @event.yesterday
@today = @event.today

然后只需迭代您视图中的每个组即可。应该让它变得更干净，你不需要那么多逻辑，因为它是它真正所属的地方。