model内記載のメソッドを綺麗にしたい

Question

カウントダウン表示の為に、以下の通りモデル内のメソッドに

残り ①日
残り ②時間
残り ③分
残り ④秒で分けて計算できるメソッドを作りました。

コードは機能しているのですが、全くDRYでなく、とはいえ、どうすれば綺麗になるのか分からず
綺麗な書き方をご教示頂けますと幸甚です。

# == Schema Information
#
# Table name: tenders
#
#  id                 :bigint(8)        not null, primary key
#  seller_id          :bigint(8)        not null
#  tender_location_id :bigint(8)        not null
#  starts_at          :datetime         not null
#  ends_at            :datetime         not null
#  preparation_status :integer          default(0), not null
#  created_at         :datetime         not null
#  updated_at         :datetime         not null
#

class Tender < ApplicationRecord
  def remaining_days
    total_seconds = (ends_at - Time.current).round
    total_seconds / (60 * 60 * 24)
  end

  def remaining_hours
    total_seconds = (ends_at - Time.current).round
    remaining_days = total_seconds / (60 * 60 * 24)
    total_seconds / (60 * 60) - (remaining_days * 24)
  end

  def remaining_minuites
    total_seconds = (ends_at - Time.current).round
    remaining_days = total_seconds / (60 * 60 * 24)
    remaining_hours = (total_seconds / (60 * 60) - (remaining_days * 24))
    total_seconds / 60 - (remaining_days * 24 * 60 + remaining_hours * 60)
  end

  def remaining_seconds
    total_seconds = (ends_at - Time.current).round
    remaining_days = total_seconds / (60 * 60 * 24)
    remaining_hours = (total_seconds / (60 * 60) - (remaining_days * 24))
    remaining_miniutes = (total_seconds / 60) - (remaining_days * 24 * 60 + remaining_hours * 60)
    total_seconds - (remaining_days * 24 * 60 * 60 + remaining_hours * 60 * 60 + remaining_miniutes * 60)
  end

end

Answer 1

Railsのビューヘルパーにdistance_of_time_in_wordsというのがあり
表示だけなら便利にやれます。

日本語資料としては、引数を減らしたtime_ago_in_wordsの方が探しやすいかもしれません。

---

また、ActiveSupport::Duration.buildという手段もあります。

class Tender
  def remaining
    ActiveSupport::Duration.build(ends_at - Time.current).parts
  end

  def remaining_hours; remaining[:hours] end
  def remaining_minuites; remaining[:minutes] end
  def remaining_seconds; remaining[:seconds] end
  def remaining_days
    rem = remaining
    rem[:days] + 7 * rem[:weeks]
  end
end

追記: 1月以上になるとこの方法だと問題が出てきます

Comments

[innjera]

ありがとうございます！そのヘルパー知りませんでした。参考になります！

Answer 2

既存のメソッドを呼ぶようにすれば、物凄くシンプルになります。

class Tender < ApplicationRecord
  def total_seconds
    (ends_at - Time.current).round
  end

  def remaining_days
    total_seconds / (60 * 60 * 24)
  end

  def remaining_hours
    total_seconds / (60 * 60) - (remaining_days * 24)
  end

  def remaining_minuites
    total_seconds / 60 - (remaining_days * 24 * 60 + remaining_hours * 60)
  end

  def remaining_seconds
    total_seconds - (remaining_days * 24 * 60 * 60 + remaining_hours * 60 * 60 + remaining_miniutes * 60)
  end

end

Comments

[innjera]

言われてみれば単純な話も、1人では分からないレベルなので、大変助かります！
ありがとうございました、かなりスッキリしました