Check if string is integer problem

[FraanXT]

Hello guys, I have a form where you can select a score between 0 and 10(only integers).

I want to check if the value that I recive, is integer.

 

Im using this code(exemple):

$integer =  intval($_POST["p1"]);
    if(is_int($integer)){
        echo 'INT';
    } else {
        echo 'NO INT';
    }

My problem is that if it recive letters, the conversion converts it to 0, and it says that it is integer.

So with this method I should discard the 0.

 

You guys know a method to avoid that?

 

Thank you guys

[mac_gyver]

the ctype_digit() function can be used to test if all the characters in a variable are numerical.

[FraanXT]

the ctype_digit() function can be used to test if all the characters in a variable are numerical.

 

But I want to check if they are integer, if you know how?

[requinix]

...

 

If all the characters are numerical then it is an integer.

Edited May 30, 2014 by requinix

[FraanXT]

...

 

If all the characters are numerical then it is an integer.

 

1 is integer

1,5 isnt integer

[Jacques1]

1,5 isnt integer

 

Who said it is?

 

Did you try ctype_digit()? Or do you just assume it's not what you need?

[FraanXT]

Who said it is?

 

Did you try ctype_digit()? Or do you just assume it's not what you need?

 

It works, ty bro, and sorry cuz I assumed that it check if its numerical only

[Jacques1]

It does check for digits only. And since a comma isn't a digit, it will not accept “1,5”.

[FraanXT]

It does check for digits only. And since a comma isn't a digit, it will not accept “1,5”.

Yeah nice one, thank you

[.josh]

Also wanted to mention that I see in your original code you are using intval. FYI ctype_digit expects a string argument, so if you are converting it to an integer type then you are going to get unexpected results from ctype_digit. So to be clear, do not cast/convert the value as an integer before using ctype_digit. Posted variables should always be a string type, so you don't need to do anything special before using ctype_digit, but if you really want to be explicit, type cast to string:

 

// this should be okay..
if ( ctype_digit( $_POST['p1'] ) ) 

// ..but if you want to be more explicit
if ( ctype_digit( (string) $_POST['p1'] ) )