コードをもう少しシンプル（綺麗）に書けないか

Question

以下の通り、＠＠分前に、＠＠時間前、＠＠日前といった形になるようにコードを書いています。
これで問題なく表記されるのですが、なんとなく原始的、冗長で、もう少しシンプル（綺麗）にか書けないでしょうか？

<% 
  time_n = Time.now
  time_p = Time.at(user.created_time.to_i)
  time_s = time_n - time_p
%>

<% if time_s/60 < 60 %>
  <%= sprintf("%.0f",time_s/60) %>min ago
<% elsif time_s/60/60 < 24 %>
  <%= sprintf("%.0f",time_s/60/60) %>hours ago
<% elsif time_s/60/60/24 < 2 %>
  <%= sprintf("%.0f",time_s/60/60/24) %>day ago
<% elsif %>
  <%= sprintf("%.0f",time_s/60/60/24) %>days ago
<% end %>

Answer 1

まず、60は1分間、60*60は1時間などの表記をわかりやすく書く方法がRailsには用意されています。
Numeric - Ruby on Rails API
つまり、1.minutesとか1.hoursとか1.daysとかで表現できますので、それと比較や割ったりすることで、時間や分数が得られます。比較もそのまま、time_s < 1.minutesみたいな書き方が可能です。

次に、if-elseですが、うーん、ちょっと微妙ですが、case-when文にすることもできます。case-when文はwhenは===で比較なのですが、Rangeと比較すると===は範囲に含むか含まないかになるため、そのような処理も可能です。

<% case time_s %>
<% when 0 ... 1.hours %>
  <%= sprintf("%.0f", time_s / 1.minutes) %>min ago
<% when 1.hours ... 1.days %>
  <%= sprintf("%.0f", time_s / 1.hours) %>hours ago
<% when 1.days ... 2.days %>
  <%= sprintf("%.0f", time_s / 1.days) %>day ago
<% else %>
  <%= sprintf("%.0f", time_s / 1.days) %>days ago
<% end %>

Comments

[innjera]

なるほどーー、本片手にやっていましたが、知りませんでした。
大変有難う御座います。ご教示頂いた書き方の方がずっと、プログラミングら引くかっこいいですね。
レベルアップできました。

Answer 2

そのような処理が書かれたヘルパーが既に存在しています。
http://apidock.com/rails/ActionView/Helpers/DateHelper/time_ago_in_words

もし、どうしてもコードを書きたいなら、このような感じで。

def humanize secs
  [[60, :seconds], [60, :minutes], [24, :hours], [1000, :days]].map{ |count, name|
    if secs > 0
      secs, n = secs.divmod(count)
      "#{n.to_i} #{name}"
    end
  }.compact.reverse.join(' ')
end

p humanize 1234
#=>"20 minutes 34 seconds"
p humanize 12345
#=>"3 hours 25 minutes 45 seconds"
p humanize 123456
#=>"1 days 10 hours 17 minutes 36 seconds"
p humanize(Time.now - Time.local(2010,11,5))
#=>"4 days 18 hours 24 minutes 7 seconds"

http://stackoverflow.com/questions/4136248/how-to-generate-a-human-readable-time-range-using-ruby-on-rails