What's wrong with this short Javascript code?

Question

I'm trying to make the short Javascript code throw an alert and show 93 instead of 0-93 but it's not working ?

<html>
    <head>
    <script type="text/javascript">
    function numberFromInput(value) {
  return  alert(Number(value.match(/[-]?(\d*)$/).pop()));
}
</script>
</head>
<body>

numberFromInput(0-93);

</body>
</html>

You're not actually calling your function. If you want to call it, you need to do that within a <script> tag as well. Otherwise, your numberFromInput(0-93); is just a text node on the page. — Brad
– Brad, Commented Jan 31, 2013 at 21:55
I recommend to read w3.org/wiki/Your_first_look_at_JavaScript. — Felix Kling
– Felix Kling, Commented Jan 31, 2013 at 22:00
You're trying to perform value.match, but the value you pass is a Number (0 - 93 will evaluate to -93) — jbabey
– jbabey, Commented Jan 31, 2013 at 22:10

bfavaretto · Accepted Answer · 2013-01-31 21:57:05Z

3

You have to call the function (you're just displaying its call code as content). And you have to pass the value as a string (you need quotes around 0-93):

<script type="text/javascript">
function numberFromInput(value) {
    return alert(Number(value.match(/[-]?(\d*)$/).pop()));
}
numberFromInput("0-93");
</script>

answered Jan 31, 2013 at 21:57

bfavaretto

72k18 gold badges118 silver badges162 bronze badges

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

1 Comment

bfavaretto Over a year ago

It works for me: jsbin.com/izalux/1/edit. Are you sure you did include the quotes around the string when you first tried it?

Collectives™ on Stack Overflow

What's wrong with this short Javascript code?

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related