loop: if last

[ripcurlksm]

I have seen for $i loops which give special instructions if it is the last element in a loop.

I have this "if last in loop, do this" script working, but can anyone revise it? I have seen a simpler version somewhere using $i < satan, etc

// This outputs "1, 2, 3, 4, 5, 6, 7, 8, 9"
$counter = 1; 
$limit = 10;

while($counter < $limit) {

    if ($counter == $limit - 1){
        print $counter;
                
    } else {
    
    print $counter;
    echo ", ";
    
    }
     
$counter++;
} 

[paul_wilkins]

// This outputs "1, 2, 3, 4, 5, 6, 7, 8, 9"
$counter = 1; 
$limit = 10;
echo $counter;
for (var $i = $counter + 1; $i < $limit; $i++) {
    echo ', ' . $counter;
}

[ripcurlksm]

Thats it, thanks!

[Cups]

or just rtrim(); the trailing comma you want to get rid of

$counter = 1; 
$limit = 10;
$output='';

while($counter < $limit) {
 $output . = $counter . ',' ;
}

echo rtrim( $output , ',' ); 

I suppose it depends on where/when you want to echo the values out.

[galen]

what about this

$start = 1; 
$limit = 10;
echo implode(', ', range($start, $limit-1));

[Stormrider]

That's a pretty neat way of doing it

[paul_wilkins]

I like that. I was wanting to use implode earlier, but couldn't recall about the range function.

I'll be adding that to my repertoire now, thanks galen.

[ripcurlksm]

sweet, glad I asked

[galen]

I like that. I was wanting to use implode earlier, but couldn't recall about the range function.

I'll be adding that to my repertoire now, thanks galen.

[ripcurlksm]

How can I apply your script to this while loop? I am trying to put a pipe after every result except the last.

// outputs "red | green | yellow | orange | blue"

$sql = "SELECT DISTINCT category FROM script ORDER BY category ASC";
$result = mysql_query($sql) or die(mysql_error().': '.$sql);
$num = mysql_num_rows($result); 
$count = 0;

while($row = mysql_fetch_assoc($result)) 
{
    $category = $row['category'];
    echo "<a href='view.php?category=$category'>$category</a> ";
    $count++;
    
    if ($num > $count){
    echo " | ";
    }
} 

[AHA7]

// outputs "red | green | yellow | orange | blue"

$sql = "SELECT DISTINCT category FROM script ORDER BY category ASC";
$result = mysql_query($sql) or die(mysql_error().': '.$sql);
$count = 0;
$links = array();

while($row = mysql_fetch_assoc($result)) 
{
    $links[$count++] =  "<a href='view.php?category={$row['category']}'>{$row['category']}</a>";
}

echo implode(' | ', $links); 

[galen]

that whole count variable is unnecessary. keep in my mind this method is a waste of memory if you dont plan on ever using the array.

[paul_wilkins]

That would have been a copy/paste error

[ripcurlksm]

thanks pmw57

galen which would you recommend for efficiency?

[galen]

i'd probably do this

$sql = "SELECT DISTINCT category FROM script ORDER BY category ASC";
$result = mysql_query($sql) or die(mysql_error().': '.$sql);
 
while($row = mysql_fetch_assoc($result)) {
     $links[] =  "<a href='view.php?category={$row['category']}'>{$row['category']}</a>";
}
 
echo implode(' | ', $links);

You could keep the links in a string variable and then rtrim at the end, but i figured keeping a string variable in memory is about the same as an array.

[ripcurlksm]

:: you rule ::