is_int useage

[webguy262]

this does not work for me...

 

$category=mysql_real_escape_string($_GET['category']);
if (is_int($category)) {

 

i echo out $category and confirm it is an integer and yet the if statement does not fire

 

what am i missing?

[AyKay47]

you are retrieving this from either a query string or a form value correct?

If so, the number is passed as a numeric string, not an integer. you can either cast the numeric string into an int before using is_int(), or you can use is_numeric() instead.

[kney]

Shouldn't you use is_numeric?

 

$category=mysql_real_escape_string($_GET['category']);
if (is_numeric($category)) {

 

Edit: What AyKay47 says

[AyKay47]

I normally cast numeric string values right off the bat when I am transferring data. Makes for neater and easier code..

 

$category= (int)$_GET['category'];
      if(is_int($category)) {  }

[ManiacDan]

$category= (int)$_GET['category'];
      if(is_int($category)) {  }

This code is unnecessary.  Of course a variable you just cast as an int will be an int.  You should check to make sure it's not zero.

 

If zero is a valid value for $category, you shouldn't cast at all and use is_numeric.

[AyKay47]

$category= (int)$_GET['category'];
      if(is_int($category)) {  }

This code is unnecessary.  Of course a variable you just cast as an int will be an int.  You should check to make sure it's not zero.

 

If zero is a valid value for $category, you shouldn't cast at all and use is_numeric.

true that would definitely be overkill.

[webguy262]

thanks for all the info!