Optimizando código PHP para ahorrar recursos

[nachopro]

Buenas tardes, en esta oportunidad les quiero hacer una consulta:

los sitúo el caso de que usamos mod_rewrite por parte de apache para hacer nuestro sitio con urls amigables.

ahora bien, una función típica puede ser algo como:

Código PHP:

  function trozarUrl($url) 
{ 
    $trozos_tmp = explode('/', $url); 
    unset($url); // Agiliza el sistema o libera recursos realmente importantes 
     
    foreach ($trozos_tmp as $clave => $valor) 
    { 
        if (!empty($valor)) 
        { 
            $trozos[$clave] = trim($valor); 
        } 
        unset($trozos_tmp[$clave]); // Agiliza el sistema o libera recursos realmente importantes? 
    } 
     
    return $trozos; 
}  
   

En el código está la pregunta (repetida dos veces).

Quiero que me den su opinión si conviene o no realizar esta limpieza de "basura".

es decir, se libera un recurso preciado (como es la memoria) y vale la pena frente a la labor de la función unset?

Esto se aplica a cualquier función, procedimiento o lo que sea en PHP.

Espero sus respuesta.

[Saganxis]

Mi humilde opinión:
El uso de unset creo es una buena práctica, pero no creo q en cualquier lugar. Por ejempo, aca

Código PHP:

  foreach ($trozos_tmp as $clave => $valor) 
    { 
        if (!empty($valor)) 
        { 
            $trozos[$clave] = trim($valor); 
        } 
        unset($trozos_tmp[$clave]); // Agiliza el sistema o libera recursos realmente importantes? 
    }  
   

no creo q el unset deba ir dentro de un foreach, yo por lo menos lo pondría cuando termina la función, o despues del foreach..

Como dije mi humilde opinión sin haberlo usado muchas veces, asique espero respuestas de otros q sepan más!!

[nachopro]

Iniciado por Saganxis

Mi humilde opinión:
...
no creo q el unset deba ir dentro de un foreach, yo por lo menos lo pondría cuando termina la función, o despues del foreach..
...

Mirá, lo hice para ir liberando los datos que ya se procesaron....
es decir, si ya evalué el dato en la posición 1 de la matríz $trozos_tmp, o bien lo descarto o lo paso a la matriz $trozos, por lo tanto podría borrar $trozos_tmp[1]

no te parece?

ojo, tmb es mi humilde opinión

[nicolaspar]

Yo creo que no. Lo que estas haciendo es liberar memoria, una url no será muy pesada (a no ser que tenas 16mb de ram:P), y poniendo en la balanza este caso creo que el que esta dentro de la iteración no aporta nada y puede hasta sacar rendimiento, lo mismo con los if.

El que esta afuera puede ser, aunque yo suelo hacer directamente un $url = explode('/', $url); pisando la memoria. Si es mejor o no no sabría decirte, igual es muy poco el proceso que se tiene como para preocuparse en este ejemplo. Igual, yo lo haría así (que no quiere decir que sea mejor eh!:P):

Código PHP:

  function trozarUrl($url){
    $url = explode('/', $url);
    $return = array();
    foreach ($url as $c => $v) $return[$c] = $v;
    return $return;
}
print_r(trozarUrl($_SERVER['REQUEST_URI']));  
   

[nachopro]

nico, pero en ese caso copiaría las URLs que sean vacías :P

pero sí, era a lo que iba. si la relación consumo de micro / consumo de memoria valía la pena :P

igual sigo leyendo opiniones y consejos!

[nicolaspar]

Nacho, yo dejaría la validación de url vacías a apache con un simple error 404, que cada parte haga lo suyo ;), acá que nadie se rasque y labure :P

Con la memoria es todo un tema tenes que ponerla en la balanza, aunque repito, el ejemplo no consume nada, liberarla será mas trabajo que lo que ganes de espacio en memoria.

El mayor consumo de los scripts están en las consultas a la db y loops anidados, y sobre todo en malas praxis ;).