曜日による分岐をスッキリさせたい

Question

曜日による分岐をスッキリさせたい

現状

if Time.zone.today.sunday?
  duty = "資源ごみ"
  toban = Tobanhyo.where(role: 1)
elsif Time.zone.today.tuesday?
  duty = "火曜のゴミ"
  toban = Tobanhyo.where(role: 2)
elsif Time.zone.today.wednesday?
  duty = "木曜日のゴミ"
  toban = Tobanhyo.where(role: 3)
elsif Time.zone.today.Thursday? || Time.zone.today.friday?
  duty = "燃えるゴミ"
  toban = Tobanhyo.where(role: 4)
end

理想

変数は分岐のたびに作りたくない
ifの分岐は愚直に１つずつやりたくない

よろしくお願いいたします。

Comments

[mather]

if文が完了していないので、endまで書いたほうがいいと思います。また、読みやすくするためにインデントを使いましょう。

[og33]

失礼いたしました。修正いたしました。引き続きよろしくお願いします。

Answer 1

duty, toban_id = case Time.zone.today.strftime("%a")
                 when "Sun" then ["資源ごみ", 1]
                 when "Tue" then ["火曜のゴミ", 2]
                 when "Wed" then ["水曜のゴミ", 3]
                 when "Thu","Fri" then ["燃えるゴミ", 4]
                 end
toban = Tobanhyo.where(role: toban_id)

strftime("%a")じゃなくてwdayもありますが、それだと数字になるので分かりにくい。

Comments

[otn]

case/whenで分岐せず、曜日でハッシュを引くのもあるかと思います。

[mkgrei]

ハッシュに1票。
なんなら当番表もハッシュに…

[otn]

曜日がどこにも該当しないケースの考慮が要りましたね。

[og33]

ありがとうございました！
一番すっきりしていて初心者の自分にもわかりやすかったので、ベストにさせていただきました！

Answer 2

そもそも気になっているポイントなのですが、 Time.zone って動作しますか？なにかライブラリを使っているのかな…？

曜日を取得する場合、 Time.now.wday とすると 0..6 の範囲で曜日が取得できます。

これを使って書く場合、僕ならこうします。

SUN = 0
MON = 1
TUE = 2
WED = 3
THU = 4
FRI = 5
SAT = 6

def garbage_day(weekday)
  case weekday
  when SUN
    {garbage_type: "資源ごみ", id: 1}
  when TUE
    {garbage_type: "火曜のゴミ", id: 2}
  when WED
    {garbage_type: "水曜のゴミ", id: 3}
  when THU, FRI
    {garbage_type: "燃えるゴミ", id: 4}
  else
    nil
  end
end

garbage_today = garbage_day(Time.now.wday)

if garbage_today
  toban = Tobanhyo.where(role: garbage_today[:id])
end
  

Answer 3

INFOS = [
  # duty, sql param                                                                                                        
  nil,
  nil,                 # 1: 月曜                                                                                           
  ['火曜のゴミ',   2], # 2: 火曜                                                                                           
  nil,                 # 3: 水曜                                                                                           
  ['木曜日のゴミ', 3], # 4: 木曜                                                                                           
  ['燃えるゴミ',   4], # 5: 金曜                                                                                           
  nil,                 # 6: 土曜                                                                                           
  ['資源ごみ',     1], # 7: 日曜                                                                                           
].freeze

duty = nil
toban =	nil

infos =	INFOS[Time.zone.today.cwday]
if info
  duty = info[0]
  toban = Tobanhyo.where(role: info[1])
end

名前と where する時のパラメータを 曜日 (月曜が 1) を indexにもつ配列にしてみました。