Checking if a variable is an integer in javascript

Question

I made a form where the user inputs values for width and height that they want for the pop up window to be. I am using window.open for that.

So I think I need to check if the values for width and height are integer. I have a function that checks that a variable is an integer that is...

function isInteger(possibleInteger) {
    return !isNaN(parseInt(possibleInteger));
}

but I don't know how to call this function to the width and height function to check if the user inputted an integer. Can any one help?

Zafer · Accepted Answer · 2013-04-04 05:55:04Z

6

This is an answer to question mentioned in the topic, not the actual one in the body of the text :).

The following method is more accurate on determining if the string is a real integer.

function isInteger(possibleInteger) {
    return /^[\d]+$/.test(possibleInteger);
}

Your current method validates "7.5" for instance.

EDIT: Based on machineghost's comment, I fixed the function to correctly handle arrays. The new function is as follows:

function isInteger(possibleInteger) {
        return Object.prototype.toString.call(possibleInteger) !== "[object Array]" && /^[\d]+$/.test(possibleInteger);
}

edited Apr 4, 2013 at 5:55

answered Sep 5, 2010 at 10:50

Zafer

2,17517 silver badges30 bronze badges

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

1 Comment

machineghost Over a year ago

/^[\d]+$/.test([1]) returns true, and [1] is definitely an array, not an integer

zupa · Accepted Answer · 2013-04-29 17:05:31Z

4

An alternative answer if you worry about performance.

var isInteger1 = function(a) {
    return ((typeof a !== 'number') || (a % 1 !== 0)) ? false : true;
};

Load test results compared to Zafer's answer in Chrome:

undefined => 4ms vs 151ms
1 => 10ms vs 390ms
1.1 => 61ms vs 250ms
'1' => 8ms vs 334ms
[1] => 9ms vs 210ms
{foo: 'bar'} => 8ms vs 478ms

See for yourself: jsfiddle

answered Apr 29, 2013 at 17:05

zupa

13.5k5 gold badges43 silver badges40 bronze badges

Comments

Darin Dimitrov · Accepted Answer · 2010-09-05 09:06:18Z

1

var isWidthAnInteger = isInteger(document.getElementById('width').value);
var isHeightAnInteger = isInteger(document.getElementById('height').value);
if (isWidthAnInteger && isHeightAnInteger) {
    // TODO: window.open
}

where you have the following textboxes:

Width: <input type="text" id="width" name="width" />
Height: <input type="text" id="height" name="height" />

answered Sep 5, 2010 at 9:06

Darin Dimitrov

1.0m275 gold badges3.3k silver badges3k bronze badges

4 Comments

Sam Over a year ago

ok cool thanks! Do you know if i could do what u did but do if and else statements because i also want to check if it is an int and if it is not i want to send an error message back to the user. Thanks for ur help sorry new to javascript.

Sam Over a year ago

oh i am checking if width and height is int in a function and my window.open is another function called page load will this effect if (isWidthAnInteger && isHeightAnInteger) { // TODO: window.open } ???

Sandeepan Nath Over a year ago

just add else condition like this after the if else{ alert("Please enter integer values for width and height")}

Sandeepan Nath Over a year ago

this will be better -

if (!isWidthAnInteger && !isHeightAnInteger){alert("Please enter integer values for width and height");}else if(!isWidthAnInteger){alert("Please enter integer value for width");}elseif(!isHeightAnInteger){alert("Please enter integer value for height");}else{// TODO: window.open}

Collectives™ on Stack Overflow

Checking if a variable is an integer in javascript

3 Answers 3

1 Comment

Comments

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related