if-else statement in python django

Question

I am new to Django and I have a problem that I couldn't solve. I am trying to display a specific question and other related attribute from my Question model based on a field from the Participant model. The issue here is that it directly goes to the else statement even when the condition is true.I tried to print(participant.condition) and it works so I am not sure why its not working with the if statement.

@login_required
def LPSC_VIEW1(request):
    participant=request.user.participant
    if participant.condition == 'LPN':
        First_question= Question.objects.get(id=1)
        all_choices = First_question.choices.all()
        context = {'First_question': First_question, 'all_choices': all_choices}
        return render(request, 'study/FirstQN.html', context)
    else:
        First_question= Question.objects.get(id=12)
        all_choices = First_question.choices.all()
        context = {'First_question': First_question, 'all_choices': all_choices}
        return render(request, 'study/FirstQSC.html', context)

my models as the following:

class Question(models.Model):
    question_text = models.CharField(max_length=200)
    caption = models.CharField(max_length=200, default="this is a caption")
    choices = models.ManyToManyField(Choice)
    vis_image = models.ImageField(default= "this is an image", null=False, blank=False, upload_to="static/study/img")
    def __str__(self):
        return self.question_text


class Condition(models.Model): 
    name = models.CharField(max_length=200)

    def __str__(self):
        return self.name


class Participant(models.Model): 
    user = models.OneToOneField(User, on_delete=models.CASCADE, null=True)
    created_at = models.DateTimeField(auto_now_add=True)
    updated_at = models.DateTimeField(auto_now=True)
    condition = models.ForeignKey(Condition, on_delete=models.CASCADE)
    score = models.IntegerField(default=0)

    def __str__(self):
        return self.user.username

does if participant.condition.name == 'LPN': work?

RickyA
– RickyA

2020-03-31 13:40:49 +00:00
Commented Mar 31, 2020 at 13:40 — RickyA
– RickyA, Commented Mar 31, 2020 at 13:40

Peter DeGlopper · Accepted Answer · 2020-03-31 14:01:13Z

2

condition is a foreign key, not a string. You're comparing it against 'LPN', but no instance of your Condition model will be equal to that string.

Try if participant.condition.name == 'LPN': to compare the name field on the Condition instance to that string.

Your print statement shows them as apparently being the same because you've defined how to present Condition instances as strings with your __str__ method - it will print the name for the Condition instance, but that doesn't mean that the Condition value is actually equal to that string.

edited Mar 31, 2020 at 14:01

answered Mar 31, 2020 at 13:41

Peter DeGlopper

37.5k7 gold badges95 silver badges88 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

shoytov · Accepted Answer · 2020-03-31 13:40:03Z

2

You must change this:

participant=request.user.participant

to:

participant=Participant.objects.get(user=request.user)

answered Mar 31, 2020 at 13:40

shoytov

6055 silver badges10 bronze badges

Comments

Vikrant Chauhan · Accepted Answer · 2020-03-31 13:41:54Z

0

You might have to use

from .models import Participant
participant = Participant.objects.get(user = request.user)

answered Mar 31, 2020 at 13:41

Vikrant Chauhan

439 bronze badges

3 Comments

London-35 Over a year ago

already on my view and participant has been created in another view

Vikrant Chauhan Over a year ago

Django queries the database at the time of render, if it is in a different view it won't be accessed in another, you have to query it everytime you need it

London-35 Over a year ago

thanks, but this did not solve the problem

Collectives™ on Stack Overflow

if-else statement in python django

3 Answers 3

Comments

Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related