Mostrar resultados de query en varias columnas

[Cain]

En vista a la pregunta del mensaje <a href='ir.asp?http://www.forosdelweb.com/mensaje.asp?id=98893' target='_blank'>http://www.forosdelweb.com/mensaje.asp?i...</a>, que no es la primera vez que veo, os dejo este código de &quot;columnización&quot; de resultados. Si alguien tiene algún código más optimizado, que no deje de ponerlo.

Código:

$columnes = 4;
echo "<tr><td colspan=$columnes>$rows Resultados </td></tr>";
if ($rows=0){echo "<tr><td colspan=$columnes>No hay resultados en la BD.</td></tr> ";}

for ($i=1; $row = mysql_fetch_row ($result); $i++) {
$resto = ($i % $columnes);
if ($resto == 1) {echo "<tr>";}
	echo "<td>$row[1]</td>"; 
if ($resto == 0) {echo "</tr>";}
}
if ($resto <> 0) {
$ajust = $columnes - $resto;
for ($j = 0; $j < $ajust; $j++) {echo "<td> </td>";}
if ($ajust < $columnes) {echo "</tr>";}
}

Como ya habréis visto, $columnes es el número de columnas en que queréis distribuir el resultado.

<div style="font:small Verdana;">Proximamente...</div>
<div align="center" style="font:bold small Verdana;"><a style="text-decoration:none;" href="http://www.traduim.com" target="_blank">Traduim</a>: El portal de los traductores</div>

[josemi]

Siento ser tan quisquilloso, pero creo que no hace falta al final comprobar que $adjust sea menor que $columnes, ya que para llegar a ese codigo $resto debe ser distinto de 0, es decir, vale 1..($columnes-1) y por tanto $adjust siempre sera menor que $columnes. Resumiendo, que yo cambiaria esta linea:

Código:

if ($ajust < $columnes) {echo "</tr>";}

por esta otra:

Código:

echo "</tr>";

No es que el codigo este mal, es simplemente que la condicion siempre se va ha cumplir, lo que hace innecesaria su comprobacion.

Cain, cree que esto se merece su lugar en las FAQs, ¿no crees?.

Josemi

Aprendiz de mucho, maestro de poco.

[Cain]

Pues tienes toda la razón, ese if está de más.
Gracias

<div style="font:small Verdana;">Proximamente...</div>
<div align="center" style="font:bold small Verdana;"><a style="text-decoration:none;" href="http://www.traduim.com" target="_blank">Traduim</a>: El portal de los traductores</div>