Запись GET в SESSION
Здравствуйте! Нужна помощь. Набросала скрипт, но уверена, что он некорректен:
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, параметр передается уже через сессию. Как сделать код корректным? Помогите, пожалуйста. |
$_GET - это суперглобалный массив, который всегда есть, но если не было запросов то пустой.
Если вывод предупреждений подавлен, то то к примеру $_GET['a'] вернет NULL, если такового запроса не было. А пустой массив возвращает false, и это даже не породит предупреждений: if(!$_GET) массив пуст или наличие конкретной переменной в массиве при подавлении предупреждений - if(!$_GET['code']). Зачем же проверять заполнен ли массив или существование в нем ключа, то есть были GET запросы или нет, строгой проверкой да еще и на null? Что null, что false, что пустая строка вам вернет "ложь". А вот действительно проверка существования ключа в массиве, без проверки его значения, это if(!isset($_GET['code'])). Тоже самое и с $_SESSION. |
Пробовала до этого кода именно if(isset($_GET['code'])) только без логического отрицания, там у меня с логикой были проблемы. Страница обновляется и при обновлении, GET не передается (такое условие), поэтому записываю в сессию, которая при обновлении остается.
|
Если я правильно понял, то нужно следующее:
if(isset($_GET['code']) && $code = (int)$_GET['code']) $_SESSION['code'] = $code; $code = $_SESSION['code'] ?: 769; Не помню в какой версии, но вот такое $code = $_SESSION['code'] вызывало ошибку. В версиях начиная с 5 не наблюдал такого, хотя и не со всеми и работал, так что имейте ввиду если что. |
Цитата:
При обновлении страницы уже не будет передан параметр GET, не передаст ли он NULL в сессию? |
Цитата:
С учетом ранее сказанного, при подавлении предупреждений, сократим код вообще для одного условия записи в сессию: if($code = (int)$_GET['code']) $_SESSION['code'] = $code; //если не было GET запроса, чему будет равно $code и выполнится ли это условие? $code = $_SESSION['code'] ?: 769; Проверьте. |
Хорошо, сейчас попробую.
|
Работает! Спасибо. Тот код, что я показала в самом начале тоже срабатывал, но со второго раза. А этот работает превосходно. Сейчас попробую "прогнать" в разных вариациях.
|
$code = 769; if (false === empty($_GET['code'])) { $code = (int)$_GET['code']; $_SESSION['code'] = $code; }так более читабельное и без нотисов |
Цитата:
|
Часовой пояс GMT +3, время: 14:45. |