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

ALEXANDER MARININ
seal-mask
.a{fill-rule:evenodd;}techdegree
ALEXANDER MARININ
Full Stack JavaScript Techdegree Student 5,276 Points

What do you think?

let html = '';
for (let i = 0; i < 10; i++) 
{
  let color = () => Math.floor(Math.random() * 256 );
  let RGB = `${color()},${color()},${color()}`;
  html += '<div style=background-color:' + `rgb(${RGB})` + '></div>';
}
document.write(html);
Greg Heffley
Greg Heffley
6,371 Points

What is the problem that you are having?

1 Answer

Ignacio Rocha
Ignacio Rocha
7,462 Points

I don't know what is the problem you're having... but let me help you with something that your code has wrong.

// When you use template literals you don't need to use quotes
// so if you use quotes... stick with it don't mix them

html += `<div style='background-color: rgb( ${rgb} )'></div>`

// Also... never use uppercase when declaring a variable,
// always use lowercase and upper when is a compound word