Запись GET в SESSION

[Sonya]

Здравствуйте! Нужна помощь. Набросала скрипт, но уверена, что он некорректен:

if ($_SESSION['code'] === null){
   if($_GET['code'] === null){
   $_SESSION['code'] = '796';
  }
   elseif($_GET['code'] !== null){
   $_SESSION['code'] = $_GET['code'];
  }
}else{
$code = $_SESSION['code'];	
}

Суть - через GET-запрос, записываю параметр в сессию. Т.к. страница в определенный момент обновляется, но уже без GET, параметр передается уже через сессию. Как сделать код корректным? Помогите, пожалуйста.

[laimas]

$_GET - это суперглобалный массив, который всегда есть, но если не было запросов то пустой.

Если вывод предупреждений подавлен, то то к примеру $_GET['a'] вернет NULL, если такового запроса не было. А пустой массив возвращает false, и это даже не породит предупреждений:

if(!$_GET) массив пуст или наличие конкретной переменной в массиве при подавлении предупреждений - if(!$_GET['code']). Зачем же проверять заполнен ли массив или существование в нем ключа, то есть были GET запросы или нет, строгой проверкой да еще и на null? Что null, что false, что пустая строка вам вернет "ложь". А вот действительно проверка существования ключа в массиве, без проверки его значения, это if(!isset($_GET['code'])).

Тоже самое и с $_SESSION.

[Sonya]

Пробовала до этого кода именно if(isset($_GET['code'])) только без логического отрицания, там у меня с логикой были проблемы. Страница обновляется и при обновлении, GET не передается (такое условие), поэтому записываю в сессию, которая при обновлении остается.

[laimas]

Если я правильно понял, то нужно следующее:

if(isset($_GET['code']) && $code = (int)$_GET['code']) $_SESSION['code'] = $code;
$code = $_SESSION['code'] ?: 769;

Не помню в какой версии, но вот такое

$code = $_SESSION['code'] вызывало ошибку. В версиях начиная с 5 не наблюдал такого, хотя и не со всеми и работал, так что имейте ввиду если что.

[Sonya]

$code = $_SESSION['code'] вызывало ошибку. В версиях начиная с 5 не наблюдал такого, хотя и не со всеми и работал, так что имейте ввиду если что.

Версия PHP 5.6.
При обновлении страницы уже не будет передан параметр GET, не передаст ли он NULL в сессию?

[laimas]

не передаст ли он NULL в сессию?

Каким образом? Условие записи в сессию какое?

С учетом ранее сказанного, при подавлении предупреждений, сократим код вообще для одного условия записи в сессию:

if($code = (int)$_GET['code']) $_SESSION['code'] = $code; //если не было GET запроса, чему будет равно $code и выполнится ли это условие? 
$code = $_SESSION['code'] ?: 769;

Проверьте.

[Sonya]

Хорошо, сейчас попробую.

[Sonya]

Работает! Спасибо. Тот код, что я показала в самом начале тоже срабатывал, но со второго раза. А этот работает превосходно. Сейчас попробую "прогнать" в разных вариациях.

[Vlasenko Fedor]

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

так более читабельное и без нотисов

[laimas]

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

Ой ли так? )