Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

JavaScript JavaScript Basics (Retired) Creating Reusable Code with Functions Random Number Challenge

Rwanda Muhammad
seal-mask
.a{fill-rule:evenodd;}techdegree
Rwanda Muhammad
Full Stack JavaScript Techdegree Student 4,116 Points
Posted by Rwanda Muhammad
.a{fill-rule:evenodd;}techdegree
Rwanda Muhammad
Full Stack JavaScript Techdegree Student 4,116 Points

How is my code

https://w.trhou.se/s72rxmx08m

ian izaguirre
ian izaguirre
3,220 Points
ian izaguirre
ian izaguirre
3,220 Points

is it working like you expect it to work? I see you misspelled this part "getRamdon " , so thats why I ask if it works?

1 Answer

Tsenko Aleksiev
Tsenko Aleksiev
3,819 Points
Tsenko Aleksiev
Tsenko Aleksiev
3,819 Points

I have a question: What's the idea of having the same code in different functions and just different parameter? You saw that you have a mistake "getRamdon" and after the second function you call:

alert(getRandom(3));

OR you are just calling the first function a second time?! I don't get it. Isn't it better to do it:

function getRandom(num){
    return Math.floor(Math.random() * num ) + 1;
}
var upper = getRandom(100);
var lower = getRandom(3);
alert(upper);
alert(lower);

If you don't need the variables you can just:

alert(getRandom(100));
alert(getRandom(3));

Hope I helped :)