将时间与时间范围进行比较，并在之前、期间或之后返回

Question

我正在设备上创建一个状态，该状态将根据设备发回读数后的时间长短为该设备设置指示器颜色。这是我目前的解决方案：

def indicator(device)
  # ...
  time = latest_event_time(device.id)

  case
  when time < 12.hours.ago
    :red
  when time.between?(12.hours.ago, 2.hours.ago)
    :yellow
  when 2.hours.ago < time
    :green
  else
    :gray
  end
end

private

def latest_event_time(dev_id)
  # Get latest event time from database
end

我希望能够简单地检查 time 是在指定时间范围之前、期间还是之后，可能是这样的：

time <=> (2.hours.ago...12.hours.ago)

是否有我遗漏的方法，或者另一种更简洁的方法来完成此任务？

Answer 1

不知道您是否认为这是一种改进，但是：

color = [ [12, :red], [2, :yellow], [0, :green] ].find do |n, c| 
  n.hours.ago >= time
end.last

Answer 2

怎么样：

def time_to_color(time)  
  { green: 2, yellow: 12, red: Float::Infinity }.find { |_,t|
    time <= t.hours_ago }.first
end

也许我遗漏了一些东西，但我不知道如何返回 :gray。我以这种方式组织它以处理未来的时间。

Answer 3

您可以为Range 重载<=> 运算符：

class Range
  # return index into a 3-element array for before, in, and after
  def <=>(x)
    x < self.begin ? 0 : include?(x) ? 1 : 2
  end
end

def indicator(device)
  # ...
  time = latest_event_time(device.id) or return :gray

  [:red,:yellow,:green].at (2.hours.ago...12.hours.ago) <=> time
end

整合

关于如何将重载定义集成到 rails 4 项目的一个建议是将 Range 代码存储在 lib/ext/range.rb 中并添加行

require 'ext/range'

到config/initializers/extensions.rb。根据需要创建目录和文件。