[SOLUCIONADO] Liberar memoria automáticamente después de cada consulta

[SeaPirates]

Hola, me preguntaba si esta forma de liberar la memoria de las consultas de forma automática sería correcta:

Código PHP:

  class database
{

    public function __construct() 
    {
    
        $this->MySQLi['hostname'] = 'localhost';
        $this->MySQLi['username'] = '';
        $this->MySQLi['password'] = '';
        $this->MySQLi['database'] = 'db';
        
        $this->Connection = mysqli_connect($this->MySQLi['hostname'], $this->MySQLi['username'], $this->MySQLi['password'], $this->MySQLi['database']); 
        
        if(!$this->Connection)
        {
            die('Error de conexion: ' . mysqli_connect_error());
        }
        
    }
    
    public function query($sql)
    {
        $this->Query = $this->Connection->query($sql);
        return $this->Query;
                //Liberar la memoria de la consulta
        $this->Query->free();
    }
    
    public function close()
    {
        $this->Connection->close();
    }
    
    
} 
   

¿O hay que liberar la memoria manualmente después de utilizar fetch_assoc, num_rows, etc? Estoy haciendo un sitio web, y no se realmente cual seria el modo óptimo de liberar la memoria.

[Triby]

De entrada, es incorrecto generar una conexión para cada consulta realizada.

Luego, si tienes un return, todo lo que esté después, dentro del mismo método, será ignorado.

PHP cuenta con un "recolector de basura" que se encarga de liberar recursos (memoria, conexiones, etc.) y no es necesario hacerlo manualmente, a menos que tus scripts asi lo requieran, por la cantidad de datos manipulados, por falta de memoria disponible, etc.

[SeaPirates]

Gracias por tu respuesta, al final he optado por utilizar PDO:

Código PHP:

  class database
{
 
    public function connect() 
    {
    
        try {
            $this->Connection = new PDO('mysql:dbname=DB;host=127.0.0.1', 'root', '');
            $this->Connection->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            
            return $this->Connection;
        } catch (PDOException $e) {
            echo "ERROR: " . $e->getMessage();
        }
        
        
    }
    
} 
   

Creo que de ese modo seria correcto, no? No se a que te referías con lo de una conexión para cada consulta

[Triby]

Ouch, parece que vi mal tu código... sorry, fue mi error!

[SeaPirates]

Ah, entonces marco tema como resuelto. Muchas gracias