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

Cati lucian
Cati lucian
10,119 Points
Posted by Cati lucian
Cati lucian
10,119 Points

My code

http://codepen.io/amadeuscam/pen/pjVvKv

1 Answer

Nejc Vukovic
seal-mask
.a{fill-rule:evenodd;}techdegree seal-36
Nejc Vukovic
Full Stack JavaScript Techdegree Graduate 51,574 Points
Nejc Vukovic
.a{fill-rule:evenodd;}techdegree seal-36
Nejc Vukovic
Full Stack JavaScript Techdegree Graduate 51,574 Points 
var html = '';
var red;
var green;
var blue;
var rgbColor;

function getColorsDiv() {
  console.log('Started the function!');
  var count= 0; // you don't need this if you use for

  do {
    var red = Math.floor(Math.random() * 256 );
    var green = Math.floor(Math.random() * 256 );
    var blue = Math.floor(Math.random() * 256 );

  var rgbColor = 'rgb(' + red + ',' + green + ',' + blue + ')';
  html += '<div style="background-color:' + rgbColor + '; width:100px;    height:100px"></div>';

  count +=1;
  } while(count < 9)

//EVEN BETTER IS WITH A FOR LOOP

for(var i=0; i < 10; i++){
    var red = Math.floor(Math.random() * 256 );
    var green = Math.floor(Math.random() * 256 );
    var blue = Math.floor(Math.random() * 256 );
    var rgbColor = 'rgb(' + red + ',' + green + ',' + blue + ')';
    html += '<div style="background-color:' + rgbColor + '; width:100px;    height:100px"></div>';
}

    return html;  
}
document.write(getColorsDiv());

Why nothing shows -> you have to give the divs a height and width. If no height is present and no data is in the div then the div will be masked.

Hope this helped.