17

如果我的问题被证明是愚蠢的，我深表歉意，但我对 Django 很陌生，我无法在任何地方找到答案。

我有以下模型：

class BlackListEntry(models.Model):
  user_banned = models.ForeignKey(auth.models.User,related_name="user_banned")
  user_banning = models.ForeignKey(auth.models.User,related_name="user_banning")

现在，当我尝试创建这样的对象时： 

BlackListEntry.objects.create(user_banned=int(user_id),user_banning=int(banning_id))

我收到以下错误：

Cannot assign "1": "BlackListEntry.user_banned" must be a "User" instance.

当然，如果我用这样的东西替换它：

user_banned = User.objects.get(pk=user_id)
user_banning = User.objects.get(pk=banning_id)
BlackListEntry.objects.create(user_banned=user_banned,user_banning=user_banning)

一切正常。问题是：

我的解决方案是否命中数据库以检索两个用户，如果是，是否可以避免它，只传递 id？

4

2 回答 2

25

你的问题的答案是：是的。

Django 将（至少）3 次访问数据库，2 次检索两个 User 对象，第三次提交您想要的信息。这将导致绝对不必要的开销。

你试一试：

BlackListEntry.objects.create(user_banned_id=int(user_id),user_banning_id=int(banning_id))

这些是 Django ORM 生成的 FK 字段的默认名称模式。这样您可以直接设置信息并避免查询。

如果要查询已保存的 BlackListEntry 对象，可以使用双下划线导航属性，如下所示：

BlackListEntry.objects.filter(user_banned__id=int(user_id),user_banning__id=int(banning_id))

这是您在 Django 查询集中访问属性的方式。带有双下划线。然后你可以比较属性的值。

虽然非常相似，但它们的工作方式完全不同。第一个直接设置属性，而第二个由django解析，在'__'处拆分它，并以正确的方式查询数据库，第二部分是属性的名称。

您始终可以与实际的用户对象进行比较user_banned，user_banning而不是他们的 ID。但是，如果您还没有随身携带这些物品，那么这样做是没有用的。

希望能帮助到你。

于 2012-10-01T23:10:39.827 回答
0

我确实相信，当您获取用户时，它会访问数据库...

为避免这种情况，您必须使用此处描述的方法编写原始 sql 来进行更新：

https://docs.djangoproject.com/en/dev/topics/db/sql/

如果您决定走这条路，请记住您有责任保护自己免受 sql 注入攻击。

另一种选择是缓存 user_banned 和 user_banning 对象。

但很可能，简单地抓住用户并创建 BlackListEntry 不会给您带来任何明显的性能问题。缓存或执行原始 sql 只会带来一点好处。在这成为问题之前，您可能会遇到其他问题。

于 2012-10-01T22:55:49.467 回答