詹戈尝试插入而不是更新保存（）

Question

我有一个模型

class DeviceAdmin(models.Model):
    dev_id = models.AutoField(db_column='dev_id', primary_key=True)
...

视图定位对象：

device = DeviceAdmin.objects.get(uuid=uuid)

然后它会做出一些改变（或者可能不会）

if (...):
    device.os_type = 'windows'
...

然后它尝试保存任何更改：

device.save()

这里是Django尝试插入另一行而不是更新的地方，导致了一个DB错误

django.db.utils.IntegrityError: duplicate key value violates unique constraint 

解决方案中有一种类似的问题：

一旦我将primary_key字段设置为自动字段，问题就消失了。

但是，我的主键已经是一个自动字段。

所以我用调试器逐步浏览了django代码，并在文件中资助了这个...站点包\django\models\base.py：

    # If possible, try an UPDATE. If that doesn't update anything, do an INSERT.
    if pk_set and not force_insert:
        base_qs = cls._base_manager.using(using)
        values = [(f, None, (getattr(self, f.attname) if raw else f.pre_save(self, False)))
                  for f in non_pks]
        forced_update = update_fields or force_update
        updated = self._do_update(base_qs, using, pk_val, values, update_fields,
                                  forced_update)
        if force_update and not updated:
            raise DatabaseError("Forced update did not affect any rows.")
        if update_fields and not updated:
            raise DatabaseError("Save with update_fields did not affect any rows.")
    if not updated:
        if meta.order_with_respect_to:
            # If this is a model with an order_with_respect_to
            # autopopulate the _order field
            field = meta.order_with_respect_to
            filter_args = field.get_filter_kwargs_for_object(self)
            order_value = cls._base_manager.using(using).filter(**filter_args).count()
            self._order = order_value

        fields = meta.local_concrete_fields
        if not pk_set:
            fields = [f for f in fields if f is not meta.auto_field]

        update_pk = meta.auto_field and not pk_set
        result = self._do_insert(cls._base_manager, using, fields, update_pk, raw)
        if update_pk:
            setattr(self, meta.pk.attname, result)
    return updated

似乎如果没有更改（因此更新不做任何事情），那么它将尝试插入重复记录。

我通过在< code>get之后添加一个保存来测试这一点:

device = DeviceAdmin.objects.get(uuid=uuid)
device.save()

它试图插入一个副本。

这是否意味着，我的代码需要跟踪一个对象是否必须保存到DB？

解决这个问题的最简单方法是什么？

更新：

作为一个测试，在用< code>get找到一个对象后，我立即执行< code>save，而不做任何更改:

device = DeviceAdmin.objects.get(uuid=uuid)
device.save()

Django应该知道这个对象是一个已经存在的对象。然而，它试图插入一个副本。

同样，创建一个新对象并调用保存两次：

device = DeviceAdmin(...) # create a new object (it has null `dev_id`)
device.save()             # insert the new object into DB; get 'dev_id'
                          #   for the newly inserted record
device.save()             # This shouldn't do anything!

第二次< code>save尝试插入一个副本——正如我可以看到的，如果我用调试器单步执行django代码。

Answer 1

可能发生这种情况的一个原因是如果您的主键字段使用默认值定义。我在您的示例中没有看到默认值，但我也没有看到数据类型是uuid，所以我想我会提供这个答案以防万一。

如果您的模型的pkey如下所示：models.UUIDField（primary_key=True，默认=uuid.uuid4），调用保存将始终导致INSERT而不是UPDATE。我相信即使您将force_update=True传递给保存方法，这也是真的。这曾经作为错误提交，但实际上是Django 3根据文档的预期行为。

如果要使用 uuid 作为主键，有几种方法可以解决此问题。最好的方法可能是遵循文档的建议，并使用更新或update_or_create方法来更新模型实例。

>>> MyModel.objects.update_or_create(pk=existing_pk, defaults={'name': 'new name'})
>>> MyModel.objects.filter(pk=existing_pk).update(name='new name')

或者，您也可以从主键定义中删除默认参数，并通过覆盖模型的save方法来确保每个实例在保存时都有一个分配的uuid例如:

def save(self, *args, **kwargs):
    if not self.id:
        self.id = uuid.uuid1()
        super(DeviceAdmin, self).save(*args, **kwargs)

Answer 2

在我的例子中，我在班上有一个独特的CharField。在我看来，Django有时会将它视为主键，有时不会——结果是有时没有意识到它应该是更新而不是插入。

添加显式的id=AutoField（primary_key=True）为我解决了问题。

Answer 3

您在字段定义方面肯定有一些问题(可能出于某种原因，您应该添加< code>unique=True)。但是如果您仍然不能确定更新失败的原因(如果< code>UPDATE返回“no rows updated”，则发生< code>INSERT)，那么您可以使用< code >。保存(force_update=True)作为最后的手段。

此外，您的代码中存在一些不一致：

• 字段被命名为dev_id但您在筛选中使用 uuid （.get（uuid=uuid））。是否确实为primary_key字段设置了名称？
• AutoField假设使用整数字段，因此在get_prep_value期间，它会执行int（self.pk），其工作而不会在uuid实例上抛出任何错误。UUID，但如果 db 内部使用 varchar 字段，则它将被转换回具有错误值的 str（例如 str（int（uuid.UUID（'579aea21-49a4-4819-90ec-a192014b4a44'）） == “116447198070669671892400511866020514372”）。所以这就是可能导致问题的原因。尝试使用 UUID 字段或字符字段，而不是自动字段：

import uuid


class SomeModel(models.Model): 

    dev_id = models.UUIDField(primary_key=True, blank=True, default=uuid.uuid4)

# Or if your column is a varchar:

class SomeModel(models.Model): 

    def dev_id_default():
        return uuid.uuid4().hex  # or str(uuid.uuid4()) if you need values with `-`

    dev_id = models.CharField(primary_key=True, blank=True, default=dev_id_default, max_length=33_OR_36)

看起来您正在使用数据库中预先存在的表：尝试使用python manage.py inspectdb生成的模型的代码。