for loop statment...

[crawfd]

I need to limit the number of images displayed to 3. At the moment the while loop is displaying all array images in the variable $MyImages.

• Justin

while(list(,$v) = each($MyImages))
	{


	echo "<a href=\"../images/pics/$v\" target=_blank<br><br>";

}

[NogDog]

You could just add a counter and break out of the loop if it reaches 3:

$counter = 0;
while(list(,$v) = each($MyImages))
{
    echo "<a href=\"../images/pics/$v\" target=_blank<br><br>";
    if(++$counter >= 3)
    {
        break;
    }
} 

[Piranha]

You could have two things that have to be true in the while-loop:

$counter = 0;
while($counter < 3 && list(,$v) = each($MyImages))
{
        $counter++;

    echo "<a href=\"../images/pics/$v\" target=_blank<br><br>";

} 

[Weedpacket]

Or use the for loop statement mentioned in the subject line....

for($i=0; $i<3; $i++)
{
    $v = each($MyImages);
    //...
}

$MyImages might not have that many elements.

for($i=0; $i<min(3, count($MyImages)); $i++)

But that makes for extra work (the count and min both get checked three times)

for($i=0, $count=min(3, count($MyImages)); $i<$count; $i++)

If this isn't going to be called multiple times to list images in $MyImages three at a time, then a temporary array might be useful

$threeImages = array_slice($MyImages, 0, 3);
for($i=0; $i<count($threeImages); $i++)

Since we're going to be looping over all of the elements in $threeImages we don't really need to keep count any more:

foreach($threeImages as $v)

Since this is the only place where $threeImages would be used

foreach(array_slice($MyImages, 0, 3) as $v)

If it is going to be called multiple times, then 0 would be replaced by a counter that starts at 0 and increases by 3 each time. Or the whole $MyImages array could be array_chunk()ed, with an outer loop going through the chunks and an inner loop going through the contents of each chunk.

[NogDog]

I like the final array_slice() idea: clean and fairly clearly self-documenting.