laimas,
че смутило строгое сравнение, оно немного быстрее работает, чем с далнейшим приведением типов
if (!empty($_GET['code']))
именно так, потому как код читается

Дело не в копеечном выигрыше, а в том, что проверяется есть ли значение, но не проверяется какое, а сценарий при этом ожидает число. Если запросом прислать "вот вам, нате", то ваш код схавает его и в сессию запишет 0, которым сценарий потом оперировать должен? Ноль в качестве идентификатора (не знаю что это, но к примеру), это только при очень богатой фантазии. )

проверку и валидацию типов получаемых значений, это к ТС http://php.net/manual/ru/filter.filters.validate.php
Не в скорости дело, а в читаемости

Ну это дело вкуса, читаемо не читаемо. Ваш код схавает непотребное, а зачем?

$code = filter_input(INPUT_GET, 'code', FILTER_VALIDATE_INT);
if (false === empty($code)) 
{
	$_SESSION['code'] = $code;
}
else //$code 0, Null, false
{
	$code = 769;
}

$code = filter_input(INPUT_GET, 'code', FILTER_VALIDATE_INT);
if (empty($code))//$code 0, Null, false
{
	$code = 769;
}
else 
{
	$_SESSION['code'] = $code;
}

:victory:

Poznakomlus,

Это я знаю, но зачем, если гораздо проще if($a = (int)$_GET['key']) используем $a, ведь другого ничего и не ожидается?

Об этом и был вопрос. )

$_GET['key'] - это ,будет Notice при отсутствии в адресной строке одноименного гет параметра
empty не генерит Notice, так же в php7 тернарный оператор не генерит

Об этом уже писалось, а на реальном сервере нотайсы нужно отключать, и коли разработчик контролирует ситуацию, то это не смертельно.

Я не о empty разговор завел, а о том, что ваш код пропускает на сохранение ненужное сценарию значение, и причина не в empty, а в логике.

Я сомневаюсь, что у автора темы есть метод/функция которой перенапраляется ввод, и которая делает проверки всех входных данных, используя при этом фильтры или нет. Просто в данном случае если уж вас так беспокоят нотайсы, то хватило бы isset() && (int). А то сперва "сомнительная забота о чем-то", а потом прицепом еще и фильтры. :)

то есть вы пердлагаете писать код не взирая на предупреждения, ошибки
а затем скомпилить php без вывода ошибок и радоватся, так привыкнете говнокодить
где этот сценарий узрели, это ваши домыслы так как вы не знаете как дальше работаю с $code
к примеру

if($code === 0){
	echo 'параметр code должен быть числом';
}

:haha:

Предупреждения, это не ошибки, но могут быть предпосылками к ним, поэтому в режиме отладки вывод их включен, но и отладкой надо заниматься не на удаленном сервере. На удаленном же сервере вывод их должен быть подавлен. Так что "предлагаю писать не взирая ни на что" я и не писал, да и речь не об этом.



Да елки палки, вот ваш код, запустите и посмотрите результат.

$_GET['code'] = 'text';

$code = 769;
if (false === empty($_GET['code'])) {
    $code = (int)$_GET['code'];
    $_SESSION['code'] = $code;
}

echo $_SESSION['code'];