Запись GET в SESSION

[Vlasenko Fedor]

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

[laimas]

че смутило строгое сравнение, оно немного быстрее работает, чем с далнейшим приведением типов

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

[Vlasenko Fedor]

есть ли значение, но не проверяется какое

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

[laimas]

Не в скорости дело, а в читаемости

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

[Vlasenko Fedor]

$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;
}

[laimas]

Poznakomlus,

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

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

[Vlasenko Fedor]

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

[laimas]

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

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

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

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

[Vlasenko Fedor]

то это не смертельно

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

ваш код пропускает на сохранение ненужное сценарию значение

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

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

[laimas]

то есть вы пердлагаете писать код не взирая на предупреждения, ошибки

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

где этот сценарий узрели, это ваши домыслы

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

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

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

echo $_SESSION['code'];