not getting desired output in javascript

Question

i was trying to make a javascript program to convert a number between 20 and 100 to in words. so i wrote this-

var num = prompt("enter a number");
if (num>20 && num<100)
{
 words(num);
}
else alert("Please enter a number between 20 and 100");

function words(num)
{
var ones = ["","one","two","three","four","five","six", "seven","eight",  "nine"];
var tens = ["", "", "twenty","thirty","fourty","fifty","sixty","seventy","eighty","ninety"];

var div= num/10;
var rem= num%10;

if (rem==0)
    document.write(num+" = "+tens[div]);
else
    document.write(num+" = "+tens[div]+" "+ones[rem]);

}

the problem is if i enter 30 ,40 like that numbers which are divisible by 10 i get correct output but if i enter 32 it will show "32 = undefined two". what did i do wrong? i am new to JS so dont know much.

Denys Séguret · Accepted Answer · 2015-02-20 08:23:05Z

2

32/10 is 3.2, not 3. You must round the result.

Change

 var div= num/10;

to

 var div= Math.floor(num/10);

answered Feb 20, 2015 at 8:23

Denys Séguret

384k90 gold badges813 silver badges780 bronze badges

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

Comments

void · Accepted Answer · 2015-02-20 08:24:41Z

1

You should be doing

var rem= num%10;
var div= (num - rem)/10;

Because 25/10 = 2.5 not 2

Working Fiddle

answered Feb 20, 2015 at 8:24

void

36.7k10 gold badges71 silver badges111 bronze badges

4 Comments

Sajan Over a year ago

i dont have enough reputation to down vote, dont know who did that .

Denys Séguret Over a year ago

This looks like a valid alternative. I'd like the downvoter to explain his vote.

void Over a year ago

I guess Math.floor() is relatively more complex operation then this direct Subtraction. Why to use it unnecessarily?

Sajan Over a year ago

yeah,this looks valid and simple without using math().

Collectives™ on Stack Overflow

not getting desired output in javascript

2 Answers 2

Comments

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related