1

やりたいこと

hour:1 minute:2 second:3 とデータがある場合 → viewで1時間2分3秒と表示したい

分岐条件

hourはnilの可能性があるので、hourがnilの場合は時間は表示させない。
minute,secondは値が入っている。

質問

普通にifで条件分岐させるやり方しか思いつかないのですが、メソッドなどでスッキリ実装させる方法はないでしょうか。

コード

  unless result.run_hour.nil?
    hour = result.run_hour.to_s + "時間"
  else
    hour = ""
  end

  unless result.run_minute.nil?
    minute = result.run_minute.to_s + "分"
  else
    minute = ""
  end

  unless result.run_second.nil?
    second = result.run_second.to_s + "秒"
  else
    second = ""
  end

  hour + minute + second
2
  • 「1時間2分3秒」の2と3の数字を全角から半角に修正しました。
    – htb
    Commented 2018年3月23日 2:16
  • @manche 問題が解決したのでしたら、回答を承認することができます :)
    – nekketsuuu
    Commented 2018年3月23日 10:03

2 件の回答 2

1

スッキリしてるかどうかわかりませんが、こんな感じでどうでしょう。

[hour,minute,second].zip(["時","分","秒"]).select(&:first).join
3
  • さっそくの回答ありがとうございます。 目から鱗のやり方で。。。 select(&:first) ここのコードの意味がググっても不明でした。 ご教示いただけないでしょうか。
    – manche
    Commented 2018年3月22日 13:13
  • 1
    select(&:first)select{|obj| obj.first} と同じ意味です。ここでの obj[hour, "時"] 等の2要素の配列なので、その最初の要素が真のもの(nilで無いもの)を選択しているということになります。
    – tmtms
    Commented 2018年3月22日 14:01
  • ありがとうございます。 相当スッキリしました。
    – manche
    Commented 2018年3月22日 23:25
0

こういう書き方もできます。

"#{hour&.to_s&.+'時間'}#{minute&.to_s&.+'分'}#{second&.to_s&.+'秒'}"
2
  • 回答ありがとうございます。 hour&.to_s&. 二つ目の&.はなぜ必要なのでしょうか。 最初の&.だけでは、ダメなのでしょうか。
    – manche
    Commented 2018年3月22日 13:45
  • 1
    やってみれば分かると思いますが、hourがnilのときに例外が発生してしまうからです。
    – itmammoth
    Commented 2018年3月23日 3:10

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。