Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

JavaScript JavaScript Loops, Arrays and Objects Tracking Multiple Items with Arrays Build a Quiz Challenge, Part 1 Solution

Daniel Lee
PLUS
Daniel Lee
Courses Plus Student 566 Points

Counter not working

Everything works except the count doesn't go up. It seems like my code should work but I may be missing something. Please help.

https://w.trhou.se/1zvnnweuz0

2 Answers

andren
andren
28,507 Points

There are two issues:

The first is with this line:

response = parseInt(prompt(question));

You parse the response as an int, but the answers to the questions are not numbers but words. Words can't be converted to numbers so this code will just convert all responses into NaN (Not a Number).

The second issue is in your if statement:

if (response === answer) {
    var correctAnswers = 0;
    correctAnswers += 1;
}

You redeclare the correctAnswer variable and set it equal to 0. Since you do this every time a question is answered correctly correctAnswer will be reset to 0 every time. That first line is not needed at all since the correctAnswer variable already exists outside the if statement.

Here is the fixed code:

var questions = [
  ['What technique is used to remove weight?', 'layers'],
  ['What techique is used to build weight?', 'graduation'],
  ['What technique is used to create weight?', 'one length']
];
var question;
var answer;
var response;
var html;
var correctAnswers = 0;



for (var i = 0; i < questions.length; i++) {
  question = questions[i][0];
  answer = questions[i][1];
  response = prompt(question); // Removed parseInt
  if (response === answer) {
    // Removed correctAnswer redeclaration
    correctAnswers += 1;
  }
}

html = "You got " + correctAnswers + " question(s) correct";

function print(message) {
  document.write(message);
}
print(html);
clemhlrdt
clemhlrdt
20,534 Points

In your code, your if statement is always false. Plus, you reset the counter to 0 each time you loop.

response = parseInt(prompt(question)); //parseInt ? You should compare a string to another string ;)
  if (response === answer) {
    correctAnswers += 1;
  }