Issue with alert statement.
lowNumber not defined.

Question

I keep getting an error with the alert line  below.
alert(Your random number between ${lowNumber} and ${highNumber} is ${generatedNumber}.);
Keeps saying the variables aren't defined.
Full code is below, please let me know what's messing me up.
// Collect input from a user
const firstenteredNumber = prompt("Please provide a number.");
const secondenteredNumber = prompt ("Please provide a second number.");
// Convert the input to a number
    const firstNumber = parseInt(firstenteredNumber);
    const secondNumber = parseInt(secondenteredNumber);
if(firstNumber && secondNumber){
//Define the high and low numbers  
   if(firstNumber > secondNumber){
       const highNumber = firstNumber;
       const lowNumber = secondNumber;
}else{
 const highNumber = secondNumber;
 const lowNumber = firstNumber;

}

// Use Math.random() and the user's number to generate a random number
const generatedNumber = Math.floor(Math.random() * (highNumber-lowNumber + 1) + lowNumber;

// Create a message displaying the random number
alert(`Your random number between ${lowNumber} and ${highNumber} is ${generatedNumber}.`);
}else{
  alert('You need to provide two number. Please try again.');
}

Martin Sole · Answer

Hi
It looks like you're assigning lowNumber and highNumber inside the nested if statement, but the alert message is outside of this.  As const(and let) uses block scoping the variables are not accessible outside of the if statement, so your alert message cannot reach them.  You would need to declare them outside of the nested if statement,

Welcome to the Treehouse Community

Looking to learn something new?

Gene Smith

Gene Smith

Issue with alert statement. lowNumber not defined.

1 Answer

Martin Sole

Martin Sole