Мужики, у меня скоро будет аудит сайта. И нужна помочь ваша. Есть 2 вопроса. Сразу скажу - возможно детский сад, но всё же они есть. А в инете инфа не совсем для меня понятна.

1. есть ли разница между

if(isset($_SESSION['user']))
{
	if($_SESSION['user'] == 1)
	{
		return true;
	}
}

и

if(isset($_SESSION['user']) && $_SESSION['user'] == 1)
{
	return true;
}

Обычно использую второй. тк вроде как если первое условие не выполнилось, то дальше && не идет.

2. При проверке POST переменной (+ через AJAX) на ЧИСЛО всегда использую проверку if(!is_numeric($_POST['var'])) { return; }

хотя в документации написано проверять число строго функцией is_int
Как правильней?

_____________
..Работает - не трогай!

is_numeric != is_int
Разные вещи.

_____________
Mysql, Postgresql, Redis, Memcached, Unit Testing, CI, Kohana, Yii, Phalcon, Zend Framework, Joomla, Open Cart, Ymaps, VK Api

1. Первый случай не правильный, избыточное второе условие
2.
Вообще не нужно return true;

return (isset($_SESSION['user']) && $_SESSION['user'] == 1);

Для провеки -
is_bool() - Finds out whether a variable is a boolean
is_float() - Finds whether the type of a variable is float
is_numeric() - Finds whether a variable is a number or a numeric string
is_string() - Find whether the type of a variable is string
is_array() - Finds whether a variable is an array
is_object() - Finds whether a variable is an object

правельно

if(isset($_SESSION['user']) && $_SESSION['user'] == 1){
	return true;
}

_____________
Ex3m.com.ua — Активный образ жизни