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 Tracking Multiple Items with Arrays Iterating through an Array

Posted by Tyrone Cartwright
Tyrone Cartwright
4,023 Points

Code took too long to run

My code took to long, is their something I'm doing wrong?

script.js 
var temperatures = [100,90,99,80,70,65,30,10];
    for (var i = 0; temperatures.length; i += 1) {
        console.log(temperatures[i]);
}
index.html 
<!DOCTYPE html>
<html>
<head>
  <meta charset="utf-8">
  <title>JavaScript Loops</title>
</head>
<body>
<script src="script.js"></script>
</body>
</html>

2 Answers

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

The conditional clause of this loop is simply "temperatures.length", which will be considered true because it is not zero. This causes the loop to run continuously.

Normally the conditional clause would contain a test that determines when the loop should continue. It's commonly done by comparing something to the loop variable, so you probably intended to write "i < temperatures.length".

Clayton Perszyk
MOD
Clayton Perszyk
Treehouse Moderator 48,723 Points
Clayton Perszyk
Clayton Perszyk
Treehouse Moderator 48,723 Points

check your exit clause. currently it will always evaluate to true, and as a result it is creating an infinite loop.