JavaScript JavaScript Loops, Arrays and Objects Simplify Repetitive Tasks with Loops The Refactor Challenge, Part 2

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

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

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

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;
}
Cara Jones
Cara Jones
Front End Web Development Techdegree Student 6,214 Points

It looks like your beginning html variable is missing a quotation mark. In the video it is var html = '';