ActiveRecord自定义验证问题

Question

我的RoR模型验证有问题:

def save
  self.accessed = Time.now.to_s
  self.modified = accessed
  validate_username
  super
end

def validate_username
  if User.find(:first, :select => :id, :conditions => ["userid = '#{self.userid}'"])
    self.errors.add(:userid, "already exists")
  end
end

正如您所看到的,我已经用自己的方法替换了Model的save方法,在调用父.save方法之前调用validate_username.我的问题是,即使添加了错误,Rails仍会尝试将新行插入数据库,即使用户名是重复的.我在这做错了什么？

PS:我没有使用,validate_uniqueness_of因为以下问题具有区分大小写:https://rails.lighthouseapp.com/projects/8994/tickets/2503-validates_uniqueness_of-is-horribly-inefficient-in-mysql

更新:我尝试了weppos解决方案,它可以工作,但并不像我想的那样.现在,该字段被标记为不正确,但仅当所有其他字段都正确时.我的意思是,如果我输入一个错误的电子邮件地址,例如,电子邮件字段标记为es faulty,userid字段不是.当我提交正确的电子邮件地址时,userid字段被标记为不正确.希望你们明白我的意思:D

Update2:应该以某种方式验证数据,不应该将重复的用户ID插入数据库,不区分大小写.用户id具有"用户域"格式,例如."test-something.net".不幸的是,validates_uniqueness_of :userid它不起作用,它试图将"test-something.net"插入到数据库中,即使已经存在"Test-something.net".validate_username应该是我(快速)解决此问题的方法,但它不起作用.weppos解决方案确实有效,但并不像我想要的那样(正如我在第一次更新中所解释的那样).

还没想出来......有人吗？

最好的问候,x3ro

Answer 1

为什么不使用回调并保持保存方法不变？另外,避免直接SQL值插值.

class ... < ActiveRecord::Base

  before_save :set_defaults
  before_create :validate_username

  protected

  def set_defaults
    self.accessed = Time.now.to_s
    self.modified = accessed
  end

  def validate_username
    errors.add(:userid, "already exists") if User.exists?(:userid => self.userid)
    errors.empty?
  end

end