Как заставить условия работать?

Question

[Azami]

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

Помогите исправить, чтобы он работал. Сейчас блок все равно показывается на всех страницах.

<?php $user = cmsUser::getInstance();
if ((($user->is_logged) && ($_SERVER['REQUEST_URI'] != '/page-1' || $_SERVER['REQUEST_URI'] != '/page-2')) 
 
|| 

((!$user->is_logged) && ($_SERVER['REQUEST_URI'] != '/' || $_SERVER['REQUEST_URI'] != '/index.php' || $_SERVER['REQUEST_URI'] != '/page-3'))) { ?>
 
 <div class="is_left_menu_main">
<?php $this->widgets('left-menu-main'); ?>
</div>
 
<?php } ?>

В php слаб, но стараюсь. А тут ничего не выходит.
Заранее благодарю

Answer 1

[Максим Хан-Магомедов]

Какая-то адовая смесь получилась.

Для начала стоит разделить условие "пользователь авторизован"/"пользователь не авторизован". Потом убедиться, что логика сравнения правильная.

if ($user->is_logged) {
  if ($_SERVER['REQUEST_URI'] == '/page-1' || $_SERVER['REQUEST_URI'] == '/page-2') {
    // пользователь авторизован и запрашивает page-1 или page-2
  }
} else {
  if ($_SERVER['REQUEST_URI'] == '/' || $_SERVER['REQUEST_URI'] == '/index.php' || $_SERVER['REQUEST_URI'] == '/page-3') {
    // Пользователь не авторизован и запрашивает /, index.php или page-3
  }
}

Comments

[Azami]

Спасибо.

В общем эту смесь я налопатил только из-за одного.
Я хочу, чтобы вот этот блок

<?php $this->widgets('left-menu-main'); ?>

НЕ показывался на главной если пользователь не авторизован.
Не подскажете как это сделать исходя из моих переменных?

[Максим Хан-Магомедов]

Что-то из этой серии.
if ($_SERVER['REQUEST_URI'] == '/') {
if ($user->is_logged) {
$this->widgets('left-menu-main');
}
}
То бишь он будет показываться, если запрашивают главную страницу и если пользователь авторизован.

[Azami]

ozgg: Но дело в том, что он должен показываться на всех страницах если авторизован, а если не авторизован, то на всех кроме главной

[Максим Хан-Магомедов]

Azami: ну значит нужно условие поменять соответствующим образом :) Что-то типа того (можно «подсушить», но лень).
if ($_SERVER['REQUEST_URI'] == '/') {
if ($user->is_logged) {
$this->widgets('left-menu-main');
}
} elseif ($user->is_logged) {
$this->widgets('left-menu-main');
}

[Azami]

Максим Хан-Магомедов: Теперь он показывается на всех страницах только если авторизован. (((

Если не авторизован, то как на главной, так и на остальных его нет. Хотя его не должно быть только на главной

[Максим Хан-Магомедов]

Azami: да, там во втором условии забыл отрицание добавить (elseif (!$user->is_logged))

Answer 2

[Узаир Ижа]

Во втором случае вместо ! попробуй так
((empty($user->is_logged))