経験年数を◯年◯ヶ月と表示させたいです

Question

質問投稿失礼いたします。
勤続年数を◯年◯ヶ月と表示させたいのですがいいアイディアが思いつきません。。。
もともとは◯年と表示されていました。
下記が現時点のコードで、一応は自分で無理やり◯ヶ月と表示させるところまでは至ったのですが、明らかに冗長で違和感しかありません。
ヒントいただけないでしょうか。。

下のコードの

def experience_months
省略
end
と
def get_month(value)
省略
end
の部分が自分で書き足して◯ヶ月と表示させたものです。

実際のコード↓

user.rb

def experience_years
get_year(start_engineer)
end

def experience_months
if get_month(start_engineer) == -1
p 11
elsif get_month(start_engineer) == -2
p 10
elsif get_month(start_engineer) == -3
p 9
elsif get_month(start_engineer) == -4
p 8
elsif get_month(start_engineer) == -5
p 7
elsif get_month(start_engineer) == -6
p 6
elsif get_month(start_engineer) == -7
p 5
elsif get_month(start_engineer) == -8
p 4
elsif get_month(start_engineer) == -9
p 3
elsif get_month(start_engineer) == -10
p 2
elsif get_month(start_engineer) == -11
p 1
else
get_month(start_engineer)
end
end

private

def get_year(value)
value.blank? ? 0 : (Date.today.strftime('%Y%m%d').to_i - value.strftime('%Y%m%d').to_i) / 10000
end

def get_month(value)
value.blank? ? 0 : (Date.today.strftime('%m').to_i - value.strftime('%m').to_i)
end
end

show.html.slim

th = User.human_attribute_name(:experience_years)

td = @user.experience_years.to_s + '年' + @user.experience_months.to_s + 'ヶ月'

今のところはModel内で現在の日時から職務スタート日を引き算しています。
この方法しか思いつかず途方に暮れています。。ヒントいただけたらと大変嬉しいです。
うまく伝わっていたら幸いです。

Answer 1

細かな仕様を決める必要がありますが、経歴書に書くような 開始年月、終了年月を書いて、期間を年月で書くような
場合の例で書いてみました。

y.rb

require 'date'

def diff_month(d0, d1)
  (d1.year * 12 + d1.month) - (d0.year * 12 + d0.month)
end

def diff_str(d0, d1)
  diff = diff_month(d0, d1) + 1  # 仕様によっては +1 しないほうが良いかも
  d_year = diff / 12
  d_month = diff % 12
  str = ''
  str += "#{d_year}年" if d_year > 0
  str += "#{d_month}ヶ月" if d_month > 0
  str = "1ヶ月未満" if str == ''
  str
end

TESTS = [
  [Date.new(2016, 1, 1), Date.new(2016, 1, 31)],
  [Date.new(2016, 1, 1), Date.new(2016, 2, 29)],
  [Date.new(2016, 1, 1), Date.new(2016, 12, 31)],

  [Date.new(2016, 1, 1), Date.new(2017, 1, 31)],
  [Date.new(2016, 1, 1), Date.new(2017, 2, 28)],
  [Date.new(2016, 1, 1), Date.new(2017, 12, 31)]
].freeze

TESTS.each do |test|
  d0, d1 = test
  d_str = diff_str(d0, d1)
  p "#{d0.year}年#{d0.month}月 〜 #{d1.year}年#{d1.month}月 : #{d_str}"
end

実行結果

$ ruby y.rb
"2016年1月 〜 2016年1月 : 1ヶ月"
"2016年1月 〜 2016年2月 : 2ヶ月"
"2016年1月 〜 2016年12月 : 1年"
"2016年1月 〜 2017年1月 : 1年1ヶ月"
"2016年1月 〜 2017年2月 : 1年2ヶ月"
"2016年1月 〜 2017年12月 : 2年"

Comments

[glvty83]

返信遅れまして申し訳有りませんでした。
わざわざコードまで用意していただきお手数お掛けしました。
感謝しております。
上記のコード例は今後の参考にさせていただきます。
現在はテンプレートなどを検索して持ってきてコードに反映するのが精一杯なので、
自分の力で自作メソッドなどを柔軟に扱えるようになりたいです。。。>_<

Answer 2

if ~ elsifのあたりが冗長になってるのが気になってるんですよね?
問題は、月の差を求めたときに、値がマイナスになった時の場合で、
その場合の処理って、実は12を足してるだけです。
あと、いちいちメソッドを呼ばなくても、変数に入れればメソッド呼び出しは1回ですみます。
また、date.strftime("%m").to_iは一度文字列に直してから整数にしてるので、
これも冗長ですから、date.monthにしてみました。

def experience_month
  month = Date.today.month - start_engineer.month
  if month < 0
    month += 12
  end
  return month
end

Comments

[glvty83]

返信遅れてしまいすみませんでした。
とてもシンプルな方法で私にとってわかりやすいコードでしたので取り入れさせていただきます。確かに12足しているという法則ですね。。。今後はこういった法則性も自分で発見していくよう努めます!
おかげさまで現状下記のようになりました。

def experience_months
if get_month(start_engineer) < 0
p get_month(start_engineer) + 12
else
get_month(start_engineer)
end
end
だいぶすっきりしてよかったです。

どうもありがとうございました。

Answer 3

参考情報

• RubyのDate型で日付の差分を「□年◯ヶ月△日」で取得する方法
• 日付の差分を「○年△ヶ月」で取得する in Rails

Comments

[glvty83]

こちらのリンクもありがとうございます!
拝見させていただきました。
確かに閏年とか細かいところも気をつけないといけないのですね。。。