[SOLUCIONADO] Migración de una clase que gestiona las conexiones a BD de mysql a mysqli

[DandyCC]

Buenas. Hasta ahora siempre he utilizado una clase que encontré por internet que gestionaba las conexiones con la bd haciendo uso de mysql. Quiero migrar a mysqli manteniendo la clase, para así no tener que tocar todo el código. He intentado modificarla yo sólo pero me es imposible, así que si alguien pudiera ayudarme se lo agradecería, porque sólo de pensar en modificar cada script que accede a la BD me entran sudores fríos

Mil gracias!

Esta es la clase:

Código PHP:

<?php
/* Clase encargada de gestionar las conexiones a la base de datos */
Class DB {
    private $servidor='localhost';
    private $usuario='root';
    private $password='';
    private $base_datos='dbname';
    private $link;
    private $stmt;
    private $array;
    
    static $_instance;
    
    /* La función construct es privada para evitar que el objeto pueda ser creado mediante new */
    private function __construct() {
        $this->conectar();
    }
    
    /* Evitamos el clonaje del objeto. Patrón Singleton */
    private function __clone(){ }
    
    /* Función encargada de crear, si es necesario, el objeto. Esta es la función que debemos llamar desde fuera de la clase para instanciar el objeto, y así, poder utilizar sus métodos */
    public static function getInstance() {
        if (!(self::$_instance instanceof self)) {
            self::$_instance=new self();
        }
        return self::$_instance;
    }
    
    /* Realiza la conexión a la base de datos */
    private function conectar() {
        $this->link=mysql_connect($this->servidor, $this->usuario, $this->password);
        mysql_select_db($this->base_datos,$this->link);
        @mysql_query("SET NAMES 'utf8'");
    }
    
    /* Realiza la desconexión de la base de datos */
    public function desconectar() {
        mysql_close($this->link);
    }
    
    /* Método para ejecutar una sentencia sql */
    public function ejecutar($sql) {
        $this->stmt=mysql_query($sql,$this->link);
        return $this->stmt;
    }
    
    /* Método para obtener una fila de resultados de la sentencia sql */
    public function obtener_fila($stmt,$fila){
        if ($fila==0) {
            $this->array=mysql_fetch_array($stmt);
        }
        else {
            mysql_data_seek($stmt,$fila);
            $this->array=mysql_fetch_array($stmt);
        }
        return $this->array;
    }
    
    // Devuelve el último id del insert introducido
    public function lastID() {
        return mysql_insert_id($this->link);
    }
}
?>
 

Y era muy fácil de usar. Por ejemplo, para conectar, simplemente:

Código PHP:

$bd=DB::getInstance();
if(!$bd) { // error en la conexion }
 

Para hacer una consulta simple:

Código PHP:

$sql = "SELECT nombre, apellidos FROM tbl_user WHERE iduser = 1";
$stmt = $bd->ejecutar($sql);
if (!$stmt) {// error en la consulta}
 

Para recoger los resultados:

Código PHP:

if (mysql_num_rows($stmt) > 0) {
    while ($row=$bd->obtener_fila($stmt,0)) {
        echo $row['nombre'] . " " . $row['apellidos'];
    }
}
 

[Alexis88]

Los cambios que vas a tener que realizar, no son tan significativos, sobretodo porque tu clase no es muy grande. En el manual oficial indican la manera de cómo implementar la extensión MySQLi, te dejo los enlaces:

- mysqli_connect: http://www.php.net/manual/es/mysqli.construct.php
- mysqli_query: http://www.php.net/manual/es/mysqli.query.php
- mysqli_fetch_array: http://www.php.net/manual/es/mysqli-...etch-array.php
- mysqli_data_seek: http://www.php.net/manual/es/mysqli-....data-seek.php
- mysqli_insert_id: https://www.php.net/manual/es/mysqli.insert-id.php
- mysqli_num_rows: http://www.php.net/manual/es/mysqli-result.num-rows.php

Recuerda que con la extensión MySQLi, tienes la posibilidad de trabajar de la forma orientada a objetos, te sugiero poner atención a eso en la explicación del manual, hay ejemplos.

Saludos

[DandyCC]

Ya ví los cambios que tendría que hacer, en principio simplemente cambiar las funciones mysql por mysqli, pero no funciona haciendo esto.
Sé que se puede usar poo ahora, y lo usaría si empezara un proyecto nuevo, pero no quiero usarlo en este porque tendría que cambiar todos los scripts donde se usa esta clase, por eso prefiero usar el modo procedimental.
Gracias igualmente.

[Alexis88]

Ojo que no solamente se trata de cambiar nombres, fíjate bien en los parámetros que se le pasa a cada función y el orden de las mismas.

[DandyCC]

Joer tienes razón, era por eso!! Muchas gracias. Ya funciona!

Por cierto, repasando el manual he visto que en esa clase no hay una función para cerrar la conexión ni para liberar la memoria. ¿Es muy malo que no lo haya? He probado esto y no da ningún warning ni error. Es suficiente así no?

Código PHP:

public function close() {
    mysqli_free_result($this->stmt);
    return mysqli_close($this->link);
}
 

[Alexis88]

La extensión MySQLi sí tiene lo que buscas:

- mysqli_free_result: http://www.php.net/manual/es/mysqli-result.free.php
- mysqli_close: http://www.php.net/manual/es/mysqli.close.php

Nuevamente, te insto a que leas bien lo que indica el manual, la manera en como se trabaja con respecto a la extensión MySQL es similar, pero no igual.

Saludos

[DandyCC]

Ya había leído la documentación, de hecho de ella saqué esas 2 líneas de código, sólo quería confirmar que lo estaba haciendo bien, puesto que parece que sabes de esto.
Gracias igualmente

[Alexis88]

Bueno, yo uso mysqli_free_result cuando ya no voy a darle uso a los resultados de una consulta a la BD, el cierre de la conexión es otra cosa.

Un modelo básico:

Código PHP:

$conexion = mysqli_connect("servidor", "usuario", "password", "bd");
 
if (mysqli_connect_error())
    exit ("No se pudo realizar la conexión");
 
$consulta = mysqli_query($conexion, "SELECT * FROM tabla");
 
if (mysqli_num_rows($consulta)) {
    while ($datos = mysqli_fetch_array($consulta)) {
        echo "Dato 1: " . $datos[0] . "<br />";
        echo "Dato 2: " . $datos[1] . "<br />";
        echo "Dato 3: " . $datos[2] . "<br /><br />";
    }
    mysqli_free_result($consulta);
}
else {
    echo "No se encontraron datos en la consulta";
}
 
mysqli_close($conexion);
 

Adáptalo a tus necesidades, saludos.

[DandyCC]

Ok, lo haré como indicas.

Muchas gracias por tu ayuda.

Saludos.