Rails 環境の、日付配列の操作について

Question

Rails5で、下記 array1 を元に、array2 を生成したいです。シンプルな方法を教えて下さい。

array1 = ["12/28", "30", "31", "1/1", "2", "4"]
array2 = ["2018/12/28", "2018/12/30", "2018/12/31", "2019/01/01", "2019/01/02", "2019/01/04"]

2018年というのは今年で、2019年は来年（未来）という想定です。

array1 = ["12/28", "30", "31", "1/1", "2", "4"]

this_year  = Time.now.strftime("%Y") #=> "2018"
this_month = Time.now.strftime("%m") #=> "08"
year   = ""
month  = ""

array2 = []
array1.each do |elm|
  if elm.match(/^(\d{1,2})/(\d{1,2})$/)
    month = $1 #=> ex. "12", "1"
    day   = $2

    if month.to_i <= this_month.to_i #=> ex. 12 <= 8, 1 <= 8
      year = (this_year.to_i + 1).to_s #=> "2019"
    else
      year = this_year
    end

  else
    day = elm
  end

  d = Time.strptime(year + "/" + month + "/" + day, "%Y/%m/%d").strftime("%Y/%m/%d")
  array2 << d
end
#=> ["2018/12/28",
 "2018/12/30",
 "2018/12/31",
 "2019/01/01",
 "2019/01/02",
 "2019/01/04"]

一応できたのですが、どうも強引な感じがして、もっと上手い書き方を知りたいです。

Comments

[takumiabe]

まず自分で書いてみる姿勢が大事かと思いますよ。

[blue_1997]

おっしゃる通りだと反省し、コードを記載しました。

Answer 1

なるほど。
正規表現ならば(\d{4}/)?みたいな書き方を使う場面だと思います。

%r[(((\d{4})/)?(\d{1,2})/)?(\d{1,2})]

というところですね。
しかしながら、この場合は正規表現を使うよりもsplitしてしまうほうがシンプルにかけそうです。

あとは、TimeではなくDateを使いつつ…

today = Date.today

...

day, month, year = *elm.split('/').map(&:to_i).reverse
month ||= today.month
year ||= today.year
d = Date.new(year, month, day)
d = d.next_year if d < today # 過去の日付になったら翌年

とすれば、目的の日付が1つ得られると思います。

• 分割代入を上手く使うとデータの分解が上手にできることが多いです
• デフォルト値を取り扱う時は、||= が活きます。

あたりがポイントかと思います。

---

条件を誤読していたので修正します。
前に設定された情報の中に「月」等があれば、継続してそれを用いる、ということなので、

today = Date.today
prev_month = nil

array1.each do |elem|
  day, month, year = *elem.split('/').map(&:to_i).reverse
  month ||= prev_month || raise("一度もmonthが指定されなかった")
  year ||= today.year

  d = Date.new(year, month, day)
  d = d.next_year if d < today # 過去の日付になったら翌年

  prev_month = d.month # 次のelemの処理のために保持
  array2 << d
end

というところかと思います。

Comments

[blue_1997]

多くの示唆をいただき、ありがとうございます。自分基本ひとりで書いているため、自分が書いているものをどうしたらもっとよく出来るのかという点に悩みを持っています。大変参考になりました。

３点教えて下さい。
1. `month ||= today.month` については、デフォルトを今月にするとまずいので、直近で使用された値を使いたいたいです。良い方法はありませんか？
2. `d = Date.new(year, month, day)`の引数は、それぞれ .to_i を付けたらうまくいきました。その解釈で問題ないでしょうか？
3. `elm_date < today` の、elm_date は、d と同一と考えてよいでしょうか？

[takumiabe]

1について、反映してみました。どうでしょうか？
2,3については私の書きミスです。訂正しました。

[blue_1997]

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

期待した動作が、スマートな記述で実現できる事を知りました。

.map(&:to_i).reverse という表記は、自分ではこれまでしたことが無かったので、使う機会を見つけたいと思いました。また、raise の使い方も、これまでbegin〜rescure〜endといったベタな例外処理でしか使ったことが無かったので新鮮です。

早く自分も、スマートで間違いの起きづらい記述ができるようになりたいと、心から思いました。
※さっそく、Effective Ruby という本を買って読み始めました。

心から、感謝いたします！

[takumiabe]

つたない回答ですが、助力になれば幸いです。
応援してます！