ActiveRecord 异常未获救

Question

我有以下代码块：

unless User.exist?(...)
  begin
    user = User.new(...)
    # Set more attributes of user
    user.save!
  rescue ActiveRecord::RecordInvalid, ActiveRecord::RecordNotUnique => e
    # Check if that user was created in the meantime
    user = User.exists?(...)
    raise e if user.nil?
  end
end

正如您可能猜到的那样，原因是多个进程可能会同时调用此方法来创建用户（如果它尚不存在），因此当第一个进程进入该块时并开始初始化一个新用户，设置属性，最后调用 save!，该用户可能已经创建了。 在这种情况下，我想再次检查用户是否存在，并且仅在仍然不存在时才引发异常（=如果同时没有其他进程创建它）。

问题是，保存时会定期引发 ActiveRecord::RecordInvalid 异常！并且没有从救援区获救。 有什么想法吗？

编辑：

好吧，这很奇怪。我一定是错过了什么。我根据 Simone 的提示重构了代码，如下所示：

unless User.find_by_email(...).present?
  # Here we know the user does not exist yet
  user = User.new(...)
  # Set more attributes of user
  unless user.save
    # User could not be saved for some reason, maybe created by another request?
    raise StandardError, "Could not create user for order #{self.id}." unless User.exists?(:email => ...)
  end
end

现在我得到了以下异常：

ActiveRecord::RecordNotUnique: Mysql::DupEntry: Duplicate entry '[email protected]' for key 'index_users_on_email': INSERT INTO `users` ...

在“除非 user.save”所在的行中抛出。 怎么可能呢？ Rails 认为可以创建用户，因为电子邮件是唯一的，但 Mysql 唯一索引会阻止插入？这有多大可能？如何避免呢？

Answer 1

在这种情况下，您可能希望使用迁移在用户表键上创建唯一索引，这样数据库就会引发错误。

另外，不要忘记在用户模型中添加 validates_uniqueness_of 验证。

验证并不总是可以防止重复数据（两个并发请求在同一毫秒写入的可能性确实很小）。
如果您将 validates_uniqueness_of 与索引结合使用，则不需要所有这些代码。

unless User.exist?(...)
  begin
    user = User.new(...)
    # Set more attributes of user
    user.save!
  rescue ActiveRecord::RecordInvalid, ActiveRecord::RecordNotUnique => e
    # Check if that user was created in the meantime
    user = User.exists?(...)
    raise e if user.nil?
  end
end

变成

user = User.new(...)
# Set more attributes of user
if user.save
  # saved
else
  # user.errors will return
  # the list of errors
end

Answer 2

Rails 验证无法检测数据库中的竞争条件；我们使用的解决方案是还添加数据库约束。

以下是我们关于此内容的简短链接页面： Rails ActiveRecord 验证：验证种族的唯一性