在Rails中保存模型中的记录之前,执行检查重复项的功能

Mad*_*han 2 ruby ruby-on-rails rails-activerecord

这就是我的模型中的内容.我想检查重复记录,我不能只为字段使用唯一索引,因为每个作业的联系人是唯一的)我无法使用以下代码执行此操作.我做错了吗?任何帮助表示赞赏.谢谢!

before_save :check_for_duplicates?

def check_for_duplicates?
  JobsPeople.find(:all, :conditions => ["job_id = ? and contact_id = ?", self.job_id, self.contact_id])
end
Run Code Online (Sandbox Code Playgroud)

aNo*_*ble 7

我相信这应该做你想要的

class JobsPeople
  validates_uniqueness_of :contact_id, scope: :job_id
end
Run Code Online (Sandbox Code Playgroud)


Clu*_*ter 6

回调除了运行您告诉他们运行的代码之外,不会做任何事情.如果要阻止执行某些操作,则有两个选项,请使用验证或引发错误.

在您当前的代码中,您可以检查您运行的查询是否返回任何内容,然后如果出现则引发错误.该错误将冒泡到包装保存并触发回滚的事务,然后将错误传播到您可以根据需要处理它的代码中.

before_save :check_for_duplicates?

def check_for_duplicates?
  if JobsPeople.find(:all, :conditions => ["job_id = ? and contact_id = ?", self.job_id, self.contact_id]).any?
    raise 'some error'
  end
end
Run Code Online (Sandbox Code Playgroud)

虽然已经有一个内置的唯一性验证器来处理这种情况

validates :job_id, :uniquness => {:scope => :contact_id, :message => 'must be unique per contact}
Run Code Online (Sandbox Code Playgroud)

当然,由于存在潜在的竞争条件,因此仍应使用数据库中的复合唯一索引进行备份,其可能性很大程度上取决于应用程序的行为.

# in a migration

add_index :table_name, [:job_id, :contact_id], :unique => true, :name => 'index_job_id_and_contact_id_on_table_name'
Run Code Online (Sandbox Code Playgroud)