Ruby on Railsカスタムモデルバリデーションの注文を変更する

Question

ローカルシステムで正しく動作するモデルバリデーションを持つフォームがありますが、ライブサイトでチェックすると、モデルのバリデーションオーダーの順序は変更されますが、コードは同じですどちらも。

この

は、モデル内のコードのブロックです：

def validate 

    #email validation 
    if !email.blank? 
     #errors.add(:email,I18n.t(:ismissing)) 
     #else 
     if email != email_confirmation 
     errors.add(:email,I18n.t(:ErrorMessageConfirmEmailNotmatch)) 

     else 
     if email.length <=200 then 
      if email.match(/^[^@][\w.-]*@[\w.-]+[.][a-z]{2,4}$/i).nil? 
      errors.add(:email,I18n.t(:ErrorMessageInvalid)) 
      else 

      if @new_record==true 
       if User.find(:all, :conditions => ['lower(email) = ?', email.downcase]).count>0 
       #errors.add(:email," ID already exists. Provide another Email ID") 
       errors.add(:email,I18n.t(:ErrorMessageAlreadyExists)) 
       end 
      else 
       if @changed_attributes["email"]!=nil 
       if User.User.find(:all, :conditions => ['lower(email) = ?', email.downcase]).count>0 
        #errors.add(:email," ID already exists. Provide another Email ID") 
        errors.add(:email,I18n.t(:ErrorMessageAlreadyExists)) 
       end 
       end 
      end 


      end 
     else 
      errors.add(:email, I18n.t(:ErroeMessageMustlessthen,:size=>200)) 
     end 
     end 
    else 
     errors.add(:email,I18n.t(:ismissing)) 
    end 
    #end : Email validation 

    if email_confirmation.blank? 
     errors.add(:email_confirmation,I18n.t(:ismissing)) 
    end 

    #pasword validation 
    if @new_record==true 
     if password.blank? 
     errors.add(:password,I18n.t(:ismissing)) 
     else 
     if password_confirmation != password 
      errors.add(:password,I18n.t(:ErrorMessageConfirmPasswordNotmatch)) 
     end 
     if !password.nil? 
      if password.length < 4 || password.length > 50 then 
      errors.add(:password,I18n.t(:ErroeMessageShouldBetween,:from=>"4",:to=>"50")) 

      end 
      errors.add(:password,I18n.t(:ErrorMessageInvalidPassword)) if password.match('^[a-z0-9@#*-_]*$').nil? 
     end 
     end 
    end 
    #end password validation 

    if @new_record==true 
    if password_confirmation.blank? 
     errors.add(:password_confirmation,I18n.t(:ismissing)) 
    end 
    end 

    if dob.blank? 
     errors.add(:dob,I18n.t(:ErrorMessageInvalid)) 
    else 
     begin 
     #dt = DateTime.strptime(dob, "%m/%d/%Y").to_date 
     if dob.year <= 1900 then 
      errors.add(:dob,I18n.t(:ErrorMessageInvalidYear)) 
     end 
     if dob>=Date.today then 
      errors.add(:dob,I18n.t(:ErroeMessageInvalidBirthday)) 
     end 

     rescue Exception => ex 
     #errors.add(:dob,'is Invalid (MM/DD/YYYY format)') 
     errors.add(:dob,I18n.t(:ErroeMessageInvalidBirthday)) 
     end 
    end 

    end 

と誰もが任意の提案やアイデアを持っている場合、コントローラが必要ですregistration.An緊急援助のValidateメソッドを呼び出します。 ありがとうございます

Answer 1

なぜこれらが順番に実行されないのか分かりません。あなたはプロダクションでそれを示すために何かを記録しましたか？

すべてを大規模な検証メソッドに入れるのではなく、いくつかの方法（おそらく一般的にはより良い方法）に分割し、必要な順序で呼び出します。

例えば、

before_save :validate_email, :validate_dob 


def validate_email 
... 
end 

def validate_dob 
... 
end 

Answer 2

デフォルトvalidations..Iは、電子メールのためにしたと、ここで、あなたにサンプルを与えたレールを使用することができます。..

validates :email, 
     :presence =>{ :message => I18n.t(:ismissing)}, 
     :length => {:allow_blank => true, :maximum => 200, :message => I18n.t(:ErroeMessageMustlessthen,:size=>200)}, 
     :format => {:allow_blank => true, :with => /^[^@][\w.-]*@[\w.-]+[.][a-z]{2,4}$/i, :message => I18n.t(:ErrorMess 
     :uniqueness => {:allow_blank => true, :message => I18n.t(:ErrorMessageAlreadyExists)}, 
     :confirmation => {:message => I18n.t(:ErrorMessageConfirmEmailNotmatch)} 

同様に、あなたはまた、他の分野のために行うことができます。