[SOLUCIONADO] Problema con conexión a BD y clases

[Alexis88]

Hola amigos del foro, hace unas semanas, publiqué un hilo acerca del problema del que trataré ahora, en aquel momento creí que lo había solucionado, pero después de analizar bien todo y realizar múltiples pruebas, comprobé que no fue así.

Sucede lo siguiente:

Tengo un formulario de datos, los cuales envío a un archivo que invoca a cuatro clases: para la conexión, para desinfectar los datos (XSS, SQL Injection), para validarlos (que los datos respeten el formato) y para guardarlos en la BD (esta clase también realiza otras consultas).

El problema se suscita cuando intento utilizar más de 1 método de la clase de guarda los datos, el código de ese archivo es el siguiente:

Código PHP:

<?php
//Incluimos el archivo de conexión a la Base de datos y nos conectamos
include("conexion.php");
$miConexion = new Cls_Conexion();
 
//Incluimos el archivo de seguridad de datos
include("seguridad.php");
$desinfectar = new Cls_Desinfeccion();
 
//Incluimos el archivo de consultas a la Base de datos
include("consultas.php");
$miConsulta = new Cls_Consultas($miConexion->conexion);
 
//Incluimos el archivo de scripts PHP
include("scripts.php");
$scripts = new Cls_Scripts();
 
//Código
$codigo = $scripts->codigo();
$verificarCodigo = $miConsulta->codigo($codigo);
if ($verificarCodigo->num_rows) {
    do {
        $codigo = $scripts->codigo();
        $verificarCodigo = $miConsulta->codigo($codigo);
    } while ($verificarCodigo = $miConsulta->codigo($codigo));
    $verificarCodigo->free();
}
 
//Nombre
$nombre = (strlen($_POST["nombre"]) && isset($_POST["nombre"]) && !is_null($_POST["nombre"])) ? $scripts->validarNomApe($desinfectar->sanitize($miConexion->conexion, $_POST["nombre"])) : $scripts->formulario();
if (is_null($nombre)) $scripts->formulario("Debe de ingresar su nombre en un formato válido");
 
//Apellidos
$apellidos = (strlen($_POST["apellidos"]) && isset($_POST["apellidos"]) && !is_null($_POST["apellidos"])) ? $scripts->validarNomApe($desinfectar->sanitize($miConexion->conexion, $_POST["apellidos"])) : $scripts->formulario();
if (is_null($apellidos)) $scripts->formulario("Debe de ingresar sus apellidos en un formato válido");
 
//E-Mail
$email = (strlen($_POST["email"]) && isset($_POST["email"]) && !is_null($_POST["email"])) ? $scripts->validarEMail($desinfectar->sanitize($miConexion->conexion, $_POST["email"])) : $scripts->formulario();
if (is_null($email))
    $scripts->formulario("Debe de ingresar su E-Mail en un formato válido");
else {
    $verificarEmail = $miConsulta->email($email);
    if ($verificarEmail->num_rows) {
        $scripts->formulario("El E-Mail ingresado se encuentra registrado. Por favor, ingrese otro.");
        $email = NULL;
        $verificarEmail->free();
    }
}
 
//Domicilio
$domicilio = (strlen($_POST["domicilio"]) && isset($_POST["domicilio"]) && !is_null($_POST["domicilio"])) ? $scripts->validarDomicilio($desinfectar->sanitize($miConexion->conexion, $_POST["domicilio"])) : $scripts->formulario();
if (is_null($domicilio)) $scripts->formulario("Debe de ingresar su dirección domiciliaria en un formato válido y utilizando menos de 256 caracteres");
 
//Fecha de nacimiento
$fechanac = (strlen($_POST["fechanac"]) && isset($_POST["fechanac"]) && !is_null($_POST["fechanac"])) ? $scripts->validarFechNac($desinfectar->sanitize($miConexion->conexion, $_POST["fechanac"])) : $scripts->formulario();
if (is_null($fechanac)) $scripts->formulario("Debe de ingresar la fecha de su nacimiento en un formato válido");
 
//País
$pais = (strlen($_POST["pais"]) && isset($_POST["pais"]) && !is_null($_POST["pais"])) ? $scripts->validarPaisCiudad($desinfectar->sanitize($miConexion->conexion, $_POST["pais"])) : $scripts->formulario();
if (is_null($pais)) $scripts->formulario("Debe de elegir un país de la lista");
 
//Ciudad
$ciudad = (strlen($_POST["ciudad"]) && isset($_POST["ciudad"]) && !is_null($_POST["ciudad"])) ? $scripts->validarPaisCiudad($desinfectar->sanitize($miConexion->conexion, $_POST["ciudad"])) : $scripts->formulario();
if (is_null($ciudad)) $scripts->formulario("Debe de elegir una ciudad de la lista");
 
//Añadimos todos los datos a un array que será enviado a la función de procesamiento de datos
$datos = array(
            "codigo" => $codigo,
            "nombre" => $nombre,
            "apellidos" => $apellidos,
            "email" => $email,
            "domicilio" => $domicilio,
            "fechanac" => $fechanac,
            "pais" => $pais,
            "ciudad" => $ciudad
        );
        
$bandera = true; //Comodín para determinar si se guardarán o no los datos
foreach ($datos as $dato)
    if (is_null($dato)) {
        $bandera = false;
        break;
    }
 
if ($bandera)
    //Guardamos los datos del formulario en la Base de datos
    if ($miConsulta->guardarForm($datos))
        echo "Los datos han sido guardados satisfactoriamente";
    else 
        echo "Ha ocurrido un error al intentar guardar los datos";
?>
 

La conexión la realizo de esta forma:

Código PHP:

<?php
class Cls_Conexion {
    public $conexion;
 
    function __construct() {
        //Conexión a la base de datos
        $servidor = "localhost"; //Nombre del servidor
        $usuario = "root"; //Nombre de usuario en tu servidor
        $password = "miclave"; //Contraseña del usuario
        $base = "bd_formulario"; //Nombre de la Base de Datos
 
        //Se realiza la conexión
        return $this->conexion = new mysqli($servidor, $usuario, $password, $base) or exit("Se ha producido un error en la conexión al servidor");
    }
 
    function __destruct() {
        $this->conexion->close();
    }
}
?>
 

Y la clase que contiene los métodos que realizan consultas a la BD es el siguiente:

Código PHP:

<?php
class Cls_Consultas {
    public $conexion;
 
    function __construct($conex) {
        $this->conexion = $conex;
    }
 
    function paises() {
        return $this->conexion->query("CALL USP_listCountries()");
    }
 
    function ciudades($pais) {
        return $this->conexion->query(sprintf("CALL USP_listCities(%d)", $pais));
    }
 
    function codigo($codigo) {
        return $this->conexion->query(sprintf("CALL USP_validateCode('%s')", $codigo));
    }
 
    function email($email) {        
        return $this->conexion->query(sprintf("CALL USP_validateEmail('%s')", $email));
    }
 
    function guardarForm($datos) {
        return $this->conexion->query(sprintf("CALL USP_saveForm('%s', '%s', '%s', '%s', '%s', '%s', %d, %d)",
                $datos["codigo"],
                $datos["nombre"],
                $datos["apellidos"],
                $datos["email"],
                $datos["domicilio"],
                $datos["fechanac"],
                $datos["pais"],
                $datos["ciudad"]
            ));
    }
} 
?>
 

Luego de varias pruebas, he notado que el problema se da luego de la llamada al método codigo con el que genero un código numérico aleatorio, ese bloque se ejecuta sin problemas, incluso probé insertando una línea de impresión antes del bucle Do-While para ver si de verdad estaba trabajando el método y no me mostró la impresión, con lo cual, comprobé que todo marchaba bien con dicho método, sin embargo, cuando intento llamar a los métodos email (con el que busco en la BD si ya se registró un email como el ingresado en el formulario) y guardarForm (para guardar los datos en la BD), el método email no muestra el mensaje de advertencia aún cuando ingreso un email existente en la BD y luego de ejecutar el método guardarForm, se muestra el segundo mensaje que predeterminé para cuando ocurra un problema al intentar guardar los datos.

Ambos métodos invocan a dos procedimientos almacenados cada uno, que, realiza una consulta en la BD y guarda los datos, respectivamente. He probado colocando queries sencillas como INSERT INTO... o SELECT * FROM..., pero el resultado es exactamente el mismo, incluso probé invocando a otro método, como paises para mostrar la lista de países en la BD, pero tampoco funciona, sin embargo, he probado creando una nueva instancia tanto para la conexión a la BD como para la clase de consultas antes de invocar a los dos métodos restantes (email y guardarForm) y de este modo sí funciona:

Código PHP:

<?php
//E-Mail
$email = (strlen($_POST["email"]) && isset($_POST["email"]) && !is_null($_POST["email"])) ? $scripts->validarEMail($desinfectar->sanitize($miConexion->conexion, $_POST["email"])) : $scripts->formulario();
if (is_null($email))
    $scripts->formulario("Debe de ingresar su E-Mail en un formato válido");
else {
    $miConexion2 = new Cls_Conexion();
    $miConsulta2 = new Cls_Consultas($miConexion2->conexion);
 
    $verificarEmail = $miConsulta2->email($email);
    if ($verificarEmail->num_rows) {
        $scripts->formulario("El E-Mail ingresado se encuentra registrado. Por favor, ingrese otro.");
        $email = NULL;
        $verificarEmail->free();
    }
}
 
//Aquí va el resto del código hasta antes del guardado de datos
 
if ($bandera) {
    $miConexion3 = new Cls_Conexion();
    $miConsulta3 = new Cls_Consultas($miConexion3->conexion);
 
    //Guardamos los datos del formulario en la Base de datos
    if ($miConsulta3->guardarForm($datos))
        echo "Los datos han sido guardados satisfactoriamente";
    else 
        echo "Ha ocurrido un error al intentar guardar los datos";
}
?>
 

Instancias a ambas clases antes de cada llamada a los métodos, todo marcha bien, incluso el método email muestra el mensaje de advertencia cuando se ingresa un email existente.

Mi pregunta es, ¿por qué tengo que instanciar dos veces a los métodos de conexión y consultas para que todo marche bien?

Saludos y, de antemano, gracias por la deferencia.

P.D.: El enlace que puse al inicio, los redirigirá al hilo que publiqué hace unas semanas sobre este tema, ahí podrán tener más detalles del código, aunque el problema se centra en lo que muestre en éste.

[xSkArx]

Siempre que.he visto a alguien trabajar con clases utiliza require_once en vez.de include, puede que nada tenga que.ver con tu problema

[Alexis88]

Iniciado por xSkArx

Siempre que.he visto a alguien trabajar con clases utiliza require_once en vez.de include, puede que nada tenga que.ver con tu problema

Claro, no tiene nada que ver porque su función es prácticamente la misma, además, hubiera tenido problemas con las cuatro clases que incluyo. El problema se da luego de invocar al primero de los tres métodos que uso, pareciera que luego de invocarlo, se clausurara la instancia a la clase, mas no la conexión porque, como verás en el código, utilizo la variable de conexión para invocar métodos de otras clases, todas ellas sin problema alguno (las probé una por una).

[Alexis88]

¿Será correcto que instancie a la clase y a la conexión para cada vez que vaya a utilizar un método de la clase o estoy fallando en algo?, pues solo así funciona.

[marlanga]

Yo te recomendaría dos cosas.
Las clases que no necesiten atributos, declararas estáticas con métodos estáticos. Así no necesitarás instanciarlas, podrás usar sus métodos directamente con:

Código PHP:

MiClase::metodoEstatico();
 

Es difícil adivinar qué error está ocurriendo. Prueba una cosa. Comenta las consultas una a una, y ejecuta tu código, para ver cuál es la que está matando la conexión.
por ejemplo, para empezar, comenta toda la validación y desinfección de código. Pon directamente $codigo="ABC012"; Y ejecuta una vez.
Si el resto de validaciones funciona, la culpa la tiene el procedimiento almacenado que ejecuta código.
Si sigue fallando, comenta otro campo más, por ejemplo email, y pon uno a pelo, como con el código.

Hazlo así hasta que compruebes exactamente quién está matando la conexión.

[alexperaza]

Código PHP:

error_reporting(E_ALL);
ini_set("display_errors", 1);
 

implementa esto y publica que errores te tira

[Alexis88]

marlanga, seguí tu consejo, deshabilité las validaciones y desinfecciones y coloqué datos de forma directa, lo mismo con el código que se autogeneraba con un métdo de la clase Cls_Consultas(). Obtuve exactamente los mismos resultados que en las pruebas que había realizado antes de publicar este hilo: Basta con que utilice un método de la clase, para que, después de usarlo satisfactoriamente, la instancia se pierda. En este caso, sucedió con el método email(), pero lo mismo sucedía con el método que generaba el código.

Esta es la clase con los métodos:

Código PHP:

<?php
class Cls_Consultas {
    public $conexion;
 
    function __construct($conex) {
        $this->conexion = $conex;
    }
 
    function paises() {
        return $this->conexion->query("CALL USP_listCountries()");
    }
 
    function ciudades($pais) {
        return $this->conexion->query(sprintf("CALL USP_listCities(%d)", $pais));
    }
 
    function codigo($codigo) {
        return $this->conexion->query(sprintf("CALL USP_validateCode('%s')", $codigo));
    }
 
    function email($email) {        
        return $this->conexion->query(sprintf("CALL USP_validateEmail('%s')", $email));
    }
 
    function guardarForm($datos) {
        return $this->conexion->query(sprintf("CALL USP_saveForm('%s', '%s', '%s', '%s', '%s', '%s', %d, %d)",
                $datos["codigo"],
                $datos["nombre"],
                $datos["apellidos"],
                $datos["email"],
                $datos["domicilio"],
                $datos["fechanac"],
                $datos["pais"],
                $datos["ciudad"]
            ));
    }
} 
?>
 

Y esos son los procedimientos almacenados críticos:

Código MySQL:

/* PA que verifica que el código no se encuentre registrado */
 
CREATE DEFINER=`root`@`localhost` PROCEDURE `USP_validateCode`(IN `code` VARCHAR(10))
    LANGUAGE SQL
    NOT DETERMINISTIC
    CONTAINS SQL
    SQL SECURITY DEFINER
    COMMENT ''
SELECT codigo FROM datos WHERE codigo = code
 
 
/* PA que verifica que el email ingresado, no se encuentre registrado */
 
CREATE DEFINER=`root`@`localhost` PROCEDURE `USP_validateEmail`(IN `mail` VARCHAR(80))
    LANGUAGE SQL
    NOT DETERMINISTIC
    CONTAINS SQL
    SQL SECURITY DEFINER
    COMMENT ''
SELECT email FROM datos WHERE email = mail
 
/* PA que guarda los datos del formulario en la BD luego de todas las validaciones */
 
CREATE DEFINER=`root`@`localhost` PROCEDURE `USP_saveForm`(IN `codigo` VARCHAR(10), IN `nombre` VARCHAR(150), IN `apellidos` VARCHAR(150), IN `email` VARCHAR(80), IN `domicilio` VARCHAR(255), IN `fechanac` DATE, IN `pais` INT(11), IN `ciudad` INT(11))
    LANGUAGE SQL
    NOT DETERMINISTIC
    CONTAINS SQL
    SQL SECURITY DEFINER
    COMMENT ''
INSERT INTO datos VALUES (codigo, nombre, apellidos, email, domicilio, fechanac, pais, ciudad)
 

Hay otros dos PA que utilizo en otros archivos, con ellos no tuve problemas pues solamente invoco a uno de los métodos de la clase, por cada archivo, sin embargo, si, por ejemplo, intento listar los datos de una tabla invocando otro método, que a su vez, invoca a un PA, éste no se ejecuta, produciéndose el mismo problema que con los otros 3 métodos en cuestión.

En síntesis, cuando invoco a un método de la clase, funciona sin problemas, pero si, en el mismo archivo, invoco a otro método o al mismo nuevamente, no funcionan por el problema que les expliqué. Como verán en el código, instancio a otras tres clases, de las cuales, utilizo más de una vez a los métodos de dos de ellas y no se produce el problema que ocurre con la clase que interactúa con la BD.

[marlanga]

Prueba a hacer un

$miConexion->conexion->next_result();

Justo despues de ejecutar un procedimiento almacenado, despues de recuperar todos los datos necesarios, y antes de ejecutar el siguiente call.

Por ejemplo, en el codigo (2 lineas nuevas):

Código PHP:

//Código
$codigo = $scripts->codigo();
$verificarCodigo = $miConsulta->codigo($codigo);
if ($verificarCodigo->num_rows) {
    do {
        $miConexion->conexion->next_result();
        $codigo = $scripts->codigo();
        $verificarCodigo = $miConsulta->codigo($codigo);
    } while ($verificarCodigo = $miConsulta->codigo($codigo));
    $verificarCodigo->free();
}
$miConexion->conexion->next_result();
 

[marlanga]

Por otro lado, cuando yo quiero comprobar que un valor no existe en una base de datos antes de insertarlo, lo que hago es poner ese campo de la tabla con la restricción UNIQUE. Cuando ejecuto la insercción, compruebo si ha ocurrido un error, y sabiendo el número de error, sé lo que ha ocurrido y actúo en consecuencia. Y así, solo he ejecutado una insercción, en vez de un select y una insert (en tu peor caso, que es cuando el valor no está repetido).

EDIT: Este trozo de código no me gusta:

Código PHP:

//Código
$codigo = $scripts->codigo();
$verificarCodigo = $miConsulta->codigo($codigo);
if ($verificarCodigo->num_rows) {
    do {
        $miConexion->conexion->next_result();
        $codigo = $scripts->codigo();
        $verificarCodigo = $miConsulta->codigo($codigo);
    } while ($verificarCodigo = $miConsulta->codigo($codigo));
    $verificarCodigo->free();
}
$miConexion->conexion->next_result();
 

Lo he cambiado a:

Código PHP:

//Código
do {
    $codigo = $scripts->codigo();
    $verificarCodigo = $miConsulta->codigo($codigo);
    $rows=$verificarCodigo = $miConsulta->codigo($codigo);
    $verificarCodigo->free();
    $miConexion->conexion->next_result();
} while ($rows);
 

[Alexis88]

Eres grande, marlanga.

Imagino que el uso de next_result() se debe a que llamo a más de un PA en la misma instancia de conexión. Debo señalar que solamente hizo falta añadir dicha línea de código antes de invocar a cada un método, no después.

Saludos y gracias.

[marlanga]

Lo sé, y muchos me odian por ello.

Hay que llamar después. Si llamas antes, lo que haces es que la primera vez que se ejecuta, no hace nada. Por el mismo motivo, después del último CALL tampoco hace falta usarlo, porque si no hay más calls, no hace falta.

[Alexis88]

Claro, es que contesté antes de ver la segunda publicación que hiciste, es lo mismo que hice.

Una consulta más, me dices que a los campos de la tabla en donde no quiero valores repetidos, les coloque la restricción UNIQUE y ya cuando intente guardar los datos y se hayan ingresado datos que ya se encuentren en la BD, sabré qué error ocurrió por el número de error que se produzca. En esta inserción de datos, debo de comprobar que dos de ellos no se encuentren ya registrados, entonces, si se produce el error y solamente tengo el código de error, ¿cómo sabré cuál es el dato repetido?.

Gracias, nuevamente.

[marlanga]

También te devuelve un mensaje de error, del que puedes extraer el campo donde ocurrió el error. Y creo que en el caso de que varias restricciones fallen, sólo te devuelve la primera que ocurre.

[Alexis88]

Claro, sería cuestión de buscar en el mensaje de error, el nombre del campo.

Gracias, marlanga.