Эхо с сокращением, если не ведет себя должным образом

Я программирую на PHP, я использую "стенографию, если" для вывода некоторого HTML-кода на страницу, но он ведет себя странным образом.

echo '<div id="filter_bar">
<ul>';
    echo '<li><a href="?select=trending"' . ($_GET['select'] == "trending") ? ' class="filter_selected">Trending</a></li>' : '>Trending</a></li>';
    echo '<li><a href="?select=most_picked"' . ($_GET['select'] == "most_picked") ? ' class="filter_selected">Most Picked</a></li>' : '>Most Picked</a></li>';
    echo '<li><a href="?select=newest"' . ($_GET['select'] == "newest" || empty($_GET['select'])) ? ' class="filter_selected">Newest</a></li>' : '>Newest</a></li>';
echo '</ul></div>';

В результате код, который я получаю в результате этого

class="filter_selected">Trending</a></li> class="filter_selected">Most Picked</a></li> class="filter_selected">Newest</a></li>

Как видите, теги начального списка не отображаются... но они появляются, если я заменяю первый период '.' в каждой строке с запятой ','.

Так что это работает, с запятыми

Должен ли я использовать запятую здесь? Повсюду в сети показывается точка "."

Источник

5 ответов

Решение

Может быть, вы можете сделать свою жизнь немного проще:

echo '<div id="filter_bar"><ul>',
    '<li><a href="?select=trending"', $_GET['select'] == "trending" ? ' class="filter_selected">' : '>', 'Trending</a></li>',
    '<li><a href="?select=most_picked"', $_GET['select'] == "most_picked" ? ' class="filter_selected">' : '>', 'Most Picked</a></li>',
    '<li><a href="?select=newest"', ($_GET['select'] == "newest") || empty($_GET['select']) ? ' class="filter_selected">' : '>', 'Newest</a></li>',
'</ul></div>';

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

Затем вы использовали круглые скобки там, где они не были нужны (см. Документы о приоритетах операторов), но там, где они нужны, вы их не использовали (последний случай).

Кроме того, лучше заполнить эти значения переменными заранее, чтобы вы могли легче отлаживать вещи (и не смешивать $_GET с выходом, чтобы предотвратить смешивание выходной логики с входными переменными).

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

$class_trending = $_GET['select'] == "trending" ? ' class="filter_selected"' : '';
$class_most_picked = $_GET['select'] == "most_picked" ? ' class="filter_selected"' : '';
$class_newest = ($_GET['select'] == "newest") || empty($_GET['select']) ? ' class="filter_selected"' : '';

echo '<div id="filter_bar"><ul>',
    '<li><a href="?select=trending"', $class_trending, '>Trending</a></li>',
    '<li><a href="?select=most_picked"', $class_most_picked, '>Most Picked</a></li>',
    '<li><a href="?select=newest"',$class_newest , '>Newest</a></li>',
'</ul></div>';
Источник

Причина в том, что:

echo '<li>' . true ? 'aaa' : 'bbb'; даст тебе ааа,

потому что то же самое с '<li>1' ? 'aaa' : 'bbb'

И вы должны сделать так:echo '<li>' . (true ? 'aaa' : 'bbb');

Источник

Не знаю, какая у вас проблема, но этот код для меня выглядит ужасно. Я бы сделал это так:

в коде PHP я бы подготовил переменные в первую очередь.

$sections = array(
  'newest'      => 'Newest',
  'trending'    => 'Trending',
  'most_picked' => 'Most Picked',
);
if (empty($_GET['select']) OR !$choice = array_search($sections,$_GET['select'])) {
  $choice = 'newest';
}

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

<div id="filter_bar">
 <ul>
<? foreach ($sections as $sect => $name): ?>
  <li>
<a href="?select=<?=$sect?><? if ($choice == $sect) ?>" class="filter_selected"<? endif ?>><?=$name?></a>
  </li>
<? endforeach ?>
 </ul>
</div>
Источник

Одним из возможных решений является

    echo "<li><a href='?select=newest'"; 
echo ($_GET['select'] == "newest" || empty($_GET['select'])) ? ' class="filter_selected">Newest</a></li>' : '>Newest</a></li>';
Источник

Измените свои круглые скобки на следующее:

echo '<div id="filter_bar">
<ul>';
    echo '<li><a href="?select=trending"' . ($_GET['select'] == "trending" ? ' class="filter_selected">Trending</a></li>' : '>Trending</a></li>');
    echo '<li><a href="?select=most_picked"' . ($_GET['select'] == "most_picked" ? ' class="filter_selected">Most Picked</a></li>' : '>Most Picked</a></li>');
    echo '<li><a href="?select=newest"' . (($_GET['select'] == "newest" || empty($_GET['select'])) ? ' class="filter_selected">Newest</a></li>' : '>Newest</a></li>');
echo '</ul></div>';

Если вы этого не сделаете, PHP не знает, каково именно ваше состояние. Также обратите внимание на приоритет операторов, поскольку это объясняет, почему он работает с использованием запятых.

Кстати:?: Называется троичным оператором.

Источник
Другие вопросы по тегам