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 Sal Sal
Sal Sal
47 Points

My code does not previews in the webpage. Can anyone help me identify the problem.

My code does not previews in the webpage. Can anyone help me identify the problem.

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 <= 10; i += 1) {

rgbColor= randomColor(); html += '<div style="background-color:' + rgbColor + '"></div>'; }

print(html);

3 Answers

Xin Zhu
Xin Zhu
19,826 Points
Xin Zhu
Xin Zhu
19,826 Points

Did you set the styles to your DIVs? If you just run this in html file, the height of those DIVs is zero, so you can't see them. Try to give the DIVs a height.

Petter Östergren
Petter Östergren
6,920 Points
Petter Östergren
Petter Östergren
6,920 Points

For this, case a div hight was specified its part of a refactor challenge.

Petter Östergren
Petter Östergren
6,920 Points
Petter Östergren
Petter Östergren
6,920 Points

Try removing the last ',' in randomColor()

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