如何覆盖最旧的记录而不是创建新记录？

Question

我正在使用 Ruby on Rails 4，在 before_create 回调中，我想让它覆盖现有记录，而不是在数据库中创建新记录。也就是说，例如，当我尝试创建新记录时，我有 10 条记录存储到数据库表中，那么第 11 条记录应该覆盖最旧的记录，而不是创建新记录。

我尝试了以下代码：

before_create do
  if more_than_ten?
    # Overwriting oldest record
    oldest_record = self.class.order(:created_at).first

    self.id = oldest_record.id
  else
    true
  end
end

但我收到此错误：

ActiveRecord::RecordNotUnique (Mysql2::Error: Duplicate entry '1' for key 'PRIMARY': INSERT INTO ...)

如何覆盖最旧的记录而不是创建新记录？

Answer 1

我在这里看到 2 个选项：

• 删除最旧的记录并创建一个新记录
• 用新的属性更新最旧的记录

我们将选择第一个选项：

before_create do
  if more_than_ten?
    oldest_record = self.class.order(:created_at).last.destroy
  else
    true
  end
end

短版：

before_create do
  self.class.order(:created_at).last.destroy if more_than_ten?
  true # if more_than_ten? return false, it will also return false for this before_create
end

为什么使用第一个选项？因为如果我们更新“最旧”的记录，下次我们添加新记录时，它会更新相同的记录（我们依赖created_at字段）。

我们可以使用updated_at 字段来依赖，但正如@NickVeys 所说，你在before_create 回调中，这意味着如果我们用新记录的属性更新最旧的记录，我们不会实际上创造了一个记录。它正在改变 Rails 的默认行为（更新而不是创建），并可能导致混乱（太多的黑魔法甚至会迷惑最优秀的巫师！）。

---

第二个选项：我认为这是行不通的，因为 before_create 回调不会为创建的对象返回 true 或有效属性：

before_create do
  if more_than_ten?
    oldest_record = self.class.order(:created_at).last # get the record with the lowest created_at

    oldest_record.update_attributes(self.attributes.merge({ created_at: DateTime.current }))
    false # returns false to stop the creation process of the new record
  else
    true
  end
end

如您所见，我们将created_at 字段更新为DateTime.now，以“模拟”现在创建的对象。它解决了上面暴露的问题：下次我们创建记录时，它不会使用相同的记录（因为.order(:created_at) 将新更新的对象放在最上面）而是另一个最旧的记录。

---

正如@Stefan 指出的那样，对于选项#1，我们应该在 10 号之后销毁所有记录：

before_create do
  self.class.order(:created_at).offset(10).destroy_all
end

Answer 2

如果您想使用与原始 ID 相同的 ID，那么您应该能够执行以下操作

before_create do
 if more_than_ten?
   # Overwriting oldest record
   oldest_record = self.class.order(:created_at).first
   old_id = oldest_record.id
   oldest_record.destroy
   self.id = old_id
  else
    true
  end
end

如果你想确保它在失败时不会删除旧记录，你可以确保它被包含在ActiveRecord::Base.transaction 的事务中

Answer 3

您正在执行创建，它映射到插入语句。将 ID 设置为现有记录并不会让它知道您想要什么，它只是将 ID 设置为现有记录，对于主键来说，这不是您的数据库所关心的。

您需要在插入之前删除该记录才能使其正常工作。

Answer 4

我会这样做：

before_create do
  if more_than_ten?
    self.id = self.class.oldest.id
    find(self.id).destroy
  end
  true
end

# You should use oldest scope
def self.oldest
  all.order(created_at: :desc).first
end

或者还有另一种方法可以将旧记录更新为新记录。新记录没有 id，因此您可以覆盖 ActiverRecord 的保存方法，而不是使用 before_create。

def save
  more_than_ten? ? all.oldest.update(attributes) : super
end

def self.oldest
 all.order(created_at: :desc).first
end