Question

I’m trying to target the last item in a foreach loop but it’s acting a little strange.

I’ve got two items in the loop, I’ve tried the following:

1. Not set a +/- for the count, this targets the first item.
2. Using -1 as the count also targets the first item.
3. Using +1 as the count doesn’t target any items until you add a 3rd
   item, then it works as intended.

Here’s the code, can anyone help?

<?php $pages = get_pages(array('child_of' => $post->ID, 'sort_column' => 'menu_order'));
foreach($pages as $key => $post)
{
setup_postdata($post);
$fields = get_fields();
?>

<div class="event<?php if( $key == (count( $pages ) +1) ) echo 'last'; ?>">             
</div>
<?php } wp_reset_query(); ?>

Answer 1

This is the way to do it:

<?php 
$pages = get_pages(array('child_of' => $post->ID, 'sort_column' => 'menu_order'));

// keep a record of the number of pages -1
// in order to compare against 0 indexed array key
$pagesNo = count($pages)-1;

foreach($pages as $key => $post)
{
setup_postdata($post);
$fields = get_fields();
?>
<div class="event<?php if( $key == $pagesNo ) echo 'last'; ?>"></div>
<?php } wp_reset_query(); ?>