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

Fawad Sami
Fawad Sami
8,719 Points

I copied the code line for line from the video, but the questions are still not listed. What am I missing?

var questions = [
  ['How many states are in the United States?', 50],
  ['How many continents are there?', 7],
  ['How many legs does an insect have?', 6]
];
var correctAnswers = 0;
var question;
var answer;
var response;
var html;

var correct = [];
var wrong = [];

function print(message) {
  var outputDiv = document.getElementById('output');
  outputDiv.innerHTML = message;  
}

function buildList( arr ) {
  var listHTML = "<ol>"
  for (var i = 0; i < arr.length ; i += 1) {
    listHTML += "<li>" + arr[i] + "</li>";
  }
  listHTML = "</ol>";
  return listHTML;
}

for (var i = 0; i < questions.length; i += 1) {
  question = questions[i][0];
  answer = questions[i][1];
  response = prompt(question);
  response = parseInt(response);
  if (response === answer) {
    correctAnswers += 1;
    correct.push(question);
  } else {
    wrong.push(question);
  }
}


html = "You got " + correctAnswers + " question(s) right.";
html += "<h2>You got these questions correct:</h2>";
html += buildList(correct);
html += "<h2>You got these questions incorrect:</h2>";
html += buildList(wrong);

print(html);

2 Answers

Aakash Srivastava
Aakash Srivastava
5,415 Points

Hey Fawad Sami , You have done an awesom job . You just did a small mistake and that was:
Instead of adding </ol> string to the listHTML variable , you have assigned it to listHTML .

Here is the code :

var questions = [
  ['How many states are in the United States?', 50],
  ['How many continents are there?', 7],
  ['How many legs does an insect have?', 6]
];
var correctAnswers = 0;
var question;
var answer;
var response;
var html;

var correct = [];
var wrong = [];

function print(message) {
  var outputDiv = document.getElementById('output');
  outputDiv.innerHTML = message;  
}

function buildList( arr ) {
  var listHTML = "<ol>"
  for (var i = 0; i < arr.length ; i += 1) {
    listHTML += "<li>" + arr[i] + "</li>";
  }
  listHTML = "</ol>";          //   it should be listHTML += "</ol>"
  return listHTML;
}

for (var i = 0; i < questions.length; i += 1) {
  question = questions[i][0];
  answer = questions[i][1];
  response = prompt(question);
  response = parseInt(response);
  if (response === answer) {
    correctAnswers += 1;
    correct.push(question);
  } else {
    wrong.push(question);
  }
}


html = "You got " + correctAnswers + " question(s) right.";
html += "<h2>You got these questions correct:</h2>";
html += buildList(correct);
html += "<h2>You got these questions incorrect:</h2>";
html += buildList(wrong);

print(html);

Now , you are ready to go :)

Fawad Sami
Fawad Sami
8,719 Points

oh wow, it's always the smallest mistake that's hardest to find. Thank you so much!

Aakash Srivastava
Aakash Srivastava
5,415 Points

That's the part of learning programming , we'll do that :)
You are welcome