Запретить выводить форму входа повторно

Всем привет, сейчас попробовал то что вы говорили вроде получилось, проверил в самом конце скрипта, верное решение судить вам, код привожу:

Спойлер: КонтроллерUser

PHP:

1.     public function actionLogin()
2.     {
3.        # создаем объект класса User()
4.        $user = new User();
5.  
6.        # Проверяем какой метод используется на веб странице
7.        if ( $_SERVER['REQUEST_METHOD'] == 'POST' )
8.        {
9.            # фильтруем данные
10.            $checkARGS = [
11.                 'user_login' => FILTER_SANITIZE_STRING,
12.                 'user_password' => FILTER_SANITIZE_STRING
13.             ];
14.  
15.             $inputFIELD = filter_input_array(INPUT_POST, $checkARGS);
16.  
17.            # Проверяем существует ли пользователь
18.            $userID = $user->checkUserData($inputFIELD['user_login'], $inputFIELD['user_password']);
19.  
20.            # проверяем на ошибки, если их нет попадаем в кабинет.
21.            if ($userID === true)
22.            {
23.                # Перенаправляем его в личный кабинет
24.                header("Location: /cabinet");
25.                exit;
26.             }else{
27.                 $_SESSION['errors'][] = $userID;
28.                 header('Location: /user/login');
29.                 exit;
30.             }
31.         }
32. /*или такая запись  if($user->isGuest() == true) обе работают, какая будет стабильней?*/
33.         if($user->isGuest() === true)
34.         {
35.            # Подключаем наш шаблон
36.            require_once(ROOT . '/views/user/login.php');
37.            return true;
38.         }else{
39.            # Перенаправляем его в личный кабинет
40.            header("Location: /cabinet");
41.            exit;
42.         }
43.     }

И метод который проверяет гость или не гость в самом низу его вызываю и сверяю со значением true:

PHP:

1.     public function isGuest()
2.     {
3.         if (isset($_SESSION['uid']))
4.         {
5.             return false;
6.         }
7.         return true;
8.     }

Правильно ли я делаю проверку user-а, на показ шаблона или редиректа?

 

в данный момент я спрашиваю за вывод шаблона и его проверку.
а на счет пароля я его шифрую методом password_hash(), просто для него еще фильтр применил,
ах да забыл, хешировать при проверки полей не думаю?
Эта же фильтрация делается для проверки полей соответствуют ли они действительности данных, которые поступают в форму, если ошибаюсь поправте.

еще вопрос, какая запись будет правильней?
такая запись:

PHP:

1. if($user->isGuest() == true)

или эквивалентна будет правильней?

PHP:

1. if($user->isGuest() === true)

или вообще хватило бы и такой записи без сравнения?

PHP:

1. if($user->isGuest())

Вопрос на засыпку, нужна ли эта проверка при выводе формы например, редактирование данных или любой другой страницы?

 

первыми строчками не срабатывает, а вот в конце срабатывает.

 

а как закрыть страницу смены пароля? если он просто перешел по пути в адресной строке, вообще не догоняю, что проверить нужно, ключ или id?
Этот метод с проверкой формы не подходит, всегда высвечивается форма.
--- Добавлено ---

так тоже сработало.

 

сделал вот таким образом:

PHP:

1.         if(!$user->isGuest() && !isset($_SESSION['r_uid']))
2.         {
3.             # Перенаправляем его в личный кабинет
4.            header("Location: /cabinet");
5.             exit;
6.         }

первая проверка проверяет является ли пользователь гостем, вторая проверка проверяет не находиться ли restore_uid в сессии если нет редирект на ту же страницу кабинет.

 

хорошо, с этим справился, вопрос такой, при проверке например ключа, я создаю ссессию и использую ее в другом контроллере, при успешном завершении, в какой момент мне нужно уничтожить сессию перед редиректом?
--- Добавлено ---

хотя так не работает, работает только по отдельности,
сработало с оператором ИЛИ || почему так?

 

понимаешь если не проверить гость ли он, а на самом деле находится в кабинете, то он сможет открыть страницу доступную только гостям) и наоборот доступную только авторизованным, думаю всегда лучше проверять.

 

ай то я вообще написал, например я сохранил в сессии ключ из базы ну или ID, при успешном завершении когда нужно уничтожать эти данные?
--- Добавлено ---

у меня логика присутствует)

 

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

 

ах ну да точно, предлагаешь переделать уменьшить код? на 3 строки)

 

да можно наверное, но так вообще будет не понятно что и зачем оно нужно)
уж лучше видеть false или true.
или ты советуешь писать так как мне показали?

 

хорошо так и сделаю, если не забуду.