【Rails】validatesとvalidateの併用ができない

Question

前提・実現したいこと

通常のバリデーションとカスタムのバリデーションを実装したい。

発生している問題・エラーメッセージ

• エラーメッセージ：undefined method `hour' for nil:NilClass
• 発生している問題：

vaildatesとvalidateを併用すると上記エラーメッセージが出る。この時、validateの方のみ機能している。
validateがなければvalidatesは正常に機能する。

該当のソースコード

class Shift < ApplicationRecord
  
  belongs_to :staff
  has_many :post_comments, dependent: :destroy
  
  validate  :start_finish_check
  validates :date, presence: true
  validates :start_time, presence: true
  validates :finish_time, presence: true
  
  def start_finish_check
    errors.add(:finish_time, "が開始時刻を上回っています。正しく記入してください。")if self.start_time.hour >= self.finish_time.hour
  end
 
  
end

試したこと

仮説：エラーメッセージ通り、hourをどこか定義すればいいと思うのだが、validateのみだと機能するので、hour以外の問題もあるのではと思う。

Answer 1

おそらく、start_timeやend_timeがnilの状態でstart_finish_checkが呼ばれてしまっているのではないかと思います。

このような場合、どちらかがnilであれば時間の前後は判定不能なので、そちらの条件はpresenceに任せて、「片方でもnilなら何もせずstart_finish_checkを抜ける」とするのが一般的です。

Comments

[MoM]

ありがとうございます！
unless :start_time.nil? || :finish_time.nil?
def start_finish_check
...

とやって上手く行かなかったですが、なんとなくイメージはつかめたので、もう少しやってみます！

[maisumakun]

> とやって上手く行かなかったですが

条件チェックは、start_finish_checkの中に書きます。

[MoM]

中に書いて見たのですが、undefined method `hour' for nil:NilClassとなってしまいました、、、