PHP OO Conexión a BD no funciona

[guardarmicorreo]

Estoy trabajando sobre una opción de una web app que sirve para limitar el uso de la web si el usuario se ha conectado y realizado preguntas en la última hora.

He creado un método llamado compareDatesHours dentro de la clase UsersDateTime_SetGetDrop();

Llegado a un punto del método la conexión a la BD da error

Warning: mysqli_query() expects parameter 1 to be mysqli, null given in /var/www/vhosts/miweb.com/httpdocs/models/Connect.php on line 38 Warning: mysqli_error() expects parameter 1 to be mysqli, null given in /var/www/vhosts/miweb.com/httpdocs/models/Connect.php on line 39

Esta es la parte del método donde me da error, lo indico en el código

Código PHP:

<?php
        //recupera la ultima conexion del usuario al servidor
        //para compararla con la fecha y hora local actual
        
        $this->sql              =   "SELECT date_time,
                                        TIMESTAMPDIFF(YEAR,     date_time, '".$this->local_date_time."') as diff_year,
                                        TIMESTAMPDIFF(MONTH,    date_time, '".$this->local_date_time."') as diff_month,
                                        TIMESTAMPDIFF(DAY,      date_time, '".$this->local_date_time."') as diff_day,
                                        TIMESTAMPDIFF(HOUR,     date_time, '".$this->local_date_time."') as diff_hour,
                                        TIMESTAMPDIFF(MINUTE,   date_time, '".$this->local_date_time."') as diff_minute,
                                        TIMESTAMPDIFF(SECOND,   date_time, '".$this->local_date_time."') as diff_second 
                                    FROM activity_date_time
                                    WHERE id_twitter=$id_twitter ORDER BY id_date_time DESC LIMIT 1";
    //hasta aquí el código no da problemas, toda la consulta está bien  
        $this->result_conexion  =   $this->connect($this->sql);
        //aquí da error
        $this->result_conexion  =   mysqli_fetch_array($this->result_conexion);
?>
 

Connect.php contiene lo siguiente, indico la línea donde me dice que está el error

Código PHP:

<?php
/*
CLASE PARA CONECTAR Y REALIZAR OPERACIONES EN LA BASE DE DATOS
*/
class Connect
{
    private $IDConexion;
    private $Escape;
    private $IDQuery;
    
    public function __construct()
    {
        //establezco los datos de autenticación para acceder a la base de datos
        $conexion = mysqli_connect("localhost", "usuario" , "password" , "BD_name");
        
        //establezco el conjunto de caracteres a nivel de servidor en tiempo de ejecucion
        mysqli_set_charset( $conexion, "utf8" );
        
        $this->IDQuery = $conexion;
        
        return $this->IDQuery;
    }
    
    public function escape( $string_escapar )
    {
        //escapo caracteres para evitar sql injection
        $string_escapar = trim( $string_escapar );
        $escape = mysqli_real_escape_string( $this->IDQuery, $string_escapar );
        
         $this->Escape = $escape;
        
        return $this->Escape;
    }
    
    public function connect ( $string_consulta )
    {
        //conecto a la base de datos
        $this->IDConsulta = mysqli_query($this->IDQuery , $string_consulta) //linea 38 del código
        OR die (mysqli_error($this->IDQuery)); //linea 39 del código
        
        return $this->IDConsulta;
    }
    public function go_out ()
    {
        mysqli_close($this->IDQuery);
    }
}
?>
 

No entiendo qué está pasando, la clase Connect funciona con todo lo demás excepto aquí, lo sé porque previamente he creado otro código que comprueba si el usuario existe para no registrar sus datos por duplicado y funciona perfectamente :S hace la comprobación.

¿Alguien puede ayudarme con este problema?

He revisado los datos de conexión y están bien. Si es necesario mostrar más código lo muestro, no hay problema por eso.

[arcanisgk1]

es necesario enviar por pantalla lo que te sale de error

[guardarmicorreo]

Iniciado por arcanisgk1

es necesario enviar por pantalla lo que te sale de error

Gracias por contestar. Los errores los puse al principio del post.

[arcanisgk1]

orientado a objeto:

Código PHP:

$this->conexion = new mysqli($servidor, $usuario, $clave, $basedatos);
        if ($this->conexion->connect_error)
            exit ("Problema de la Conexion a la Base de Datos: " . $this->conexion->connect_error);
        $this->conexion->query("SET NAMES 'utf8'");
    return $this->conexion;
 

[guardarmicorreo]

no entiendo lo que me quieres decir :S sé que eso es para trabajar con mysqli y que el if significa que en caso de que la conexión devuelva error muestre el error, pero no entiendo lo que quieres que haga con eso

[arcanisgk1]

compara y estudia tu código... el error debe estar hay...

mira el problema esta en que cuando estas corriendo el query no esta agarrando la conexión a la base de datos... por que no esta recibiendo lo parámetros de conexión.

[guardarmicorreo]

Iniciado por arcanisgk1

compara y estudia tu código... el error debe estar hay...

mira el problema esta en que cuando estas corriendo el query no esta agarrando la conexión a la base de datos... por que no esta recibiendo lo parámetros de conexión.

En un principio así pensé yo también, pero otras conexiones a la BD a través de esa misma clase funcionan perfectamente.

Acabo de volver a utilizar esa clase y funciona todo fenomenal, por eso puse el post, no sé dónde puede estar el error :S

[guardarmicorreo]

¿Tiene algo que ver que en el método compareDatesHours establezca la zona horaria con date_default_timezone_set('Europe/Madrid') y en la clase connect defina mysqli_set_charset($conexion, 'utf8')?

[arcanisgk1]

entonces algo esta mal puede ser que la variable que envias con el string este vacia prueba imprimir la variable por pantalla antes de usarla en la coneccion a ver que te sale.

[arcanisgk1]

te mandaria otro tipo de error, el error que te esta mandando es de datos faltantes

[guardarmicorreo]

Sí, eso también lo comprobé antes y el string está bien, esta es la salida que me acaba de dar ahora mismo
(quito el id_twitter por cuestiones de seguridad, pero lo he comprobado y también está bien) ¿pongo más código?

Código SQL:

SELECT date_time,
                                        TIMESTAMPDIFF(YEAR,     date_time, '2014-02-20 23:50:45') AS diff_year,
                                        TIMESTAMPDIFF(MONTH,    date_time, '2014-02-20 23:50:45') AS diff_month,
                                        TIMESTAMPDIFF(DAY,      date_time, '2014-02-20 23:50:45') AS diff_day,
                                        TIMESTAMPDIFF(HOUR,     date_time, '2014-02-20 23:50:45') AS diff_hour,
                                        TIMESTAMPDIFF(MINUTE,   date_time, '2014-02-20 23:50:45') AS diff_minute,
                                        TIMESTAMPDIFF(SECOND,   date_time, '2014-02-20 23:50:45') AS diff_second 
                                    FROM activity_date_time
                                    WHERE id_twitter=id_twitter ORDER BY id_date_time DESC LIMIT 1
 

Soy incapaz de encontrar el problema, he filtrado todos los return hasta que he encontrado dónde era, pero ahora falta saber por qué y no encuentro nada que de motivo para que falle la consulta :S

Y aún estando el string vacío diría que lo que falla es el argumento 2 por ser null, no el argumento 1 que se ejecuta siempre que se inicia una nueva conexión por estar en el __construct()

[Triby]

Si para cada consulta vas a establecer una conexión a base de datos, me parece lógico que llegará el momento en que no te queden conexiones disponibles.

Para saberlo, adapta tu código para funcionar como el ejemplo del manual de PHP:

Código PHP:

$enlace = mysqli_connect('localhost', 'mi_usuario', 'mi_contraseña', 'mi_bd');
 
if (!$enlace) {
    die('Error de Conexión (' . mysqli_connect_errno() . ') '
            . mysqli_connect_error());
}
 

[guardarmicorreo]

Hola Triby, muchas gracias por tu respuesta! :D

He revisado las conexiones a la BD, las conexiones que hay en este proceso son las siguientes:

(callback.php es el documento php al que twitter redirige y que va a recoger las credenciales de Twitter para autentificar al usuario por GET)

1) Desde el callback.php se instancia a la clase Users_SetGetDrop(), esta clase a su vez ya instancia a Connect.php y a UsersDatetime_SetGetDrop() que también instancia a Connect.php.

2) Obtengo los datos del usuario según su id_twitter (para saber si el usuario existe o no).
-Del callback.php utilizo la instancia anterior para utilizar el método getUser() que extrae los datos de un usuario específico.

3) Si el usuario no existe se guardan sus datos en la BD.
-Para ello reutilizo la instancia anterior y solo utilizo otro método (setUser()) de la clase Users_SetGetDrop().

4) Si el usuario existe, entonces utilizo otro método (compareDatas()) de la clase anterior para comparar los datos de la BD con los de twitter.

5) Compruebo si el usuario no está bloqueado por el admin o por límite de tiempo del sistema para hacer uso de la web app. Para realizar esto utilizo otro método de la clase anterior (checkUserState($id_twitter) que es la vía por donde me devuelve el error) que a su vez utiliza dos métodos:

-El método compareDatesHours() de la clase UsersDatetime_SetGetDrop() que sirve para saber si el usuario ha esperado el tiempo suficiente para volver a utilizar la web app.

-El método getUser() de la primera clase (Users_SetGetDrop()).

¿He realizado demasiadas conexiones a la BD?

En el proceso he contado dos instancias a Connect.php = 2 conexiones a la BD.

Si son demasiadas, cómo podría optimizar para poder dejarlo en una sola conexión.

Solo me queda pasar todo el código que interviene en este proceso, no me importaría si con ello aprendo a solucionar este y futuros problemas.

Un saludo! :D

[Triby]

Trabaja todos tus modelos con una misma conexión instancia de acceso a base de datos que puedes pasar por parámetro, o bien, aunque algunos dicen que es mala práctica, también podrías optar por singleton.

[xSkArx]

Por que no pones pedazo de codigo donde si te funciona la conexion a la bd, lo otro es que no se para que poner un return en el constructor