Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

JavaScript JavaScript Loops, Arrays and Objects Simplify Repetitive Tasks with Loops Create a `do...while` loop

zahir madani
zahir madani
2,031 Points
Posted by zahir madani
zahir madani
2,031 Points

what's wrong with my code, it's working but here no.

i do not know what is the problem?

script.js 
var passWord = "sesame";
var secret;
var guessSecret = " ";
var newSecret = false;
do {
var secret = prompt("What is the secret password?");
  guessSecret++;
  if ( secret === "sesame" ) {
    newSecret = true;
  }
} while ( secret !== "sesame" )

document.write("You know the secret password. Welcome.");
index.html 
<!DOCTYPE html>
<html>
<head>
  <meta charset="utf-8">
  <title>JavaScript Loops</title>
</head>
<body>
<script src="script.js"></script>
</body>
</html>

1 Answer

Siddharth Nasare
Siddharth Nasare
15,636 Points
Siddharth Nasare
Siddharth Nasare
15,636 Points

Try below code:

var passWord = "sesame";
var secret;
var guessSecret = " ";
var newSecret = false;
do {
secret = prompt("What is the secret password?");
  // guessSecret++;  <---------------------------------------------- Don't use this 
  if ( secret === "sesame" ) {
    newSecret = true;
  }
} while ( secret !== "sesame" )

document.write("You know the secret password. Welcome.");

Don't use (guessSecret++). Because you are not giving any limited options to a client for asking this questions. Until client gives you correct password, you continue to ask this question. And if you use guessSecret++, the loop will stop at this point and won't go to check the condition.