Javascript-форум (https://javascript.ru/forum/)
-   Серверные языки и технологии (https://javascript.ru/forum/server/)
-   -   Запись GET в SESSION (https://javascript.ru/forum/server/70703-zapis-get-v-session.html)

Sonya 26.09.2017 20:02

Запись 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, параметр передается уже через сессию. Как сделать код корректным? Помогите, пожалуйста.

laimas 26.09.2017 20:31

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

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

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

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

Sonya 26.09.2017 20:45

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

laimas 26.09.2017 20:53

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

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


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

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

Sonya 26.09.2017 21:05

Цитата:

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

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

laimas 26.09.2017 21:10

Цитата:

Сообщение от Sonya
не передаст ли он NULL в сессию?

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

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

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


Проверьте.

Sonya 26.09.2017 21:17

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

Sonya 26.09.2017 21:34

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

Vlasenko Fedor 26.09.2017 23:49

$code = 769;
if (false === empty($_GET['code'])) {
    $code = (int)$_GET['code'];
    $_SESSION['code'] = $code;
}
так более читабельное и без нотисов

laimas 27.09.2017 05:56

Цитата:

Сообщение от Poznakomlus
if (false === empty($_GET['code'])) {
$code = (int)$_GET['code'];

Ой ли так? )


Часовой пояс GMT +3, время: 14:45.