Railsバリデーションが2重に走る

Question

問題

RubyonRailsでWEBシステムを開発中に、
if文を用いた条件分岐バリデーションが２重で走り、
エラーメッセージが重複してしまいます

実現したいこと

エラーメッセージが二重で表示されるのを防ぎたいです

該当のソースコード

class Master::Rac < ApplicationRecord
    self.table_name = "RAC_RAC"

    #英数字・記号("@","-","_")のみの入力を許可する。空白も許可
    # VALID_PROD_REGEX = /\A[a-z0-9]+\z/i
    #英数字・記号("@","-","_")のみの入力を許可する。空白も許可
    # VALID_NO_REGEX = /\A[\w@-]*[A-Za-z][\w_-]*\z+|\A\z/
    # binding.break
    #英数字のみの入力を許可する。空白も許可
    VALID_NO_REGEX = /\A[A-Za-z0-9]+|\A\z/
    validates :rac_no_1, presence: true,length: { maximum:15}, format: { with: VALID_NO_REGEX }
    validates :rac_no_3, length: { maximum:15}, format: { with: VALID_NO_REGEX }
#この部分
    validates :rac_no_2, length: { maximum:15},format: { with: VALID_NO_REGEX }, if: :require_validation?
    
    private
    def require_validation?
        # binding.break
        if rac_no_2.blank? && rac_no_3.present? 
　　　　#このメッセージが2つ表示される
            errors.add(:base, '棚番③を入力するときは棚番②を入力してください')
        end
    end
    # binding.break
end

補足情報（FW/ツールのバージョンなど）

ruby 3.1.2
Rails6

Comments

[maisumakun]

「この部分」は、実際にコードへそう書いているのでしょうか？

（断片的でコードが正しくないように見受けられます）

[HisanoriSato]

ありがとうございます。コード修正しました。このように書いています。

Answer 1

require_validation?の名前・内容・使い方が一致していないようです。

• バリデーションをかけるかどうかの判定メソッドにしたいなら、内部でerrors.addは行わず、true/falseを返す
• バリデーションメソッドとして使いたいなら、（名前も変更すべきだし）validateを使って呼び出す形にする