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 and the DOM (Retiring) Traversing the DOM Child Traversal

Martin Coutts
Martin Coutts
18,154 Points
Posted by Martin Coutts
Martin Coutts
18,154 Points

Code keeps crashing

Hi

Bit stuck here, I have the first part of the challenge done but my code keeps crashing on the second part. I know that means it isn't quite correct but not sure what changes I need to make. I think it is due to using paragraphs[i]. Not sure whether to create a function first and then use it in the loop.

app.js 
const section = document.querySelector('section');
let paragraphs = section.children;

for (let i = 0; i < paragraphs.length; i += 0){
  paragraphs[i].style.color = 'blue';
}
index.html 
<!DOCTYPE html>
<html>
    <head>
        <title>Child Traversal</title>
    </head>
    <body>
        <section>
            <p>This is the first paragraph</p>
            <p>This is a slightly longer, second paragraph</p>
            <p>Shorter, last paragraph</p>
        </section>
        <footer>
            <p>&copy; 2016</p> 
        </footer>
        <script src="app.js"></script>
    </body>
</html>

1 Answer

Justin Iezzi
Justin Iezzi
18,199 Points

Hey Martin, looks like you have an infinite loop issue here.

for (let i = 0; i < paragraphs.length; i += 0){
  paragraphs[i].style.color = 'blue';
}

Since you're incrementing by 0, variable i never reaches the paragraphs.length value.

This can be fixed by either using i += 1 or i++ as the increment for your for loop.

Martin Coutts
Martin Coutts
18,154 Points
Martin Coutts
Martin Coutts
18,154 Points

Thanks Justin

Knew it would be something super obvious but not that obvious. Sometimes takes a second set of eyes I guess haha. Thanks again.