De entrada, según la documentación parece que en futuras versiones de PHP ya no se soportará mysqli
en estilo por procedimientos y habrá que migrar a objetos.
Un error de lógica muy importante que tienes es que para cada consulta creas una conexión y eso no es necesario. Una sola conexión te permite ejecutar todas las consultas de una misma petición y, además, no es necesario cerrarla.
mysqli::close()
Las conexiones de MySQL no persistentes y los conjuntos de resultados son automáticamente destruidos cuando un script de PHP finaliza su ejecución. Por tanto, aunque el cierre explícito de conexiones abiertas y la liberación de conjuntos de resultados sean opcionales, se recomienda hacerlos. Así, se devolverán inmediatamente los recursos a PHP y a MySQL, lo que puede mejorar el rendimiento. Para información detalla, véase la liberación de recursos
En todo caso, para mejorar el rendimiento, lo que hay que liberar son los recursos de juego de resultados.
Para simplificar un poco el proceso, te recomiendo definir constantes (en lugar de variables) para tener los parámetros que usarás al conectar a base de datos, extender tu clase directamente de mysqli
y usar el patrón Singleton para tener siempre solo una conexión activa:
<?php
define('DB_HOST', 'localhost');
define('DB_USER', 'user');
define('DB_PASS', '*****');
define('DB_BASE', 'test');
class Conexion extends mysqli {
// Solo se permite una conexión
private static $_instance = false;
// No permitir instanciar con new Conexion
private __constructor() {}
// Devolver conexión a base de datos
public static function conectar() {
// Si aún no se ha establecido la conexión
if(self::$_instance === false) {
// Conectar
self::$_instance = new self(DB_HOST, DB_USER, DB_PASS, DB_BASE);
if (self::$_instance->connect_error) {
die('Error de Conexión: (' . self::$_instance->connect_errno . ') ' . self::$_instance->connect_error);
}
}
// Devuelve la conexión activa
return self::$_instance;
}
// Este método realmente no es necesario, puedes usar query()
public function ejecutar($query){
// No es necesario asignar a otra variable, se puede devolver directamente
return $this->query($query);
}
}
Nota: Tu clase extiende de mysqli
y, por tanto, tienes acceso a todos los métodos como query(), prepare(), close(), etc.
Cada que necesites acceso a base de datos:
<?php
// Incluir el archivo con la clase
require_once 'conexion.php';
// Obtener instancia de la clase y mantener solo una conexión
$db = Conexion::conectar();
// Ejecutar una consulta
$result = $db->ejecutar('SELECT * FROM tabla');
// Puede ser con el método query() de mysqli, para eso se extendió la clase
$result = $db->query('SELECT * FROM tabla');
Importante: También va a cambiar la forma en que lees los resultados, porque $result
es un objeto y no puedes acceder con procedimientos como mysqli_fetch_assoc($result)
, sino:
// Leer una sola fila
$fila = $result->fetch_assoc();
// Ciclo para leer todas las filas
while($fila = $result->fetch_assoc()) {
var_dump($fila);
}
// Después de leer, si aún te preocupa, puedes liberar recursos
$result->free();
Aunque no es necesario (PHP lo hará por ti), al final de la petición es cuando puedes cerrar la conexión, solo asegúrate que no se van a ejecutar más consultas.
$db->close();
Finalmente, para mejorar la seguridad de tu proyecto y reducir el riesgo de errores, te recomiendo usar consultas preparadas
if( !isset($this->conexion) ) $this->conectar();
.