在 Ruby 中循环多个数组

Question

我有多个 ActiveRecord 子类 Item 实例数组，我需要根据最早的事件循环打印。在这种情况下，我需要打印付款和维护日期，如下所示：

项目 A 需要 5 天内维护
B 项需在 6 天内付款
项目 A 需要在 7 天内付款
B 项需要 8 天内维护

我目前有两个查询用于查找 maintenance 和 payment 项目（非排他查询）以及类似以下内容的输出：

<%- item_p = nil -%>
<%- item_m = nil -%>

<%- loop do -%>
  <% item_p ||= @items_p.shift %>
  <% item_m ||= @items_m.shift %>

  <%- if item_p.nil? and item_m.nil? then break -%>
  <%- elsif item_p and (item_m.nil? or item_p.paymt < item_m.maint) then -%>
    <%= item_p.name %> payment required in ...
  <%- elsif item_m and (item_p.nil? or item_m.maint < item_p.paymt) then -%>
    <%= item_m.name %> maintenance required in ...
  <%- end -%>
<%- end -%>

任何方式清理上面（丑陋的）代码的可读性？

Answer 1

采用鸭子类型并确保您的对象是多态。您希望您的付款项目与维护项目可比较，以便对它们进行排序。

因此，假设您有一个 Payment 和一个 Maintenance 类：

module Due
  include Comparable

  # Compare this object with another. Used for sorting.
  def <=>(other)
    self.due <=> other.due
  end
end

class Payment < ActiveRecord::Base
  include Due

  alias_method :due, :payment

  def action
    "#{name} requires payment"
  end
end

class Maintenance < ActiveRecord::Base
  include Due

  alias_method :due, :maintenance

  def action
    "#{name} requires maintenance"
  end
end

了解我们如何创建 action、due 和 < 两个类中的 code><=> 方法？我们还小心地包含了 Ruby 内置模块 Comparable。这使我们能够执行以下操作：

# Assuming 'payment' and 'maintenance' are date fields...
a = Payment.new :payment => 3.days.from_now
b = Maintenance.new :maintenance => 2.days.from_now
[a, b].sort
#=> [b, a]

然后视图变得简单如下：

<% (@payment_items + @maintenance_items).sort.each do |item| %>
  <%= item.action %> in <%= distance_of_time_in_words_to_now(item.due) %><br/>
<% end %>

我确信我没有正确了解您的实现细节，但我希望这能让您了解如何解决您的问题。

Answer 2

这是快速且肮脏的（即未优化）：

# In your controller:
@items = @items_p.map{ |item| {:item => item, :days => item.paymt, :description => "payment"} } 
@items += @items_m.map{ |item| {:item => item, :days => item.maint, :description => "maintenance"} }
@items = @items.sort_by{ |item| item[:day] }

# In your view:
<% @items.each do |item| %>
  <%= item[:item].name %> <%= item[:description] %> required in <%= item[:days] %> days
<% end %>

Answer 3

在你看来，你做得太过分了。实际上，您应该在控制器中弄清楚所有这些，并通过一个清理后的结构，您可以对其进行迭代以用于显示目的。

举个例子：

length = [ @items_p.length, @items_m.length ].sort.last

@messages = [ ]

length.times do |i|
  item_p = @items_p[i]
  item_m = @items_m[i]

  if (item_p and (item_m and item_p.paymt < item_m.maint) or !item_m)
    @messages << "#{item_p.name} payment required in ..."
  elsif (item_m and (item_p and item_m.maint < item_p.paymt) or !item_p)
    @messages << "#{item_m.name} maintenance required in ..."
  end
end

然后您可以根据需要迭代 @messages。

这里真正的问题是，你没有从战略上讲，为这类事情构建这些对象。如果您有一个单一的截止日期方法，而不必根据类型区分 paymt 和 maint，那就太好了。同样，如果将两者配对成一个数组而不是单独提供，那就更好了。

如果您将它们放在 [ p, m ] 对中，则可以更简单地进行迭代：

items.each do |pair|
  first_due = pair.compact.sort_by(&:due).first
  @messages << "#{first_due.name} #{first_due.action} required in ..."
end

action 方法将返回 payment 或 maintenance 根据需要。