Adding a class to the last row in a foreach loop

Question

HI all,

This code works perfectly printing all 5 results with borders on the bottom of each list item through CSS.

However the last item id like there to be no border. How could i add a class to the list item on the last iteration?

<?php list($parent) = split('/', $this->url); ?>
<?php  $last_articles = $this->find('/news')->children(array('limit'=>5, 'order'=>'page.created_on DESC')); ?>
<ul id="latest-news">
<?php foreach ($last_articles as $article): ?>
<li>
<?php echo '<h3>'.$article->link($article->title()).'</h3>'; ?>
<?php echo strip_tags(substr($article->content(),0,100)).'...'; ?>
</li>
<?php endforeach; ?>
</ul>

Thanks for your help.

Answer 1

<?php list($parent) = split('/', $this->url); ?>
<?php  $last_articles = $this->find('/news')->children(array('limit'=>5, 'order'=>'page.created_on DESC')); ?>
<ul id="latest-news">
<?php $count = count($last_articles); $num = 0; ?>
<?php foreach ($last_articles as $article): ?>
<li <?php if($num == $count-1){ ?> class="last-item" <?php } ?>>
<?php echo '<h3>'.$article->link($article->title()).'</h3>'; ?>
<?php echo strip_tags(substr($article->content(),0,100)).'...'; ?>
</li>
<?php $num++ ?>
<?php endforeach; ?>
</ul>

BTW, this adds the class "last-item" to the last item processed.

And yes, you should restructure your code so its readable.

Answer 2

You can use the :last-child CSS modifier:

ul.latest-news:last-child { border-bottom: none; }

But it's not so widely supported (yet), so adding an explicit .last class as you're about to do is probably best.