Question

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

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

Answer 1

我相信这应该做你想做的事情

class JobsPeople
  validates_uniqueness_of :contact_id, scope: :job_id
end

Answer 2

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

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

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

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

validates :job_id, :uniquness => {:scope => :contact_id, :message => 'must be unique per contact}

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

# in a migration

add_index :table_name, [:job_id, :contact_id], :unique => true, :name => 'index_job_id_and_contact_id_on_table_name'

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

2 个答案: