Why is this do...while loop not working?

Hey Tyler,

You're very close, except for a couple minor things.

1. You have the secret variable declared outside of the loop, which is correct, so you can't re-declare it inside the loop. To fix this, you just need to get rid of the var keyword. With it there, JS is trying to re-declare it... without it, JS is just using the variable (which is what you want).

2. You want the loop (i.e. the question for password prompt) to run as long as the user is getting it wrong. Right now you are checking to see if the answer IS correct, which will keep the loop running as long as the user is getting it right (which is not what you want). So, you need to change you check to be "not equal".

Below is the corrected code for you to review. Have a look, and I hope it makes sense. And, just an extra note, to make the code more DRY, you don't need to assign a value to the secret variable outside of the loop. You can just initialize an empty variable var secret; The do loop is assigning a value to secret from the prompt and the while is comparing them. Other that that... Good job!

var secret;
do {
  secret = prompt("What is the secret password?");
} while (secret != "sesame") {
  document.write("You know the secret password. Welcome.");  
}

Keep Coding! :)

Your loop runs while the password is correct, and ends when it is wrong. You probably want it to go the other way.

Also, a few other things:

• You don't need to preset secret, it will be set by calling prompt.
• You don't want to declare secret again inside the loop, just assign it.
• You don't need those braces around the document.write.
• Semicolons after a statement at the end of a line are optional, but they are good practice.

var secret;
do {
   secret = prompt("What is the secret password?");
} while (secret != "sesame");
document.write("You know the secret password. Welcome.");

Thank you!