Javascript Function Problem

Question

I am trying to get the user to input a number, then input another number that will begin the multiplication table for the first number entered and continue on incrementing by one until the last number the user was prompted for is reached...

for example, if the user was to enter 10, then 1, then 10, the output would look like:

10 * 1 = 10
10 * 2 = 20
10 * 3 = 30
...
10 * 10 = 100

What am I doing wrong?

Thanks for the help!

function multiplierz(numberMultiplied, amountToMultiplyFirst, amountToMultiplyLast)
    {


        while (isNaN(numberMultiplied) = true || isNaN(amountToMultiplyFirst) = true || isNaN(amountToMultiplyLast) = true)
        {
            alert("YOU DID NOT ENTER ONLY NUMBERS. WAY TO GO DUMMY.");
            var numberMultiplied = prompt("Please Enter The Number to Multiply");
            var amountToMultiplyFirst = prompt("Start multiplying by which number?");
            var amountToMultiplyLast = prompt("End multiplying by which number?");
        }
        for (counter = amountToMultiplyFirst; counter <= amountToMultiplyLast; counter++)
            {
                document.write(numberMultiplied + " * " + counter + " = " + numberMultiplied * counter + "<br />");
            }
    }

    //Begin Program Below


        multiplierz(prompt("Please Enter The Number to Multiply"),prompt("Start multiplying by which number?"),prompt("End multiplying by which number?"));
</script>

Brad · Accepted Answer · 2011-04-18 04:11:29Z

4

Your isNan(numberMultiplied) = true is throwing an error. In javascript the = operator is for assignment.

Just do

while (isNaN(numberMultiplied) || isNaN(amountToMultiplyFirst) || isNaN(amountToMultiplyLast) )

answered Apr 18, 2011 at 4:11

Brad

6034 silver badges12 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Mark Cidade · Accepted Answer · 2011-04-18 04:11:41Z

2

You need to use == instead of = in your while condition:

while (isNaN(numberMultiplied) == true || isNaN(amountToMultiplyFirst) == true || isNaN(amountToMultiplyLast) == true)
{
  ...
}

answered Apr 18, 2011 at 4:11

Mark Cidade

100k33 gold badges231 silver badges238 bronze badges

Comments

Senthil · Accepted Answer · 2011-04-18 04:11:08Z

0

Simple thing that you didn't consider amountToMultiplyFirst should be less than amountToMultiplyLast :-)

answered Apr 18, 2011 at 4:11

Senthil

5,8722 gold badges24 silver badges11 bronze badges

Collectives™ on Stack Overflow

Javascript Function Problem

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related