51

パフォーマンスを向上させるためにデータの非正規化を行い、ブログ投稿が受け取った投票の合計を Post モデル内に入れたいと考えています。

class Post(models.Model):
    """ Blog entry """
    author          = models.ForeignKey(User)
    title           = models.CharField(max_length=255)
    text            = models.TextField()
    rating          = models.IntegerField(default=0) # here is the sum of votes!

class Vote(models.Model):
    """ Vote for blog entry """
    post            = models.ForeignKey(Post)
    voter           = models.ForeignKey(User)
    value           = models.IntegerField()

もちろん、Post.rating値を実際に保つ必要があります。通常はデータベース トリガーを使用しますが、今回はpost_save(データベースの処理時間を短縮するために) シグナルを送信することにしました。

# vote was saved
@receiver(post_save, sender=Vote)
def update_post_votes(sender, instance, created, **kwargs):
    """ Update post rating """
    if created:
        instance.post.rating += instance.value
        instance.post.save()
    else:
        # if vote was updated, we need to remove the old vote value and add the new one
        # but how...?

保存前のインスタンス値にアクセスするにはどうすればよいですか? データベーストリガーでは、これOLDをNEW事前定義しますが、post_save シグナルにこのようなものはありますか?

アップデート

マークの答えに基づく解決策：

# vote was saved
@receiver(pre_save, sender=Vote)
def update_post_votes_on_save(sender, instance, **kwargs):
    """ Update post rating """
    # if vote is being updated, then we must remove previous value first
    if instance.id:
        old_vote = Vote.objects.get(pk=instance.id)
        instance.post.rating -= old_vote.value
    # now adding the new vote
    instance.post.rating += instance.value
    instance.post.save()
4

3 に答える 3

73

post_save変更されていないバージョンを取得するには遅すぎると思います。名前が示すように、データはその時点ですでにデータベースに書き込まれています。pre_save代わりに使用する必要があります。その場合、pk：を介してデータベースからモデルを取得old = Vote.objects.get(pk=instance.pk)し、現在のインスタンスと前のインスタンスの違いを確認できます。

于 2011-04-07T15:00:29.330 に答える