isset и if function in php with or operator

Question

У меня есть код PHP для проверки состояния входа пользователя, который проверяет, установлены ли переменные сеанса или установлены файлы cookie. Если какое-либо из условий истинно, тогда это большое разрешение иначе перенаправляется на страницу входа. Код выглядит следующим образом:

if(isset($_SESSION["userid"]) || isset($_COOKIE["userid"]) && isset($_SESSION["username"]) || isset($_COOKIE["username"]) && isset($_SESSION["password"]) || isset($_COOKIE["password"])){ 
} else { 
header("location : register.php"); 
} 

Проблема заключается в том, что если сессия разрушаться (путем закрытия браузера или какой-либо другой причине) он перенаправляет пользователя на страницу входа в систему. Но то, что он должен делать, это считывать данные cookie и предоставлять пользователю доступ, поскольку файлы cookie все еще присутствуют (я проверял, что файлы cookie все еще присутствуют в данных echo cookie).

Answer 1

Вы должны изменить ваше состояние немного, потому что вы путаете его в SESSION и Cookie. Поместите их вместе с && и разделить их с ||, как показано ниже: -

if((isset($_SESSION["userid"]) && isset($_SESSION["username"]) && isset($_SESSION["password"])) || (isset($_COOKIE["userid"]) && isset($_COOKIE["username"]) && isset($_COOKIE["password"]))){ 

    // your action that you want 

} else { 

    header("location : register.php"); 
} 

Примечание: - Будьте осторожны, что одни и те же вещи собираются применять везде (друг на друга страницы и условия), в противном случае вы будете сталкиваться с проблемой ,

Кроме того, зависимость от cookie не является хорошей, поскольку она может быть изменена пользователем в любое время.

Answer 2

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

if( 
    (isset($_SESSION["userid"]) || isset($_SESSION["username"]) && isset($_SESSION["password"])) 
|| 
    (isset($_COOKIE["userid"]) || isset($_COOKIE["username"]) && isset($_COOKIE["password"])) 
){ 
    // Your action 
} else { 
header("location : register.php"); 
} 

Такое заявление будет проверять, если установлен COOKIE или SESSION и проверить или user_id или имя пользователя и пароль. Если вам нужно AND user_id AND username, то замените || в скобках между функциями isset() для этих полей.

Answer 3

&& имеет более высокий приоритет, чем ||, поэтому вам необходимо защитить || с помощью скобок. Кроме того, имея пустую if заявления просто лишнее:

if(!(isset($_SESSION["userid"]) || isset($_COOKIE["userid"]) || 
    !(isset($_SESSION["username"]) || isset($_COOKIE["username"])) || 
    !(isset($_SESSION["password"]) || isset($_COOKIE["password"]))) { 
    header("location : register.php"); 
} 

Answer 4

Я думаю, что ваше состояние if отсутствует в скобках. Для того, чтобы сделать код более удобным для чтения, вы можете создать две функции

function isSessionValid() 
{ 
    return isset($_SESSION["userid"]) && isset($_SESSION["username"]) && isset($_SESSION["password"]); 
} 

function isCookieValid() 
{ 
    return isset($_COOKIE["userid"]) && isset($_COOKIE["username"]) && isset($_COOKIE["password"]); 
} 

, а затем использовать эти функции в вашем случае заявление:

if (isSessionValid() || isCookieValid()) { 

} else { 

}