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 `do ... while` Loops

Posted by Maceo Lewis
Maceo Lewis
14,200 Points

My prompt will not loop

Can't figure it out for the life of me

Imgur

2 Answers

Jennifer Nordell
seal-mask
STAFF
.a{fill-rule:evenodd;}techdegree
Jennifer Nordell
Treehouse Teacher
Jennifer Nordell
.a{fill-rule:evenodd;}techdegree
Jennifer Nordell
Treehouse Teacher

Well I see a problem on your first line which affects the rest of your code. You have this:

var getRandomNumber = getRandomNumber(10);

But later you have this:

if(parseInt(guess === randomNumber) {

I sincerely believe that your first line should be this:

var randomNumber = getRandomNumber(10);

Try this change and see if you can get some different results! :thumbsup:

Maceo Lewis
Maceo Lewis
14,200 Points

Thanks Jennifer. I'll take this is as a hint that I need to take more breaks.

Julien riera
Julien riera
14,665 Points
Julien riera
Julien riera
14,665 Points

Hi,

Here's my code for this exercise since I didn't notice the issue in yours. Hope it will help.

Does the code at least run once ?

var randomNumber = getRandomNumber(10);
var guess;
var guessCount = 0;
var correctGuess = false;

function getRandomNumber( upper ) {
  var num = Math.floor(Math.random() * upper) + 1; 
  return num;
}

do {
  guess = prompt("I am thinking of a number between 1 and 10. What is it?");
  guessCount += 1;
  if (parseInt(guess) === randomNumber) {
    correctGuess = true;
  }
} while ( ! correctGuess ) 
document.write("<h1>You guessed the number !</h1>");
document.write("It took you " + guessCount + " tries to guess the number " + randomNumber);

Edit, the previous comment, from Jennifer, should probably help you. :)

Maceo Lewis
Maceo Lewis
14,200 Points

Thanks Julien! I definitely appreciate the help.