¿Qué es más rápido, switch dentro de foreach o foreach dentro de switch?

[chiwuan]

Hola a todos,

Me hallo implementando un proceso iterativo. Tengo dos formas de plantearlo, me gustaría me dijeran cual es más rápida en cuestión de tiempo de proceso.

FORMA A:

Código PHP:

  foreach($datos as $dato) {
switch($letra) {
case 'A': instruccionA($dato); break;
case 'B': instruccionB($dato); break;
case 'C': instruccionC($dato); break;
case 'D': instruccionD($dato); break;
case 'E': instruccionE($dato); break;
}
} 
   

FORMA B:

Código PHP:

  switch($letra) {
case 'A': 
foreach ($datos as $dato)
instruccionA($dato);
break;
case 'B': 
foreach ($datos as $dato)
instruccionB($dato);
break;
case 'C': 
foreach ($datos as $dato)
instruccionC($dato);
break;
case 'D': 
foreach ($datos as $dato)
instruccionD($dato);
break;
case 'E': 
foreach ($datos as $dato)
instruccionE($dato);
break;
 
} 
   

Obviamente las diferencias son un switch dentro de un foreach o un foreach dentro de un switch.

Si no hay diferencia, también aclárenmelo si pueden. Gracias.

[Appletalk]

Si las funciones se llaman instruccionA, instruccionC, etc :P lo mas rapido es

@call_user_func( 'instruccion' . $letra, $dato );

Si queres hacer benchmarks usa esta funcion

function microtime_diff($a, $b) {
list($a_dec, $a_sec) = explode(' ', $a);
list($b_dec, $b_sec) = explode(' ', $b);
return $b_sec - $a_sec + $b_dec - $a_dec;
}

[chiwuan]

Gracias por el tip, aunque realmente no es ese mi caso. InstrucciónA,B,C reemplazan a una serie de líneas únicas que tengo para cada caso.

Aquí está el código real de lo que tengo:

Código PHP:

      function imprimeDirectorio($cols = 2) {
        $n = ceil(count($this->datos) / $cols);
        $i = 0;
        $this->textoDirectorio.= "<table width='100%' cellpadding='0' cellspacing='0' border='0'><tr><td><ul>";
        foreach ($this->datos as $dato) {
            if ($i == $n) { // Formateo para una columna nueva
                $i = 0;
                $this->textoDirectorio .= "</ul></td><td>";
            }
            $this->textoDirectorio .= "<li><a href='directorio.php?d="; // Inicio de cada iteración
            switch ($this->tipo) {
                case 'et':
                    $this->textoDirectorio .= "ee&i=".$dato['idCliente']."'>".urldecode($dato['Empresa']); break;
                case 'lt':
                    $this->textoDirectorio .= "cc&i=".$dato['idCat']."'>".$dato['Categoria']; break;
                case 'el':
                    $this->textoDirectorio .= "ee&i=".$dato['idCliente']."'>".urldecode($dato['Empresa']); break;
                case 'cl':
                    $this->textoDirectorio .= "cc&i=".$dato['idCat']."'>".$dato['Categoria']; break;
                case 'cc':
                    $this->textoDirectorio .= "ce&i=".$dato['idCliente']."'>".urldecode($dato['Empresa']);    break;
            }
            $this->textoDirectorio .= "</a></li>\n"; // Fin de cada iteración
            $i++;
        }
        return $this->getFrase() . $this->textoDirectorio;
    } 
   

Actualmente lo tengo como un switch dentro de un foreach. Ahora quiero saber si por tiempo de procesamiento (suponiendo que sea un número de usuarios simultáneos considerable), me conviene más repetir algunas líneas de código y pasarlo a la estructura de foreach dentro de cada caso del switch.

Gracias.

[safe]

para empezar las dos opciones que pones no cumplen la misma funcion... y definitivamente la opcion correcta seria la primera.

[chiwuan]

Si hacen lo mismo, y si acaso me equivoqué fue porque teclee el ejemplo en ese momento, pero no te fijes en eso, sino en el tiempo de los procesos.

En términos de procesamiento, no se cuanto cuesta hacer un switch y cuanto cuesta hacer un foreach.

Si es switch toma más tiempo, entonces varios foreach dentro de un switch son mejor opción.

Si el foreach toma más tiempo, entonces un switch dentro de un foreach es mejor opción.

Sabiendo que los foreach múltiples no se cumplirían a menos que el caso del switch se cumpla, quiero saber si la opción que tengo implementada (la forma A) es mejor o si me conviene usar la forma B.

En la forma A uso menos líneas de código, y es más sencilla (al menos para mi), pero quiero eficiencia más que sencillez, asi q estaría dispuesto a usar las líneas de código adicionales si es que lo valen.

[JuanJodm]

A mi parecer la forma b es la más eficiente ya que en la forma a entras al foreach y en cada vuelta del mismo evaluas el switch, cosa que en la forma b no pasa, evaluas el switch una sola vez, se ejecuta el foreach y sales del switch.

Bueno esa es mi opinión, saludos.

[jpinedo]

Yo creo que la segunda es la más eficiente.
Esto porque el switch se evalúa una sola vez y luego se ejecuta un foreach con sus acciones.
En el primer caso, se ejecuta un foreach y para cada elemento se evalúa el switch.

Mejor dicho, si la variable a evaluar en el switch no depende de lo que contenga el array que pasas por el foreach (como en este caso), mejor lo evalúas primero.

No tiene sentido estarse preguntando a cada momento $letra=="A"??, $letra=="B"??, etc... basta con evaluarlo una vez.

Otra cosa sería si esa variable $letra dependiera de $dato... ahí sí habría que evaluarla y definitivamente se tendría que utilizar el primer caso.

Saludos

[jpinedo]

Bueno... tardé demasiado en terminar de responder... Lo que yo digo es exactamente lo que pone JuanJodm.
de todos modos siempre puede evaluar el tiempo que tardan tus scripts en ejecutarse. Haz varias pruebas con una y otra manera y verás de manera experimental si se cumple lo que te pusimos.

Saludos

[safe]

poero si en cambio el contenido de la variable $letra cambia cada vez que pasas por el foreach si o si tenes que usar la forma A, porque si no solo tomaria en cuenta el primer valor de esa variable y despues evaluaria como si todas las otras fuesen la misma.

[Appletalk]

Safe, si eso fuera probable, esta pregunta no estaria planteada.

[safe]

creo que tenes razon... aunque pensandolo bien, si no es asi tampoco entiendo porque esta planteada.