Django 1.10+ 中发票计算的“最佳实践”是什么？

Question

如果一个有两个模型：

class Invoice(models.Model):
    class Meta:
        ordering = ('-date_added', )

    number = models.CharField(max_length=10,)
    comments = models.TextField(blank=True, help_text="Notes about this invoice." )
    total = models.DecimalField(max_digits=9, decimal_places=2, default="0" )
    date_added = models.DateTimeField(_('date added'), auto_now_add=True)
    date_modified = models.DateTimeField(_('date modified'), auto_now=True)

def __unicode__(self):
    return "%s: total %s" % (self.number, self.total)

class Part(models.Model):
    for_invoice = models.ForeignKey(Invoice)
    description = models.CharField(max_length=200, blank=True, help_text=_("Briefly describe the part.") )
    supplier = models.CharField(max_length=100, blank=True, help_text=_("Supplier Name.") )
    supplier_number = models.CharField(max_length=100, blank=False, help_text=_("Supplier's order number.") )
    qty = models.DecimalField(max_digits=3, decimal_places=0, blank=False, null=False, help_text=_("How many are needed?") )
    cost = models.DecimalField(max_digits=7, decimal_places=2, blank=True, null=True, help_text=_("Price paid per unit") )
    line_total = models.DecimalField(max_digits=9, decimal_places=2, default="0")
    date_added = models.DateTimeField(auto_now_add=True)
    date_modified = models.DateTimeField(auto_now=True)

def __unicode__(self):
    return "%s: total %s" % (self.for_invoice, self.line_total)

我看到的第一个选择是可以将“line_total”或“total”实现为计算模型字段。但是，如果您这样做，那么您将永远无法按“line_total”或“total”对更改列表进行排序，而用户希望能够这样做。所以我把它们做成模型上的一个保存字段。

阅读 Django 1.10 文档，我看到 4 个地方可以定义代码来计算和更新“total”和“line_total”字段：

1. ModelAdmin.save_model(request, obj, form, change) 和 ModelAdmin.save_formset(request, form, formset, change)
2. ModelAdmin.save_related（请求、表单、表单集、更改）
3. model.save(self, *args, **kwargs))
4. 保存信号

在我看来，保存信号的级别太低 - 与单个模型保存事件相关联，它永远无法访问请求或会话。所以把所有的“Part”记录都收集起来会很“烦人”吗？

同样，model.save 方法似乎也过于细化。如果我可以使用 Invoice.save 方法将是“方便的”，因为它可以通过 Invoice.part_set.all() 轻松访问所有相关部分，循环遍历该查询集将是更新 line_total 的一种简单方法，然后是主全部的。但是，此时是否会保存所有新的/更改的零件记录？而且它无法访问 HttpRequest。

我认为这同样适用于管理员的 save_model。但是，我有一个模糊的记忆，可以确保先保存相关部分。我在 Admin 的 Invoice 添加/编辑页面上使用内联列出了这些部分。

我刚刚在 ModelAdmin.save_related 中添加了！我忘了那个。由于此时将保存主发票，也许这是更新所有 Part 记录，然后更新父总计的最佳位置？

提前致谢！

Answer 1

感谢 Alasdair 的建议，但是，该客户不想要完整的自定义解决方案。所以我坚持使用管理调整。

我正在使用选项 2 进行测试：save_related 管理员的功能。代码如下：

from decimal import Decimal

  def save_related(self, request, form, formsets, change):
    total = Decimal('0.00')
    for myform in formsets:
      for mf in myform.forms:
        # Skip empty rows (if any):
        if len(mf.cleaned_data) > 0:
          # Update item line total from item quantity and price:
          mf.instance.line_total = mf.instance.qty * mf.instance.cost
          # Add line item total to grand total:
          total += mf.instance.line_total
          # re-save line item:
          mf.instance.save()
    # Update grand total on the invoice:
    form.instance.total = total
    # re-save the invoice record:
    form.instance.save()
    # Chain to inherited code:
    super(InvoiceAdmin, self).save_related(request, form, formsets, change)

很抱歉变量名不好。内联表格有 2 层深，让我措手不及。我猜第一层是一组表单集（只有一个），第一个表单集里面是我的表单。一切都有 .instance 对象，我认为这表明它们已经被保存。一个惊喜，因为我以为我读到这个函数的默认操作是保存表单集（主表单已经保存）。

无论如何，它有效，所以我假设我误解了文档。