在添加对象中使用 ManyToMany 字段 raise 需要为字段“id”设置一个值，然后才能使用这种多对多关系

Question

我有两个模型：Exam 和 Question。每个问题都有在每次考试中自动计算的分数。

这是我的文件：

#Models.py
class Exam(models.Model):
    questions = models.ManyToManyField(Question)
    title = models.CharField()
class Question(models.Model):
    title = models.CharField()
    answer = models.TextField()
    points = models.PositiveSmallIntegerField()

#Forms.py
class ExamForm(ModelForm):
    class Meta:
        model = Exam
        fields = '__all__'

#Views.py
if form.is_valid():
    new_exam = form.save(commit=False)
    # Some modify goes here. 
    new_exam.save()
    form.save_m2m()
    return redirect('view_exam')

我确实自定义了Exam模型的save()方法：

 def save(self, *args, **kwargs):
        self.point = 0
        for question in self.questions:
                self.point += question.point
        super(Exam, self).save(*args, **kwargs)

但我得到了这个错误：

"<Exam: NewObject>" needs to have a value for field "id" before this many-to-many relationship can be used.

我怎样才能做到这一点而不引发任何错误？

我的目标：对于每一个新创建的考试，计算本次考试的题目分数，并将它们放入Exam模型的points字段中。

Answer 1

将可以从数据库中的其他字段/表计算出来的东西保存到模型中绝不是一个好主意，尤其是当这取决于其他模型时。例如，当您在某个阶段创建新的Question 时，太容易忘记更新值。你只会在你的数据库中得到不一致。

删除您的自定义save() 方法，因为它没有任何作用。

如果您想知道总点数，请在 Exam 上添加一个自定义 getter，它会即时计算：

#At the first of models.py -> from django.db.models import Sum 
@property
def points(self):
    if self.pk:
       return self.questions.all().aggregate(Sum('points'))['points__sum']
    else:
       return 0

或者用你的总结：

@property
def points(self):
    if self.pk:
        point = 0
        questions = self.questions.all()
        for question in questions :
            point += question.point
        return point
    else:
        return 0

使用此属性，您可以在代码中的任何位置执行exam.points，它将是最新的。