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

brandonlind2
brandonlind2
7,823 Points
Posted by brandonlind2
brandonlind2
7,823 Points

Can someone explain to why isNaN isnt working?

When I type a string in, NaN comes up instead of the new error message

function randomNumber(top, lower) { return Math.floor(Math.random() * (top - lower)) + 1;
if (isNaN(top) || isNan(lower) ) { throw new Error("input is NOT a number");}}

1 Answer

Chyno Deluxe
Chyno Deluxe
16,936 Points

Your function is returning a value first and ignores the conditions after. All you need to do is change the order to First, check if the arguments isNan() if so, throw the error else return the value.

function randomNumber(top, lower) {
  if (isNaN(top) || isNaN(lower)) {
    throw new Error("input is NOT a number");
  }   
  return Math.floor(Math.random() * (top - lower)) + 1;
}