Warum werden meine Daten nicht aktualisiert?

Question

Dies ist ein Modell für die Datenbank, der Benutzer ändert die Wichtigkeit, und je nach der Ebene, die der Benutzer wählt, wird eine Punktzahl zugewiesen aber alles, was es zeigt, ist 300 für my_points und 0 für ihre_points, egal was gewählt wird.

from django.db import models 
from django.conf import settings 
from django.db.models.signals import post_save 
# Create your models here. 
class Question(models.Model): 
    text = models.TextField() 
    active = models.BooleanField(default = True) 
    draft = models.BooleanField(default = False) 
    timestamp = models.DateTimeField(auto_now_add=True, auto_now=False) 
    #answers = models.ManyToManyField('Answer') 

    def __unicode__(self): 
     return self.text[:10] 


class Answer(models.Model): 
    question = models.ForeignKey(Question) 
    text = models.CharField(max_length=120) 
    active = models.BooleanField(default=True) 
    draft = models.BooleanField(default=False) 
    timestamp = models.DateTimeField(auto_now_add=True, auto_now=False) 


    def __unicode__(self): #def __str__(self): 
     return self.text[:10] 
LEVELS = (
    ('Mandatory', 'Mandatory'), 
    ('Very Important', 'Very Important'), 
    ('Somewhat Important', 'Somewhat Important'), 
    ('Not Important', 'Not Important'), 
    ) 

class UserAnswer(models.Model): 
    user = models.ForeignKey(settings.AUTH_USER_MODEL) 
    question = models.ForeignKey(Question) 
    my_answer = models.ForeignKey(Answer, related_name = 'user_answer') 
    my_answer_importance = models.CharField(max_length=50, choices= LEVELS) 
    my_points = models.IntegerField(default=-1) 
    their_answer = models.ForeignKey(Answer, null=True, blank=True, related_name = 'match_answer') 
    their_importance = models.CharField(max_length=50, choices= LEVELS) 
    their_points = models.IntegerField(default=-1) 
    timestamp = models.DateTimeField(auto_now_add=True, auto_now=False) 

    def __unicode__(self): 
     return self.my_answer.text[:10] 

def score_importance(importance_level): 
    if importance_level == "Mandatory": 
     points = 300 
    elif importance_level == "Very Important": 
     points = 200 
    elif importance_level == "Somewhat Important": 
     points = 50 
    elif importance_level == "Not Important": 
     points = 0 
    else: 
     points = 0 
    return points 


def update_user_answer_score(sender, instance, created, *args, **kwargs): 
    #print sender 
    print instance 
    #print created 
    if instance.my_points == -1: 
     my_points = score_importance(instance.my_answer_importance) 
     instance.my_points = my_points 
     print my_points 
     instance.save() 
    if instance.their_points == -1: 
     their_points = score_importance(instance.their_importance) 
     instance.their_points = their_points 
     print my_points 
     instance.save() 


post_save.connect(update_user_answer_score, sender=UserAnswer) 

Answer 1

Ihr Code ist unnötig kompliziert. Sie brauchen wirklich keine getrennten Felder my_points und my_answer_importance und die Art und Weise, wie Sie Auswahlmöglichkeiten verwenden, schlägt den ganzen Punkt der Verwendung von Optionen. Ich schlage vor, Sie Ihren Code wie folgt zu ändern:

LEVELS = (
    (300, 'Mandatory'), 
    (200, 'Very Important'), 
    (50, 'Somewhat Important'), 
    (0, 'Not Important'), 
    ) 

class UserAnswer(models.Model): 
    user = models.ForeignKey(settings.AUTH_USER_MODEL) 
    question = models.ForeignKey(Question) 
    my_answer = models.ForeignKey(Answer, related_name = 'user_answer') 
    my_points = models.IntegerField(default=-1, choices=LEVELS) 
    their_answer = models.ForeignKey(Answer, null=True, blank=True, related_name = 'match_answer') 
    their_points = models.IntegerField(default=-1, choices=LEVELS) 
    timestamp = models.DateTimeField(auto_now_add=True, auto_now=False) 

    def __unicode__(self): 
     return self.my_answer.text[:10] 

Jetzt brauchen Sie nicht Ihr Signal und der gesamte Codeblock entfernt werden kann.