What went wrong? Troubleshooting JavaScript

Generally speaking, when you do something wrong in code, there are two main types of error that you'll come across:

• Syntax errors: These are spelling errors in your code that actually cause the program not to run at all, or stop working part way through — you will usually be provided with some error messages too. These are usually okay to fix, as long as you are familiar with the right tools and know what the error messages mean!
• Logic errors: These are errors where the syntax is actually correct but the code is not what you intended it to be, meaning that program runs successfully but gives incorrect results. These are often harder to fix than syntax errors, as there usually isn't an error message to direct you to the source of the error.

Okay, so it's not quite that simple — there are some other differentiators as you drill down deeper. But the above classifications will do at this early stage in your career. We'll look at both of these types going forward.

To get started, let's return to our number guessing game — except this time we'll be exploring a version that has some deliberate errors introduced. Go to GitHub and make yourself a local copy of number-game-errors.html (see it running live here).

1. To get started, open the local copy inside your favorite text editor, and your browser.
2. Try playing the game — you'll notice that when you press the "Submit guess" button, it doesn't work!

At this point, let's consult the developer console to see if it reports any syntax errors, then try to fix them. You'll learn how below.

Earlier on in the course we got you to type some simple JavaScript commands into the developer tools JavaScript console (if you can't remember how to open this in your browser, follow the previous link to find out how). What's even more useful is that the console gives you error messages whenever a syntax error exists inside the JavaScript being fed into the browser's JavaScript engine. Now let's go hunting.

1. Go to the tab that you've got number-game-errors.html open in, and open your JavaScript console. You should see an error message along the following lines:

2. The first line of the error message is:

   Uncaught TypeError: guessSubmit.addeventListener is not a function
   number-game-errors.html:87:19
   

   • The first part, Uncaught TypeError: guessSubmit.addeventListener is not a function, is telling us something about what went wrong.
   • The second part, number-game-errors.html:87:19, is telling us where in the code the error came from: line 87, character 19 of the file "number-game-errors.html".

3. If we look at line 87 in our code editor, we'll find this line:

   js

   guessSubmit.addeventListener("click", checkGuess);
   

4. The error message says "guessSubmit.addeventListener is not a function", which means that the function we're calling is not recognized by the JavaScript interpreter. Often, this error message actually means that we've spelled something wrong. If you are not sure of the correct spelling of a piece of syntax, it is often good to look up the feature on MDN. The best way to do this currently is to search for "mdn name-of-feature" with your favorite search engine. Here's a shortcut to save you some time in this instance: addEventListener().

5. So, looking at this page, the error appears to be that we've spelled the function name wrong! Remember that JavaScript is case-sensitive, so any slight difference in spelling or casing will cause an error. Changing addeventListener to addEventListener should fix this. Do this now.

1. Save your page and refresh, and you should see the error has gone.

2. Now if you try to enter a guess and press the Submit guess button, you'll see another error!

3. This time the error being reported is:

   Uncaught TypeError: can't access property "textContent", lowOrHi is null
   

   Depending on the browser you are using, you might see a different message here. The message above is what Firefox will show you, but Chrome, for example, will show you this:

   Uncaught TypeError: Cannot set properties of null (setting 'textContent')
   

   It's the same error, but different browsers describe it in a different way.

   Note: This error didn't come up as soon as the page was loaded because this error occurred inside a function (inside the checkGuess() { } block). As you'll learn in more detail in our later functions article, code inside functions runs in a separate scope than code outside functions. In this case, the code was not run and the error was not thrown until the checkGuess() function was run by line 87.

4. The line number given in the error is 79. Have a look at line 79, and you'll see the following code:

   js

   lowOrHi.textContent = "Last guess was too high!";
   

5. This line is trying to set the textContent property of the lowOrHi variable to a text string, but it's not working because lowOrHi does not contain what it's supposed to. Let's see why this is — try searching for other instances of lowOrHi in the code. The earliest instance you'll find is on line 51:

   js

   const lowOrHi = document.querySelector("lowOrHi");
   

6. At this point we are trying to make the variable contain a reference to an element in the document's HTML. Let's see what the variable contains after this line has been run. Add the following code on line 54:

   js

   console.log(lowOrHi);
   

   This code will print the value of lowOrHi to the console after we tried to set it in line 51. See console.log() for more information.

7. Save and refresh, and you should now see the console.log() result in your console. Sure enough, lowOrHi's value is null at this point, and this matches up with the Firefox error message lowOrHi is null. So there is definitely a problem with line 51. The null value means "nothing", or "no value". So our code to set lowOrHi to an element is going wrong.

8. Let's think about what the problem could be. Line 51 is using a document.querySelector() method to get a reference to an element by selecting it with a CSS selector. Looking further up our file, we can find the paragraph in question:

   html

   <p class="lowOrHi"></p>
   

9. So we need a class selector here, which begins with a dot (.), but the selector being passed into the querySelector() method in line 51 has no dot. This could be the problem! Try changing lowOrHi to .lowOrHi in line 51.

10. Try saving and refreshing again, and your console.log() statement should return the <p> element we want. Phew! Another error fixed! You can delete your console.log() line now, or keep it to reference later on — your choice.

1. Now if you try playing the game through again, you should get more success — the game should play through absolutely fine, until you end the game, either by guessing the right number, or by running out of guesses.
2. At that point, the game fails again, and the same error is spat out that we got at the beginning — "TypeError: resetButton.addeventListener is not a function"! However, this time it's listed as coming from line 95.
3. Looking at line number 95, it is easy to see that we've made the same mistake here. We again just need to change addeventListener to addEventListener. Do this now.

At this point, the game should play through fine, however after playing through a few times you'll undoubtedly notice that the game always chooses 1 as the "random" number you've got to guess. Definitely not quite how we want the game to play out!

There's definitely a problem in the game logic somewhere — the game is not returning an error; it just isn't playing right.

1. Search for the randomNumber variable, and the lines where the random number is first set. The instance that stores the random number that we want to guess at the start of the game should be around line number 47:

   js

   let randomNumber = Math.floor(Math.random()) + 1;
   

2. And the one that generates the random number before each subsequent game is around line 114:

   js

   randomNumber = Math.floor(Math.random()) + 1;
   

3. To check whether these lines are indeed the problem, let's turn to our friend console.log() again — insert the following line directly below each of the above two lines:

   js

   console.log(randomNumber);
   

4. Save and refresh, then play a few games — you'll see that randomNumber is equal to 1 at each point where it is logged to the console.

To fix this, let's consider how this line is working. First, we invoke Math.random(), which generates a random decimal number between 0 and 1, e.g. 0.5675493843.

Math.random();

Next, we pass the result of invoking Math.random() through Math.floor(), which rounds the number passed to it down to the nearest whole number. We then add 1 to that result:

Math.floor(Math.random()) + 1;

Rounding a random decimal number between 0 and 1 down will always return 0, so adding 1 to it will always return 1. We need to multiply the random number by 100 before we round it down. The following would give us a random number between 0 and 99:

Math.floor(Math.random() * 100);

Hence us wanting to add 1, to give us a random number between 1 and 100:

Math.floor(Math.random() * 100) + 1;

Try updating both lines like this, then save and refresh — the game should now play like we are intending it to!

There are other common errors you'll come across in your code. This section highlights most of them.

This could be another symptom of mixing up the assignment and strict equality operators. For example, if we were to change this line inside checkGuess():

if (userGuess === randomNumber) {

to

if (userGuess = randomNumber) {

the test would always return true, causing the program to report that the game has been won. Be careful!

This one is pretty simple — it generally means that you've missed the closing parenthesis at the end of a function/method call.

This error usually relates to an incorrectly formed JavaScript object, but in this case we managed to get it by changing

function checkGuess() {

to

function checkGuess( {

This has caused the browser to think that we are trying to pass the contents of the function into the function as an argument. Be careful with those parentheses!

This is easy — it generally means that you've missed one of your curly braces from a function or conditional structure. We got this error by deleting one of the closing curly braces near the bottom of the checkGuess() function.

These errors generally mean that you've left off a string value's opening or closing quote mark. In the first error above, string would be replaced with the unexpected character(s) that the browser found instead of a quote mark at the start of a string. The second error means that the string has not been ended with a quote mark.

For all of these errors, think about how we tackled the examples we looked at in the walkthrough. When an error arises, look at the line number you are given, go to that line and see if you can spot what's wrong. Bear in mind that the error is not necessarily going to be on that line, and also that the error might not be caused by the exact same problem we cited above!

So there we have it, the basics of figuring out errors in simple JavaScript programs. It won't always be that simple to work out what's wrong in your code, but at least this will save you a few hours of sleep and allow you to progress a bit faster when things don't turn out right, especially in the earlier stages of your learning journey.

• There are many other types of errors that aren't listed here; we are compiling a reference that explains what they mean in detail — see the JavaScript error reference.
• If you come across any errors in your code that you aren't sure how to fix after reading this article, you can get help! Ask for help on the communication channels. Tell us what your error is, and we'll try to help you. A listing of your code would be useful as well.

• Previous
• Overview: Dynamic scripting with JavaScript
• Next