[RESOLVED] error code to see if something is an integer

[ginginca]

This code is working for me correctly:

//Make Sure $amount is NOT blank before checking if it is a number
if (!empty($item_1_qty)) {
//Amount is NOT blank
if (!is_numeric($item_1_qty)) {
//Amount is NOT a number
$redirect_to = 'order_form.php?error=1';
header("Location: $redirect_to");
exit();
}
}

But when I change

!is_numeric to !is_int or !is_integer

it no longer works

???

My error message comes up every time when I do it with integer.

[rincewind456]

It works fine!
Are you sure that your not passing the number as a string, because if you are, a string 7 for example will work in [man]is_numeric/man but not in [man]is_int/man.

[NogDog]

is_int() is type-sensitive, so will return false if the argument is a string:

echo is_int(1);   // TRUE
echo is_int("1"); // FALSE

You could do something like:

if(is_numeric($value) and (int)$value == $value)
{
   $value = (int)$value; // cast to int type for rest of script
}
else
{
  // handle error condition for non-integer value being entered
}

[ginginca]

That's a very good explanation NoDog. I get it. Thanks a bunch!

I'm almost there but not quite. I am getting an error with the opening bracket on the second line:

		if ((is_numeric($item_1_qty) and (int)$item_1_qty == $item_1_qty) 
			{
			$item_1_qty =(int)$item_1_qty;

	{
		//Amount is NOT a whole number
		$redirect_to = 'order_form.php?error=1';	
		header("Location: $redirect_to");
		exit();
	}
}

[ginginca]

Got a whole other way of thinking about this ... but it still doesn't work right:

//Make Sure $qty is NOT blank before checking if it is a number
	if (!empty($item_1_qty)) {
//not empty		
	}
	if (intval($item_1_qty) !== $item_1_qty){
			//item 1 qty is not a whole number
			$redirect_to = 'order_form.php?error=6';	
			header("Location: $redirect_to");
			exit();
		}

I get the error message to come up EVERY time. I only want it to come up if the value is NOT a whole number.

[NogDog]

ginginca wrote:

Got a whole other way of thinking about this ... but it still doesn't work right:

//Make Sure $qty is NOT blank before checking if it is a number
	if (!empty($item_1_qty)) {
//not empty		
	}
	if (intval($item_1_qty) !== $item_1_qty){
			//item 1 qty is not a whole number
			$redirect_to = 'order_form.php?error=6';	
			header("Location: $redirect_to");
			exit();
		}

I get the error message to come up EVERY time. I only want it to come up if the value is NOT a whole number.

!== is a type-sensitive comparison, so if $item_1_qty is a string (or any other type that is not integer), then your if (intval($item_1_qty) !== $item_1_qty) will always return false; so you probably just want to use != there.