Immagini in una tabella

[arconte]

Ciao a tutti, sono un neofita del php, quindi non mi insultate, per la stupidità della domanda...Solo che non riesco a trovare proprio la soluzione.
Allora, io devo prendere da un db delle path di una serie di immagini, e metterle in una tabella con un numero di colonne definibile e un numero di righe che sarà conseguente (non mi interessa al momento).
quindi dovrei avere una cosa del tipo

<table>
<tr>
<td>Prima img</td>
<td>Seconda img</td>
<td>Terza img</td>
<td>Quartaimg</td>
</tr>
<tr>
<td>Quinta img</td>
<td>Sesta img</td>
<td>Settima img</td>
<td>Ottava img</td>
</tr>
...
</table>

E allora ho scritto questo:

//connessione al db
<table border="1"> 
<tr> 
<?php $sql="SELECT DISTINCT * FROM tbl_marchi order by descrizione"; 
$results = mysql_query($sql); 
while ($row = mysql_fetch_array($results)) { 
$id_marchio = $row['id']; 
$codicecostruttore = $row['codicecostruttore']; 
$descrizione = $row['descrizione']; 
?> 
<?php for ($i = 1; $i <= 4; ++$i) { 
echo "<td>" ?> 
[img]img/costruttori/<?php echo $codicecostruttore; ?>.jpg[/img] <?php }?></td> 
</tr> <?php } ?> 
</table>

Funziona tutto. Solo che, invece di mettermi una immagine per ogni td, mi mette su tutta la fila la stessa immagine. Poi va a capo e mette la seconda e via discorrendo.
Come faccio a evitare questa cosa?
Qualcuno di voi ne ha una idea?

[nickcv]

metti il codice non inline che così è un incubo da leggere.

comunque dando una lettura veloce ho già visto che copi il valore dell'array in variabili con lo stesso nome:

$ciao = $array['ciao']; 

per fare una cosa del genere ti conviene usare extract:

extract($array); 

[arconte]

Ho modificato il quote, spero che così sia più chiaro.
Non ho ben capito cosa devo fare....

[nickcv]

io eliminerei quel for che è del tutto non necessario nonchè la fonte del tuo problema.

fai una cosa così:

# all'interno del while già strutturato da te

echo $i == 1 ? '<tr>' : NULL;
?>
<td>[img]<?php echo $immagine ?>[/img]</td> 
<?php
echo $i == 4 ? '</tr>' : NULL;

if ($i == 4) {
  $i = 1;
}else{
  $i++;
}

ovviamente prima del while inizializza $i con un valore pari a 1

[FFMM]

non capisco questa esigenza di chiudere tag php, se vuoi un consiglio ,se lavori con un algoritmo creato in php, continua ad usare quello e il testo scrivilo totalmente con echo.

<?php 
//connessione al db 
echo"<table border='1'>  
        <tr>";

$cnt='0';
$sql="SELECT DISTINCT * FROM tbl_marchi order by descrizione";  
$results = mysql_query($sql);  
    while ($row = mysql_fetch_array($results)) {  
        $cnt++;
        $id_marchio = $row['id'];  
        $codicecostruttore = $row['codicecostruttore'];  
        $descrizione = $row['descrizione'];  
        echo "<td>[img]img/costruttori/$codicecostruttore.jpg[/img]</td>";
        if($cnt==4){echo"</tr><tr>";$cnt='0';}
    }
echo"    </tr>    
</table>";

?>

Questo dovrebbe andare.. facci sapere

[arconte]

Ciao scusate il ritardo...

Ok, ho capito dove sbagliavo. Grazie mille ragazzi, ora sono riuscito a far funzionare il tutto!!!

E grazie anche per i consigli: cercherò di scrivere meno tag php e codice un po' più funzionante!