Почему одновременно подсвечены две вкладки главного меню?

[Serebos]

Здравствуйте.
Создал хелпер для подстветки активной вкладки главного меню.

if(!function_exists('show_active_menu')) {
    function show_active_menu($slug) {

        $ci = & get_instance();

        $result = "";
// для главной страницы
        if($ci->uri->segment(1, 0) === $slug) {
           $result = "class='active'";
        }
// для фильмов/сериалов
        if($ci->uri->segment(3, 0) === $slug) {
           $result = "class='active'";
        }
// для рейтинга
        if($ci->uri->segment(2, 0) === $slug) {
           $result = "class='active'";
        }


        return $result;

    }
}

В "вид меню" добавил следующие записи

 <li <?php echo show_active_menu(0); ?> ><a href="/">Главная</a> </li>
                
                <li <?php echo show_active_menu('films'); ?>><a href="/movies/type/films/">Фильмы</a> </li>
                <li <?php echo show_active_menu('serials'); ?>> <a href="/movies/type/serials/">Сериалы</a> </li>
                <li <?php echo show_active_menu('rating'); ?>> <a href="/main/rating/">Рейтинг фильмов</a> </li>
                <li <?php echo show_active_menu('contact'); ?>> <a href="/">Контакты</a> </li>

Вкладка Главная, Фильмы и Сериалы подсвечены - если активны
такого вида


Но, после открытия страницы Рейтинг подсвечиваются две вкладки


Предполагаю, что причина вот в этой записи

if($ci->uri->segment(2, 0) === $slug)

Но, логически не могу понять причину!

Помогите, пожалуйста, разобраться в ситуации!

СПАСИБО!

 

[Serebos]

При нажатии на рейтинг, вызов show_active_menu('rating') возвращает значение class='active', но также и show_active_menu(0) возвращает значение class='active'

т.е.

              <ul class="nav nav-pills">
                <li class='active' ><a href="/">Главная</a> </li>
                
                <li ><a href="/movies/type/films/">Фильмы</a> </li>
                <li > <a href="/movies/type/serials/">Сериалы</a> </li>
                <li class='active'> <a href="/main/rating/">Рейтинг фильмов</a> </li>
                <li > <a href="/">Контакты</a> </li>

              </ul>

Попытка задать проверку для главной страницы и рейтинга ни к чему не привела

// для главной страницы
        if($ci->uri->segment(1, 0) === $slug && $ci->uri->segment(2, 0) !== 'rating') {
           $result = "class='active'";
        }
        
// для рейтинга
 if($ci->uri->segment(2, 0) === $slug && $ci->uri->segment(1, 0) !== '/') {
           $result = "class='active'";
        }

Каким образом можно устранить присвоение class active для вкладки "Главная" в момент открытия страницы "Рейтинг"??
СПАСИБО!

 

[c0dex]

Посмотреть почему у тебя неверно срабатывает условие. Это называется отладка, за тебя ею никто заниматься не станет.

 

[Serebos]

c0dex, спасибо. Это нужно настроить xdebuger? Я пытался его настроить как-то, два дня ушло, не заработало как следует! В саблайме работаю, но устанавливал phpstorm.
Есть ли какие альтернативы?

 

[Фанат]

отладка - это проверка значений переменных на разных этапах выполнения программы.

 

[c0dex]

Ну дебаж через var_dump

 

[WMix]

если $ci->uri->segment(...) этот метод, то боюсь ты все не правильно понял, и количество параметров отличается, и логика странная

$ci = & get_instance();

и зачем тут амперсанд?

При нажатии на рейтинг, вызов show_active_menu('rating') возвращает значение class='active', но также и show_active_menu(0) возвращает значение class='active'

да, что вернет $ci->uri->segment(3, 0) для /main/rating/?

 

[Serebos]

параметров отличается, и логика странная

WMix, спасибо.

следую программе изучения. описание, это, скорее всего! https://www.codeigniter.com/userguide3/libraries/uri.html?highlight=segment#CI_URI::segment

 

[Serebos]

и зачем тут амперсанд?

вот это пояснение, если его так можно назвать! https://drive.google.com/file/d/1N6fyP57zwrC7sxi9j3uVPMs_MRzv7-jp/view?usp=sharing

 

[Serebos]

да, что вернет $ci->uri->segment(3, 0) для /main/rating/?

ничего не вернет! rating имеет значение (2, 0)

 

[Serebos]

Ну дебаж через var_dump

рано или позже, все равно, придется разобраться с xdebag Буду пробовать вновь!

 

[WMix]

ничего не вернет! rating имеет значение (2, 0)

не знаю что ты там себе придумал "имеет значение (2, 0)",
что вернет $ci->uri->segment(3, 0) для /main/rating/ и чему равно show_active_menu(0) при "/main/rating/" в $ci->uri->segment(3, 0)?

The optional second parameter ... allows you to set the return value of this method when the requested URI segment is missing.

теперь вообще без вопросов

 

[Serebos]

что вернет $ci->uri->segment(3, 0) для /main/rating/ и чему равно show_active_menu(0) при "/main/rating/" в $ci->uri->segment(3, 0)?

нужно подумать... я отпишусь позже

 

[c0dex]

Все, очень, плохо ='(

 

[Serebos]

$ci->uri->segment(3, 0) для /main/rating/

возвращает "0" ноль

 

[Serebos]

show_active_menu(0) при "/main/rating/" в $ci->uri->segment(3, 0)

тоже ноль!

 

[Serebos]

этот комментарий "The optional second parameter ... allows you to set the return value of this method when the requested URI segment is missing. " только сейчас увидел.
Ноль будет всегда, если сегмент отсутствует.

 

[WMix]

твоя функция никуда не годится, show_active_menu('films'); будет возвращать "актив" если в любом из /x/y/z ,будет слово "films"

 

[c0dex]

твоя функция никуда не годится, show_active_menu('films'); будет возвращать "актив" если в любом из /x/y/z ,будет слово "films"

Да не его это код, это кусок говна из курсов. Курсов, где человек вообще не одупляет что он делает и зачем.

 

[WMix]

это кусок говна из курсов

я хотел сказать, но воздержался не видел ни начала ни конца