Как скрыть элемент, если он не заполнен?

Question

[Александр Коновалов]

Здравствуйте! Как скрывать элемент, если он не заполнен и показывать, если заполнен?

<?php if (!empty($row)) { ?>
<li class="size">Размер <?php echo $row['attr-size']; ?>мм</li>
 <script>
$(".size").hide();
</script>
<?php } ?>

Пробую так, но элемент скрывается даже, если attr-size заполнен

Answer 1

[Александр]

Добавить стиль для скрытия элемента.

<li class="size" style="display: <?= empty($row['attr-size']) ? 'none' : 'block'; ?>">Размер <?php echo $row['attr-size']; ?>мм</li>

Comments

[Александр Коновалов]

Александр оно скрывает даже, если размер заполнен

[Александр]

Александр Коновалов, Исправил код. А так?

[Александр Коновалов]

Александр,

<li class="size" style="display: <?= empty($row['attr-size']) ? 'none' : 'block'; ?>">Размер <?php echo $row['attr-size']; ?>мм</li>

вот так заработало

[Александр Коновалов]

Александр, спасибо!

Answer 2

[Алексей Уколов]

Почему бы не попробовать в условие проверки добавить то, что реально по вашей логике влияет на отображение? Вы говорите «даже если attr-size заполнен», так почему в условии у вас это не проверяется?

Comments

[Александр Коновалов]

<?php if (!empty($row['attr-size'])) { ?>
 <li>Размер <?php echo $row['attr-size']; ?>мм</li>
  <li>Назначение <?php echo $row['attr-coating']; ?></li>
<?php } else {?>
<li>Назначение <?php echo $row['attr-coating']; ?></li>
 <? }?>

А если нужно скрывать сразу несколько атрибутов, как это упростить?

[Алексей Уколов]

Да никак особо. Ну можно цикл по их названиям сделать, чтобы код не дублировать, но кардинально это ничего не изменит.
Конкретно в вашем куске кода «назначение» можно вообще вынести из условия и убрать else — раз в обеих ветках выводится одинаково.

[Александр Коновалов]

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

[Виктор Кожухарь]

Александр Коновалов, Я не рекомендую вам в шаблонах, которые выводят html, придумывать какие-то хитрые конструкции и что-то "упрощать". В файлах, где выводится html всё должно быть максимально читаемо, и тут должно быть как можно меньше запутанного кода.
Кстати, советую вам использовать альтернативный синтаксис PHP для html-кода без фигурных скобок. Например:

<?php if (!empty($row['attr-size'])) : ?>
 <li>Размер <?php echo $row['attr-size']; ?>мм</li>
  <li>Назначение <?php echo $row['attr-coating']; ?></li>
<?php else :?>
<li>Назначение <?php echo $row['attr-coating']; ?></li>
 <? endif;?>