1

Есть такой код

 <? foreach ($query as $data): ?>
    <div class="post"><?=$data[title] ?></div>
<? endforeach ?>

Можно ли при выводе последнего элемента массива добавить контейнеру div класс post-last?

4 ответа 4

5

Такие вещи надо делать средствами CSS.

вместо post-last надо определить класс .post:last-child и никаких изменений в РНР код вносить не нужно

1
<? $query_last = array_pop($query); ?>
<? foreach ($query as $data): ?>
    <div class="post"><?=$data[title] ?></div>
<? endforeach ?>
<div class="post post-last"><?=$query_last[title] ?></div>
2
  • Выдает ошибку Warning: array_pop() expects parameter 1 to be array, object given in
    – n000000b
    Commented 25 янв. 2016 в 15:10
  • Это очень плохой вариант, поскльку в нем приходится дублировать код. Commented 25 янв. 2016 в 15:25
0
<?php $i=1; $counter = count($data)>
<?php foreach($query as $data): ?>
    <div class="post <?php if($i == $counter) echo 'post_last'?>" ><?php echo $data['title'] ?></div>
    <?php $i++; ?>
<?php endfor; ?>

и мой вам совет не используйте short tag. Ибо даже на php.net написано што их использование не желательно. Если массив не ассоциативный то лучше использовать for для таких целей

 <?php $counter = count($data)>
 <?php for($i=0;$i<$counter;$i++): ?>
    <div class="post <?php if($i == $counter-1) echo 'post_last'?>" ><?php echo $query[$i]['title'] ?></div>
 <?php endfor; ?>
2
  • Совсем забыл сказать, что мне известно количество элементов и оно статично (пусть будет 3). Можно ли упростить код, зная это?
    – n000000b
    Commented 25 янв. 2016 в 14:51
  • @n000000b да и так в принципе всё просто. массив какого вида?
    – Naumov
    Commented 25 янв. 2016 в 14:56
0

Можно выделить последний элемент и сравнить значения с ним.

Менее эффективный метод будет выглядеть так:

<?foreach($query as $data){?>
    <div class="post <?= ($data === end($query)) ? "post-last" : "" ?>"><?=$data['title'] ?></div>
<?}?>

Но лучше все-таки использовать получение элемента в начале, чем каждый раз это делать в цикле:

<?
    $end === end($query);
    foreach($query as $data){
?>
    <div class="post <?= ($data === $end) ? "post-last" : "" ?>"><?=$data['title'] ?></div>
<?}?>
12
  • Неплохо но оформите код как следует пожалуйста. Как то был у меня баг связаный с end() и next() по этому не использовал его больше.
    – Naumov
    Commented 25 янв. 2016 в 14:41
  • @Naumov, не вижу никаких проблем в оформлении кода.
    – Alex Krass
    Commented 25 янв. 2016 в 14:46
  • Совсем забыл сказать, что мне известно количество элементов и оно статично (пусть будет 3). Можно ли упростить код, зная это?
    – n000000b
    Commented 25 янв. 2016 в 14:48
  • 2
    @Naumov, все, что Вы написали, являются вашими личными предпочтениями. А следовательно глупо просить кого-то следовать вашим личным необоснованным прихотям по стилю кодирования. Не вижу проблем с short-tags + после 5.4 они не вызывают больше вообще никаких проблем, в т.ч. синтаксис <?= => после упразднения в php.ini опции short_open_tag. Никаких уточнений по версиям в вопросе нет. Одинарные или двойные кавычки - никакой разницы. Укороченный синтаксис if прост и читабельнее в данном случае, его не за раскрывать в полноценный if. Нормальный IDE подсветит код, поэтому переносы являются лишними.
    – Alex Krass
    Commented 25 янв. 2016 в 15:08
  • 3
    @Naumov я вам настоятельно рекомендую унять преподавательский зуд, избавиться от желания всех учить, а вместо него приобрести реальные знания. Для начала попробуйте научиться писать комментарии строго по теме, а также проверять свои утверждения перед тем, как с таким пафосом садиться в лужу. Commented 25 янв. 2016 в 15:32

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.