Как отображать поле только тогда, когда оно заполнено?

Question

[tbalero]

На вордпресс-сайте на странице записи для отображения названий категорий без ссылки (категории произвольной таксономии 'genres') используется следующий код:

<?php

$array = array();
$cur_terms = get_the_terms( $post->ID, 'genres');
    foreach( $cur_terms as $cur_term ){
    $array[] = $cur_term->name;
    }
$comma_separated = implode(", ", $array);

echo '<div>Жанры:' . ' ' . $comma_separated . '</div>'; // term1, term2, term3

?>

(*на выходе внутри div-а получается текст вида: Жанры: Жанр1, Жанр2, Жанр3 )

Вопрос: Как нужно изменить данный код, чтобы отображать поле на странице сайта только тогда, когда оно (поле) заполнено? (то-есть только тогда, когда при создании/редактировании записи - для данной записи указана одна или несколько категорий из произвольной таксономии 'genres')

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

Answer 1

[Игорь Воротнёв]

$cur_terms = get_the_terms( $post->ID, 'genres' );

if( $cur_terms && ! is_wp_error( $cur_terms )  ) {

    $array = array();

    foreach( $cur_terms as $cur_term ) {
        $array[] = $cur_term->name;
    }

    $comma_separated = implode( ", ", $array );
    echo '<div>Жанры: ' . esc_attr( $comma_separated ) . '</div>'; // term1, term2, term3

}

Answer 2

[dk-web]

Ну первое, что приходит в голову...

if(!empty($array)){
$comma_separated = implode(", ", $array);
echo '<div>Жанры:' . ' ' . $comma_separated . '</div>'; // term1, term2, term3
}

Comments

[tbalero]

Спасибо, заработал такой код:

<?php

if(!empty($array)){

$array = array();
$cur_terms = get_the_terms( $post->ID, 'genres');
foreach( $cur_terms as $cur_term ){
$array[] = $cur_term->name;
}
$comma_separated = implode(", ", $array);

echo 'Жанры:' . ' ' . $comma_separated . ''; // term1, term2, term3

}

?>

dk-web подскажите плиз, получившийся код - нормальный? или как-то может быть нужно его более правильно написать?

[dk-web]

tbalero: не очень.
вы делаете проверку массива перед его созданием...
в переменную cur_terms вы получаете данные из БД.
Проверяйте ее... if(count($cur_terms)>0) {
foreach( $cur_terms as $cur_term ){
$array[] = $cur_term->name;
}
$comma_separated = implode(", ", $array);

echo 'Жанры:' . ' ' . $comma_separated . ''; // term1, term2, term3

}
Попробуйте так..

[Игорь Воротнёв]

tbalero: не нормальный. Вы сначала проверяете не пуст ли массив $array, и только если не пуст - внутри уже этот самый массив и создаете и наполняете. Этот код никогда не выполнится, если только у вас где-то раньше не определяется переменная $array. dk-web все правильно подсказал, код должен быть:

$array = array();
$cur_terms = get_the_terms( $post->ID, 'genres' );

foreach( $cur_terms as $cur_term ) {
    $array[] = $cur_term->name;
}

if( ! empty( $array ) ) {
    $comma_separated = implode( ", ", $array );
    echo '<div>Жанры:' . ' ' . $comma_separated . '</div>'; // term1, term2, term3
}

[Игорь Воротнёв]

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

[tbalero]

dk-web: Благодарю за помощь!

[tbalero]

Игорь Воротнёв: Благодарю за помощь!

[dk-web]

Игорь Воротнёв: Да, только есть риск, что если записей нет, чего хочет избежать автор... то он гипотетически может получить ошибку в foreach...

[Игорь Воротнёв]

dk-web: да, вы правы. Именно поэтому я упомянул о необходимости рефакторинга. Комментарии тут нельзя редактировать, лень было заново писать :) foreach надо поместить внутрить if а в нем проверять $cur_terms.

[tbalero]

Игорь Воротнёв: изначально предложенный в вашем ответе код - сработал корректно. Подскажите плиз, как именно его нужно отредактировать, чтобы избежать возможных ошибок ("foreach надо поместить внутрить if а в нем проверять $cur_terms." - не ясно, как должна выглядеть данная конструкция)

[Игорь Воротнёв]

tbalero: Придется таки писать :)))

$cur_terms = get_the_terms( $post->ID, 'genres' );

if( $cur_terms && ! is_wp_error( $cur_terms )  ) {

    $array = array();

    foreach( $cur_terms as $cur_term ) {
        $array[] = $cur_term->name;
    }

    $comma_separated = implode( ", ", $array );
    echo '<div>Жанры:' . ' ' . $comma_separated . '</div>'; // term1, term2, term3

}

[Игорь Воротнёв]

tbalero: Вынес еще в отдельный ответ, чтобы будущие поколения не рылись в комментах. Правильными ответами в итоге можете назначить и этот, и мой отдельный.

[tbalero]

Игорь Воротнёв: благодарю!

[Игорь Воротнёв]

tbalero: на здоровье)

[Игорь Воротнёв]

tbalero: посмотрите мой ответ еще раз, я там чуть изменил - убрал лишнюю конкатенацию пробела и обезопасил вывод меток (это хорошая security практика).

[tbalero]

Игорь Воротнёв: Благодарю!