Как правильно использовать тернарный оператор?

Question

[Andre548]

Подскажите, как правильно использовать тернарный оператор? Нужно что бы при нажатии меню было выделено активным.
в hoverUrl я получаю ссылку из $_SERVER['REQUEST_URI']
Не срабатывает второе условие

<div class="border-b">
    <div class="container mx-auto overflow-hidden px-4 sm:px-6">
        <section class="bg-white py-4">
            <div class="list-inside bullet-list-item flex flex-wrap justify-between -mx-5 -my-2">
                <?php foreach ($sort as $key => $value) { ?>
                    <ul>
                        <li class="px-5 py-2">
                            <a <?= ($hoverUrl) ? 'class="text-gray-600 hover:text-orange"' : 'class="text-orange cursor-default"'; ?>
                               href="<?= $value['path'] ?>"><?= $value['title'] ?></a>
                        </li>
                    </ul>
                <?php } ?>
            </div>
        </section>
    </div>
</div>

Comments

[Dark_Dante]

Не делай так. Никогда не смешивай логику и представление.
Правильно (но это не правильно!) сделать так

$textGray =' text-gray-600';
$hover = 'hover:';
$textOrange = 'text-orange';
$cursorDefault = 'cursor-default';

function getStyle (array $styleRules): strung 
{
   return implode(' ', $styleRules);
}

$linkStyle = !empty($hoverUrl) ? getStyle([$textGray, $hover . $textOrange]) : getStyle([$textOrange, $cursorDefault]);

и потом бла бла бла

<a class="<?=$linkStyle?>" href="<?= $value['path'] ?>"><?= $value['title'] ?></a>

А вообще это лютое говнокодерство. Форыч вынести за разметку не мешало бы, тоже в отдельную функцию например. Даже в моем варианте, стили не переменными а константами следовало бы задавать и тд и тп

[Andre548]

Dark_Dante, Ну если по правильному, то ведь классы тоже нельзя в переменные выносить

[Dark_Dante]

Andre548, по правильному надо пользоваться готовыми шаблонизаторами, все уже придумано до нас. Просто чем дальше, тем больше будет такого лапшекода и когда тебе потребуется что нибудь поправить, например поменять text-gray-600 на text-gray-700 ты задолбаешься искать где ты там что хардкодил

Answer 1

[Сергей delphinpro]

Условие всегда истинно, пототму что в $hoverUrl что-то да содержится.
Сравнивайте с урлом в ссылке
$hoverUrl == $value['path'] ? … : … ;

Comments

[Andre548]

<?= $hoverUrl == $value['path'] ? 'class="text-orange cursor-default"' : 'class="text-gray-600 hover:text-orange"'; ?>

При таком варианте он у меня все поля ставит активными

[Сергей delphinpro]

Andre548, значит смотрите, какие данные у вас в $hoverUrl и во всех элементах $sort

Answer 2

[Антон Шаманов]

$_SERVER['REQUEST_URI'] при запуске php под http сервером всегда имеет значение, как ты хотел получить второй вариант я хз

Comments

[Andre548]

Он у меня сравнивает с нужными url из моего массива