如何通过一个共同的属性将两个数组合并为一个？

Question

如何将两个数组合二为一，例如...

Day 1: Notes/form
Day 2: Notes/form
Day 3: Notes/form
Day 4: Notes/form
Day 5: Notes/form

如果用户随后在第 3 天创建笔记，它看起来像...

Day 1: Notes/form
Day 2: Notes/form
Day 3: Notes/notes
Day 4: Notes/form
Day 5: Notes/form

下面的代码怪物试图实现上述目标，但迭代全部结束。例如，如果在第 1 天和第 3 天创建注释，则输出将如下所示...

Day 1: Notes/notes # Shows both Day 1 and Day 3 note. Only want to show Day 1 note
Day 1: Notes/form
Day 2: Notes/form
Day 2: Notes/form
Day 3: Notes/notes # Shows both Day 1 and Day 3 note. Only want to show Day 3 note
Day 3: Notes/form
Day 4: Notes/form
Day 4: Notes/form
Day 5: Notes/form
Day 5: Notes/form

挑战/表演

<% @challenge.dates_challenged.first(@challenge.days_challenged).each_with_index do |date, i| %>
  <% @notes.each do |note| %>
    Day <%= i + 1 %>
    <% if note.notes_date.strftime("%m/%d/%y") == date.strftime("%m/%d/%y") %>
      <%= render 'notes/notes' %>
    <% else %>
      <%= render 'notes/form', :date => date %>
    <% end %>
  <% end %>
<% end %>

概括地说，用户创建了一个挑战。挑战具有属性days_challenged。用户选择要挑战的天数，即 10、15、30 等。对于每一天，我想显示一个注释/表格。如果用户随后在某一天输入了一条注释，则该注释/表单应在显示页面上替换为该注释（任何一天都不应有多个与之关联的注释）。

Answer 1

您在用于查找当天笔记的循环中做的太多了。这应该是正确的：

<% @challenge.dates_challenged.first(@challenge.days_challenged).each_with_index do |date, i| %>
  Day <%= i + 1 %>
  <% if @notes.any? { |note| note.notes_date.strftime("%m/%d/%y") == date.strftime("%m/%d/%y") } %>
    <%= render 'notes/notes' %>
  <% else %>
    <%= render 'notes/form', :date => date %>
  <% end %>
<% end %>

这在视图中还是太多逻辑了。

1. 正如 photoionized 所评论的那样，如果 @notes 是一个 Hash，其键为 Dates 并且其值为 Notes，这将更容易。然后，您可以将 any? 替换为哈希查找。
2. 我不清楚为什么需要比较 strftime("%m/%d/%y")。你应该可以比较Dates。 （如果您因为它们位于不同的时区或其他原因而无法做到这一点，请考虑修复它 - 这会导致整个应用程序混乱。）
3. 我会将dates_challenged.first(@challenge.days_challenged) 提取到Challenge 上名为current_dates_challenged 的方法中，然后在视图中使用它。