Take a peak at my code?

For a new programmer, having your code run without errors and successfully accomplish the task is always an occasion to reflect on with pride.

Here's a gold star for you!

Great job on your coding. It is D.R.Y. and almost error free. Because your code is not using the *"use strict" * statement; errors are ignored. but once you invoke the statement then your code fails to run.

Here are a few things to change.

1. move all variables to the top.
2. declare x and i variable
3. (optional) change correct[correct.length] = quiz[i] to correct.push(quiz[i])

jsFiddle: original code

jsFiddle: code with changed

var score = 0;
  var quiz = [
    ["What is 2 + 3?", 5],
    ["What is 3 - 1?", 2],
    ["What is 8 - 2?", 6]
  ]
  var correct = [];
  var wrong = [];

  function print(message) {
    document.write(message);
  }

  function toAList(list) {
    var htmlChange = '<ol>';
    for (var x = 0; x < list.length; x++) {
      htmlChange += "<li>" + list[x][0] + "</li>";
    }
    htmlChange += '</ol>'
    print(htmlChange);
  }

  for (var i = 0; i < quiz.length; i++) {
    var userAnswer = parseInt(prompt(quiz[i][0]));
    if (userAnswer === quiz[i][1]) {
      correct.push(quiz[i]);
      score += 1;
    } else {
      wrong.push(quiz[i]);
      score -= 1;
    }
  }

  console.log(score);
  print("You got " + correct.length + " question(s) correctly!<br>");
  print("Your final score is " + score + "<br>");
  print("<b>These are the questions you got correct:</b>")
  toAList(correct);
  print("<b>These are the questions you got wrong:</b>");
  toAList(wrong);

I hope this helps. Keep Coding.

Hey Brandon,

RE: Push[i]

So what is happening when you push(quiz[i]) you are creating an entirely new array, a copy, into either correct or wrong array. After the user has completed the quiz. The arrays within correct and wrong are then sent to the toAList() method, which iterates through each and only prints out the question (i.e **quiz[x][0]) leaving the value behind but not deleting it.

RE: Global Variables

Your variables where originally below your functions, which is fine for the size of this project but can become troublesome as the project grows and if you were wanting to access a global variable within your functions, you would be unable to do so because your functions were searching for a variable which has not yet been declared.

RE: Local Variables

Yes, variables declared inside of a loop are local variables and not accessible outside of its scope, like a variable inside of functions.

RE: use strict

This is something that you will learn later as you move forward. It is best practice to use this statement in your projects as it is limited to strict programming guidelines and will not function if errors occur.

For example, you original code when I added the "use strict" failed to run because the variable i was not declared. This error was referring to the i in your for loop.

Once i was declared, by adding var in front, The quiz began prompting the questions. BUT, it later failed when it reached the toAList() method stating that x was not declared. So when declared and run again, the quiz was able to run completely.

I hope this helps and feel free to continue asking me questions if you are still confused.