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

Roudy Antenor
Roudy Antenor
4,124 Points
Posted by Roudy Antenor
Roudy Antenor
4,124 Points

Endless loop! - yet everything is verbatim

'''javascript var randomNumber = getRandomNumber(3); var guess; var guessCount = 0; var correctGuess = false;

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

do { guess = prompt("guess a number between 1 and 3"); guessCount += 1; if (parseInt (guess) === randomNumber) { correctGuess = true; } }while (! correctGuess)

document.write ("your number of guesses " +guessCount ) '''

Hello everyone - above is my code - and i do not understand why i continue to get an endless loop - if i change the random generator to go up to 1 only - the program works, BUT anything higher than that (say 3 ) and here comes BROWSER CRASH CITY!!

Please help

Timothy Suellentrop
Timothy Suellentrop
17,120 Points
Timothy Suellentrop
Timothy Suellentrop
17,120 Points

If you refer to the Markdown Cheatsheet under where you typed your question in, you can see how to use 3 backticks (`) to make your code easier for us to read and help you out.

Chris Slack
Chris Slack
28,500 Points
Chris Slack
Chris Slack
28,500 Points

This code seems to work fine as far as I can see. Try it here:

https://jsfiddle.net/jzz54mnp/

I added the actual number you are trying to guess into the prompt so that I didn't really have to guess, and it appears to keep going back to the prompt if I get the answer wrong but prints out the number of guesses if I get it right. This seems to be the right behavior.

For a more compact version of the code see here :-)

https://jsfiddle.net/p5f8bu1e/

Chris

2 Answers

Steven Parker
Steven Parker
229,670 Points
Steven Parker
Steven Parker
229,670 Points

Your code looks good.

As Chris pointed out, the current code seems designed to loop until a correct guess is given. This might seem "infinite" if you don't guess the correct number, which will get harder as the range is expanded.

One possible enhancement to the program that might help (and make it more fun) is to print out a hint after each try saying if the guess was too high or too low.

Roudy Antenor
Roudy Antenor
4,124 Points
Roudy Antenor
Roudy Antenor
4,124 Points

Than you all for your review of my code -- I did see what i was missing/ doing wrong -- BIG THANKS !!