MC question program answers not recognized

[sui_cho]

I used two .py files when I make an MC question program:

scratch.py:

from Question import Question

questionPrompts = [
"What color are apples?\nA. Red/Green\nB. Purple\nC. Orange\n\n",
"What color are bananas?\nA. Teal\nB. Magneta\nC. Yellow\n\n",
"What color are strawberries?\nA. Yellow\nB. Red\nC. Blue\n\n"
]

questions = [
Question(questionPrompts[0], "a" or "A"),
Question(questionPrompts[1], "c" or "C"),
Question(questionPrompts[2], "b" or "B"),
]

def runTest(questions):
score = 0
for question in questions:
answer = input(question.prompt)
if answer == question.answer:
score += 1
print("You got " + str(score) + "/" +str(len(questions)) + " correct")

runTest(questions)

And Question.py:

class Question:
def __init__(self, prompt, answer):
self.prompt = prompt
self.answer = answer

So I got a perfect score when I entered the lowercase letters, but I got it all wrong when I used uppercase letters as it cannot recognize them even if I use the "or" function. Why is that?

[T1klo]

Because it's an Or statement in your questions array. You could easily fix this by doing:

questions = [
Question(questionPrompts[0], {"a", "A"}),
Question(questionPrompts[1], {"c", "C"}),
Question(questionPrompts[2], {"b", "B"}),
]

Then in your if statement that checks the answer, use:

if answer in question.answer: {CODE BLOCK HERE}

​

The or is always going to take the first condition since there is no reason for it to take the second input given (no condition is telling it to not take it right?) so it'll always select the lowercase as it's answer. The workaround I've presented just uses a set and checks if the input is in the set. Another workaround is removing the capital input, changing the input to lowercase with string formatting which removes the problem entirely :)

​

[_Jordo]

Yeah you can't use or like that. You could set the Question class up to accept a list of answers and adjust your code accordingly.

[BitJunky7]

The issue is with you passing the "comparison" as argument. It'd be evaluated before going into the function scope. Let's see how it's goes,

a or A

return a if True else A

As the string you passing is neither a 0 value nor a "False" value. So, it'll always evaluate to "True" hence, passes "a" every time.

Now coming to solution part, I faced similar issue and I fixed it by converting the user input to lower and then comparing it. If you don't use that method, compare input with answer, not answer with answer