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 The Refactor Challenge, Part 2

Posted by Robbie Baker
Robbie Baker
7,736 Points

Nothing prints to the page

Anytime i run this nothing prints to the page. Though i don't see a difference compared to the teachers code.

var html = ''; var rgbColor;

function randomRGB() {

return Math.floor(Math.random() * 256 );

}

function randomColor() { var color = 'rgb('; color += randomRGB() + ','; color += randomRGB() + ','; color += randomRGB() + ','; return color; }

function print(message) { document.write(message); }

for (var i = 1; i < 100; i += 1) { rgbColor = randomColor(); html += '<div style="background-color:' + rgbColor + '"></div>';

}

print(html);

1 Answer

Erik Nuber
Erik Nuber
20,629 Points
Erik Nuber
Erik Nuber
20,629 Points 
var html = ''; 
var rgbColor;

function randomRGB() {
return Math.floor(Math.random() * 256 );
}

function randomColor() { 
var color = 'rgb(';
color += randomRGB() + ','; 
color += randomRGB() + ','; 
color += randomRGB() + ','; 
return color; 
}

function print(message) { document.write(message); }

for (var i = 1; i < 100; i += 1) { 
rgbColor = randomColor(); 
html += '<div style="background-color:' + rgbColor + '">
</div>';

}

print(html);

The problem is in the randomRGB function. You open the color string properly but don't close it. You are using a comma instead. That appears to be the only issue.

ie

color_= randomRGB() + ')';