Question

这是models.py

class ledger1(models.Model):
    User = models.ForeignKey(settings.AUTH_USER_MODEL,on_delete=models.CASCADE,null=True,blank=True)
    Company = models.ForeignKey(company,on_delete=models.CASCADE,null=True,blank=True,related_name='Companys')
    Creation_Date = models.DateField(default=datetime.now)
    name = models.CharField(max_length=32,unique=True)
    Opening_Balance = models.DecimalField(max_digits=19,decimal_places=2)
    Closing_balance = models.DecimalField(max_digits=10,decimal_places=2)


class journal(models.Model):
    User = models.ForeignKey(settings.AUTH_USER_MODEL,on_delete=models.CASCADE,null=True,blank=True)
    Company = models.ForeignKey(company,on_delete=models.CASCADE,null=True,blank=True,related_name='Companyname')
    Date = models.DateField()
    By = models.ForeignKey(ledger1,on_delete=models.CASCADE,related_name='Debitledgers')
    To = models.ForeignKey(ledger1,on_delete=models.CASCADE,related_name='Creditledgers')
    Debit = models.DecimalField(max_digits=10,decimal_places=2,)
    Credit = models.DecimalField(max_digits=10,decimal_places=2)

我想使用django查询建立数学方程式，我的方程式结果将显示在ledger1.Closing_Balance字段中

所以我尝试了这个：

@receiver(pre_save, sender=ledger1)
def update_user_closing_balance(sender,instance,*args,**kwargs):
    Closing_balance = ledger1.objects.annotate(debitsum=Sum('Debitledgers__Debit')) + instance.Opening_Balance - ledger1.objects.annotate(creditsum=Sum('Creditledgers__Credit'))
    instance.Closing_balance = Closing_balance

在Django中可以吗？？？因为如果运行此命令，将会收到不受支持的操作数错误...

是否有其他替代代码？

如果有人知道...请帮助

提前谢谢

Answer 1

不受支持的操作数异常与查询本身无关，但与您编写的表达式无关：

Closing_balance = (
   ledger1.objects.annotate(debitsum=Sum('Debitledgers__Debit')) +
   instance.Opening_Balance -
   ledger1.objects.annotate(creditsum=Sum('Creditledgers__Credit'))
)

因此，您在此处将QuerySet与真实值一起添加，这没有任何意义。您可能想使用的是一个.aggregate(..)，然后将其中包含的值包装起来，例如：

debitsum = ledger1.objects.aggregate(debitsum=Sum('Debitledgers__Debit'))['debitsum']
creditsum = ledger1.objects.aggregate(creditsum=Sum('Creditledgers__Credit'))['creditsum']

Closing_balance = debitsum + instance.Opening_Balance - creditsum

但是，要说的是，使用信号来预先计算聚合通常不是一个好主意。例如，由于journal对象可以更改其Debit或Credit值，并且这不会“触发”信号，因此不会进行任何更新。即使您还为此类事件添加逻辑，信号仍然可能发生，因为例如批量更新将超过信号系统。

通常最好不要存储数据集合，因为这会导致data duplication，就像文章所说的“ 产生冗余和不一致。”。如果要计算这样的聚合，最好在数据库级别使用（物化）视图。

编辑：查询本身似乎没有任何意义。如果您更新ledger记录，则可以执行过滤并以这种方式计算更新：

@receiver(pre_save, sender=ledger1)
def update_user_closing_balance(sender,instance,*args,**kwargs):
    debit = instance.Debitledgers.aggregate(debit=Sum('Debit'))['debit']
    credit = instance.Creditledgers.aggregate(credit=Sum('Credit'))['credit']
    instance.Closing_balance = instance.Opening_Balance + debit - credit

但这可能仍然不够用，因为您随后需要对日期进行适当的过滤，等等。

我可以在Django pre_save信号中执行查询吗？

1 个答案: