在视图之外重构代码

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 %>

---

预期的输出是：

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

昨天的活动
苏创建了一篇新文章

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

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

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