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 Working with 'for' Loops The Refactor Challenge – Duplicate Code

Tao Zhang
Tao Zhang
1,869 Points
Posted by Tao Zhang
Tao Zhang
1,869 Points

My code does not produce colors

It just shows the numbers with colors. Can anyone look at the codes below and tell me where it went wrong:

let html = ''; let red; let green; let blue; let randomRGB;

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

for ( let i = 1; i <= 10; i++ ) { red = GetNumber (); green = GetNumber (); blue = GetNumber (); randomRGB = rgb( ${red}, ${green}, ${blue} ); html += <div style="background-color: ${randomRGB}">${i}</div>; }

document.querySelector('main').innerHTML = html;

1 Answer

Steven Parker
Steven Parker
232,176 Points
Steven Parker
Steven Parker
232,176 Points

The GetNumber function is computing the value correctly, but it is missing the return statement to pass the value back to the caller for constructing the color.

Also, for future questions be sure to use Markdown formatting when posting code to preserve formatting and special characters. You might also want to take a look at this video about Posting a Question.