0

Вообщем я столкнулся с такой проблемой: Мне надо сделать проверку сохранена ли cookie ['user'] или ['admin']. Если нет, то выводить в header болк с кнопкой входа/регистрации, а если есть, то проверить isset($_COOKIE['admin']). Если да, то вывести блок с кнопками добавить товар и выход, если isset($_COOKIE['admin']) неверно, то просто кнопку выхода. Весь код рабочий, если по одиночке делать проверку(Например:if(!isset($_COOKIE['admin'])):), то выведет кнопки добавить товар и выход, а если (if(!isset($_COOKIE['user'])):), то только кнопку выхода. Вот основной код:

<?php 
  if(!isset($_COOKIE['user']) || !isset($_COOKIE['admin'])):
     ***Кусок html разметки***
?>
<?php else: ?>
     <?php if(isset($_COOKIE['admin'])): ?>
         ***Кусок html разметки***
     <?php endif; ?>
     ***Кусок html разметки***
<?php endif; ?>

У меня не получается всё это объединить, чтобы была одновременная проверка двух cookie.

if(!isset($_COOKIE['user']) || !isset($_COOKIE['admin'])):
Не рабочий код(но в моём уме он должен работать))

Может кто-то знает как эту проблему решить?

<?php                      
   if(!isset($_COOKIE['user']) || !isset($_COOKIE['admin'])) {                         
      echo('                             
         HTML Code                         
      ');                     
   } else {                         
      if(isset($_COOKIE['admin'])) {                             
         echo('                                 
           HTML Code                             
         ');                         
   };                         
   echo('                             
      HTML Code                         
   ');                     
   };                 
?>

Код выше не решает проблему и пробный вариант. Это код создания куки при авторизации:

<!-- В auth.php (авторизация) -->
    else if ($user['admin'] != 0) {
        setcookie("admin", $user['name'], time() + (3600 * 12 * 30), "/"); // 1(имя куки) 2(у переменной $user берём имя, которое берётся с бд) 3(время жизни куки)
        header('location: /');
        session_destroy();
        exit();
    } else {
        setcookie("user", $user['name'], time() + (3600 * 12 * 30), "/"); // 1(имя куки) 2(у переменной $user берём имя, которое берётся с бд) 3(время жизни куки)
        header('location: /');
        session_destroy();
    };
14
  • Может быть проблема в том, что у Вас тело условия if(!isset($_COOKIE['user']) || !isset($_COOKIE['admin'])) пустое? :)
    – MyZik
    20 июл 2021 в 6:52
  • а какие значения лежат в куках 'user' и 'admin'? 20 июл 2021 в 6:57
  • Увы добавляя тело условию, ничего не меняется. Как работало только при одном условии, так и работает (
    – Kirich
    20 июл 2021 в 7:19
  • То есть вы сейчас в вопросе написали не тот код, который пробовали, но просите нас найти в нем ошибку? В том коде, который мы не видим? Вам не кажется это несколько нелогичным? 20 июл 2021 в 7:20
  • @Ипатьев Не совсем понял ваше недовольство. Как я понял мне MyZik предположил, что у меня не работает условие из-за того, что в нём нету тела. Я добавил и проблема не ушла. Как была, так и осталась. В куках 'user' и 'admin' лежат выгруженные из бд имена владельцев аккаунтов. В последующем они используются для обращения к ним.
    – Kirich
    20 июл 2021 в 7:25

2 ответа 2

1

В простом коде проще разобраться. У опытных программистов есть несколько приёмов, позволяющих упростить код типа вашего. Во-первых надо дать осмысленные имена выражениям, которые проверяются в if'ах:

<?php

$isUser = isset($_COOKIE['user']);
$isAdmin = isset($_COOKIE['admin']);

?>

Во-вторых лучше проверять позитивные условия (нашим мозгам так проще воспринимать утверждения):

<?php if ($isUser || $isAdmin): ?>
    <?php if ($isAdmin): ?>
        кнопка "Добавить товар"
    <?php endif ?>
    кнопка "Выход"
<?php else: ?>
    кнопка "Логин"
<?php endif ?>

Как видите, так гораздо проще.


UPD: Обратите внимание, что кука -- просто HTTP-заголовок, который отправляет пользователь (любой пользователь может отправить "admin"). Если вам надо ограничить доступ к админскому функционалу, то следует использовать сессии.

2
  • Вооо! То что нужно, спасибо всем, кто пытался мне помочь, но этот способ оказался рабочим.
    – Kirich
    20 июл 2021 в 9:02
  • 1
    Егор, я понимаю что вопрос про булеву логику, но все-таки всегда стоит смотреть на шаг вперёд и заодно предупредить читателей о принципальной уязвимости (или даже скорее полной бессмысленности) такой "защиты" 20 июл 2021 в 9:32
0

Почему не написать так?

if (isset($_COOKIE['admin']) || isset($_COOKIE['user'])) {
    if (isset($_COOKIE['admin'])) {
        //admin code
    } else {
        //user code
    }
} else {
    //login html
}
2
  • пожалуйста, постарайтесь оставлять чуть более развёрнутые ответы. дополнить ответ можно, нажав править 20 июл 2021 в 8:35
  • Попробовал ваш способ, но проблема таже. if (isset($_COOKIE['admin'])) { работает, а if (isset($_COOKIE['admin']) || isset($_COOKIE['user'])) { не работает, как бы пропуская его условие.
    – Kirich
    20 июл 2021 в 8:48

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.