foreach dentro de for?

[America|UNK]

Hola, estoy usando un foreach(contiene un array) dentro de for y tengo una pregunta, causa problemas de rendimiento al ejecutar? Gracias.

[David]

Depende de para qué estás haciendo, quizás de algún modo se pueda simplificar. Pero sin más datos, es difícil darte una respuesta acertada.

[harvestmoon]

podrias mostrar tu codigo?
a ver si entree todos podemos simplificar o cualkiier cosa saludines!

[America|UNK]

El array tiene 4 valores que son los tipos de mascota y el for con 20 resultados de mssql_fetch_row de este query:

Código PHP:

  $query = /* (SELECT TOP 5 clase_mascota,duenho FROM PERSONAS WHERE clase_mascota='0'
 UNION ALL SELECT TOP 5 clase_mascota,duenho  FROM PERSONAS WHERE clase_mascota=1 UNION ALL etc etc), 
me va dar 4 grupos de 5 personas: Total 20 */

$mascotas = array(0=>"Perro",1=>"Gato",2=>"Conejo",3=>"Ardilla");

for($i=0;$i<20;$i++){
$row = mssql_fetch_row($query);
foreach($mascotas AS $key_mascotas => $val_mascotas){
if($i==(5*$key_mascotas)){
echo "<tr><TH>Top 5 ".$mascotas[$row["clase_mascota"]]."'s</TH></tr>";
}
}
echo "<tr><td>Persona: $row[0]<td></tr>";
} 
   

Lo que estoy haciendo es que se separen en grupo, Ejem: Top 5 Gatos, perros, etc.

SIN el foreach es algo parecido a esto

Código PHP:

  for($i=0;$i<20;$i++){
$row = mssql_fetch_row($query);

if($i==0{
echo "<tr><TH colspan=\"3\">Top 5 $mascotas[0]</TH></tr>";
}

if($i==5){
echo "<tr><TH colspan=\"3\">Top 5 $mascotas[1]</TH></tr>";
}
if($i==10){
echo "<tr><TH colspan=\"3\">Top 5 $mascotas[2]</TH></tr>";
}
if($i==15){
echo "<tr><td colspan=\"3\">Top 5 $mascotas[3]</TH></tr>";
}
} 
   

la parte if($i==(5*$key_mascotas)){ multiplica la comparación del if osea retornan 0,5,10,15 asi como lo anterior pero no quiero hacer muchos if ademas en un futuro le quiero aumentar mas mascotas. Saludos y gracias denuevo

[David]

Es mejor usar un foreach en ese caso, te vas a facilitar el trabajo futuro, evitas código repetitivo, y de paso es más legible.

Saludos .

[harvestmoon]

jaaj seguro q no se te ha enciclado eso jaja me dio hasta dolor de cabeza por q no mejor le haces como dice david deja los foreach
o en tu caso para q necesitabas los for?

[America|UNK]

Ok muchas gracias...

[heymatias]

Si querés optimizar, en vez de mssql_fetch_row usá mysql_fetch_assoc (solo te genera los índices con nombres, no te hace los índices con números), cuando sea posible usá comillas simples y no dobles y es preferible usar comillas dobles a estar concatenando.

Ej

en vez de

Código PHP:

echo "<tr><TH>Top 5 ".$mascotas[$row["clase_mascota"]]."'s</TH></tr>";
 

hacé

Código PHP:

echo "<tr><TH>Top 5 {$mascotas[$row['clase_mascota']]}'s</TH></tr>";
 

Espero te sirva, saludos.