No veo el problema

[alyciashape]

Partiendo de este código:

Código PHP:

#Años a crear inventario
for ($year=2011; $year<=DATE('Y'); $year++) {
   #Tomamos las compras desde el inicio hasta 2011 por empresa
   $compras = $bd->consulta("SELECT claart, claemp, cantidad, fecha FROM compra 
   WHERE year(fecha)<=$year GROUP BY   claemp, claart");
   
   /*
   foreach ($compras AS $compra) { //Esto funciona.
        echo $compra['claart'];
        echo "<br />";
   }
   */
 
$cantidad = COUNT($compras); //Me da mas de 60 mil por cada año     
 
    for ($compra=0; $compra <= $cantidad; $compra++) { //Esto no me funciona
        echo COUNT($compras);echo "<br />"; //Sigue dando mas de 60 mil pero se imprime una vez por año
        $claart     = $compras[$compra]['claart'];
        $claemp     = $compras[$compra]['claemp'];
        $cantidad   = $compras[$compra]['cantidad'];
        $fecha      = $compras[$compra]['fecha'];       
        //echo "Artículo $claart claemp $claemp cantidad $cantidad fecha $fecha <br />";
                //Esta línea anterior se imprime una y sólo una vez por cada bucle. 3 veces porque recorro años de 2011 hasta ahora
    }
    
}
 

Quería hacerlo con FOR porque tengo entendido que el rendimiento es muy superior a foreach. Sin embargo foreach me gusta mucho porque me parece además de cómodo muy lógico.

Me está fallando el for y no se por qué. La cuestión es que cuando lo escribí la primera vez me iba perfectamente ahora solo me recorre una vez el for. No entiendo por qué

Más raro todavía. Con el for así me imprime los 60 mil holas:

Código PHP:

for ($compra=0; $compra <= $cantidad; $compra++) {  
        echo "hola <br />";     
    }
 

Pero así no. Solo lo hace 1 vez por cada año. 3 en total.

Código PHP:

for ($compra=0; $compra <= $cantidad; $compra++) {          
        $claart     = $compras[$compra]['claart'];
        $claemp     = $compras[$compra]['claemp'];
        $cantidad   = $compras[$compra]['cantidad'];
        $fecha      = $compras[$compra]['fecha'];               
        echo "Artículo $claart claemp $claemp cantidad $cantidad fecha $fecha <br />";
    }
 

No lo entiendo. Nada. Ya está, se puede cerrar. Llevaba mas de una hora con esto sin resultados. Estaba reescribiendo el valor de $cantidad. Mira que busqué y busqué precisamente buscando eso y no lo veía.

Un saludo.

[David]

Iniciado por alyciashape

Quería hacerlo con FOR porque tengo entendido que el rendimiento es muy superior a foreach. Sin embargo foreach me gusta mucho porque me parece además de cómodo muy lógico.

Ese tipo de optimización no vale la pena. Escribe el código de modo que sea fácil de leer y mantener.

[ArkangelGammar]

El inconveninente esta aca:

Código PHP:

   $claart     = $compras[$compra]['claart'];
        $claemp     = $compras[$compra]['claemp'];
        $cantidad   = $compras[$compra]['cantidad'];
        $fecha      = $compras[$compra]['fecha'];  
   

Lo estas llamando como si fuese un array de array. Creo que lo logico, dependendiendo de como te devuelva los datos $bd->consulta, seria algo como:

Código PHP:

          $claart     = $compras['claart'];
        $claemp     = $compras['claemp'];
        $cantidad   = $compras['cantidad'];
        $fecha      = $compras['fecha'];  
   

Y en cuanto a los del FOR y Foreach, concuerdo con David, no vale la pena ese tipo de optimizacion, tenes que realizar codigo sencillo y siempre de mantener, y en tu caso utilizaria un foreach, por ahi eso es lo que te esta dando a confucion.

Espero haber ayudado.

Saludos

[dashtrash]

No, el error ya lo habia encontrado el OP. Estaba usando $cantidad tanto para controlar el bucle, como para guardar 1 campo del resultado.
Pero, si el OP quiere escribir código más "optimizado", primero, no cargues toda la consulta en memoria (aunque php lo hace), cuando se puede procesar secuencialmente sin problema.
Si la query da 60000 resultados, copiar esos 60000 resultados en un array, para luego procesarlos 1 a 1, es tirar la memoria.
Mejor usarlo con un iterador php (mysql_fetch_xx), o con uno que tú te hagas.

Y, después, tanto por velocidad, como por claridad de código:

Código PHP:

for ($compra=0; $compra <= $cantidad; $compra++) {          
       $curCompra=$compras[$compra];
        echo "Articulo ".$curCompra['claart'].
                "claemp  ".$curCompra['claemp'].
                "cantidad ".$curCompra['cantidad'].
                "fecha ".$curCompra["fecha"];
     }
 

Si el procesamiento no requiere parámetros externos (todo lo que haces dentro del bucle depende sólo de los valores de dentro del array) es posible hacerlo con array_map.
Si hay parámetros externos, pero no son demasiados, puedes usar array_map con un closure.(Si son muchos, se pierde legibilidad)

[alvidales]

Iniciado por David

Ese tipo de optimización no vale la pena. Escribe el código de modo que sea fácil de leer y mantener.

Recomiendo más la utilización de un while.