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
Emanuel Rouse
1,985 PointsMy attempt at the Random Number Challenge. Looking for feedback.
Here's my attempt at the Random Number Challenge:
var userNumberAsString = prompt('Give me a number'); var userNumber = parseInt(userNumberAsString); document.write(Math.floor(Math.random() * userNumber) + 1);
What do you think? Is there anything wrong with doing it this way? I feel like my variable names could use some work.
2 Answers
Unsubscribed User
Front End Web Development Techdegree Student 33,901 PointsHi Emanuel,
you actually don't have to create two different variables.
You can just declare the variable userNumber and then reassign it with the parsInt method:
var userNumber = prompt('Give me a number');
userNumber = parseInt(userNumber);
document.write(Math.floor(Math.random() * userNumber) + 1);
or even shorter
var userNumber = prompt('Give me a number');
document.write(Math.floor(Math.random() * (parseInt(userNumber)) + 1));
That way you only need one variable.
Makes sense?
Happy coding!
Nils
PS: You can upvote my post and/or mark as "best answer" (at the bottom of my post) if it helped you. :-)
Emanuel Rouse
1,985 PointsThat does help a lot. I have a long way to go. Thanks so much!
Unsubscribed User
Front End Web Development Techdegree Student 33,901 PointsAlways a pleasure if I find something where I can help.
And you're doing great - just keep going. JavaScript is a lot of fun. :-)