Lost....::sigh::...yet again

function randomGenerator() { return Math.floor(Math.random() * 6 - 1 + 1) + 1; }

function randomNumBetween(one, two) { if (one > two) { return "Looks like " + one + " is larger than " + two; } else { return "Looks like " + two + " is larger than " + one; } }

var result = randomNumBetween(randomGenerator(), randomGenerator());

alert(result);

This seems to work for me. It looks like you have a condition to be tested in your "else" statement which I do not think is possible. If the code above doesn't make sense, just let me know and I will explain.

Hi Greg,

I understand that you asked this question a while ago, but maybe I can help clarify where you went astray.

First, you're right. We didn't learn how to generate a random upper or lower number. However, the challenge isn't asking you to generate a random upper and lower number; instead, it wants you to create parameters for your randomNumBetween function. The video " Giving Information to Functions" teaches us how to insert parameters and then use them within the function.

function randomNumBetween(upper, lower) {
    //Your code here
}

Second, by adding parameters users have the ability to determine the upper and lower values of the function for their own needs. For example, if I wanted the function to generate a number between 1 and 6 for my dice game, I would call the function with parameters filled like so:

randomNumBetween(6,1); 

The function would then give me a random number between the two numbers I inputted into the function.

Third, Dave MacFarland gives us a hint as to what code should be put into the function in order to generate a random number between an upper and lower limit:

Math.floor(Math.random() * (6 - 1 + 1)) + 1; 

As he demonstrated in the video "The Random Challenge Solution" in the "Working with Numbers" section, '6' is the upper limit while the ' - 1' is the lower limit. For the function to be changeable, you need to assign the code above to a variable, then return that variable at the end of the function. Be sure to switch the 6 and 1 with your parameters!

I hope this helps. Good Luck!

Hi Greg, I think it may have just been an understanding in the requirements in the project rather than your actual code having issues.

The goal of the project is to have a program that will take a variable range. Think of different types of dice. In the standard example, the code is using fixed values of 1 and 6, which is the range of a standard six sided die. You're basically updating the function to make it more flexible to accept different values from a user. For example, let's say they only wanted a random number between 1 and 4 for a four sided die or maybe for a D&D game, they have a die that has higher numbers. By doing this, the function now has more value since it can be used for more things.

To do this, you're only generating ONE random number between a set range - Not TWO random numbers. I think this is what threw you off. For example, a random number between 1 and 100, might be 57. Where as a random number between 1 and 6, might be four.

To do that, all you have to do is simply update the formula that generates your random value to include the range of values that may be of interest to the user.

Here's how I did it:

function randomGenerator( lower, upper ) {
  if (isNaN(lower) || isNaN(upper)) {
      throw new Error("An argument is not a number.");
      }

  var randomValue = Math.floor(Math.random() * (upper - lower + 1)) + lower;
  return randomValue;
}

var lowerInput = prompt("Please enter lower number");
var upperInput = prompt("Please enter upper number");
// updated to convert string inputs to numbers
var lowerNumber = Number(lowerInput);
var upperNumber = Number(upperInput);

document.write( randomGenerator(lowerNumber, upperNumber) );

Beyond that, try not to get discouraged. There's a secret to programming that a lot of people don't know - most of your time spent programming is fixing bugs rather than actually coding. Best of luck to you!

I don't understand why we don't have to declare variables. Just be writing "upper" and "lower", how does the computer know what number will go there? I don't get it. I would have declared a var lower = 0 and a var upper=10000000

Help? :'(

Thanks for the reply Jennifer. I will read your comment over and over until I get it. :) Thanks again!