1

Realicé una función que le da formato a un rango de fechas para que puedan ser utilizadas para una consulta en MySQL.

Al imprimir $a y $b las tengo así, eso me llevó a pensar que todo está correcto, pero no. MySQL no me hace la consulta:

a)       2021-01-21 06:00:00
b)       2021-01-22 06:00:00

El código PHP es:

function FAC0014($a, $b){

    
    echo $a; echo $b; 
    $query = $this->connect()->prepare('SELECT Id_user, Id_state FROM binplaza_colas.quotes WHERE Time_registry BETWEEN :in AND :to'); 
    $query->execute([
        'in'           => $a,
        'to'           => $b
        ]);
    return $query;
}

Debo mencionar que el tipo de dato de Time_registry es datatime y que realizando la consulta en el MDB sí me lanza los resultados.

editado

Ejecutando:

$query->debugDumpParams()

Obtuve esto:

2021-01-22 06:00:002021-01-23 06:00:00SQL: [91] SELECT Id_user, Id_state FROM binplaza_colas.quotes WHERE Time_registry BETWEEN :in AND :to
Sent SQL: [127] SELECT Id_user, Id_state FROM binplaza_colas.quotes WHERE Time_registry BETWEEN '2021-01-22 06:00:00' AND '2021-01-23 06:00:00'
Params:  2
Key: Name: [3] :in
paramno=-1
name=[3] ":in"
is_param=1
param_type=2
Key: Name: [3] :to
paramno=-1
name=[3] ":to"
is_param=1
param_type=2

Y la función que itera esto dice que es nulo:

[22-Jan-2021 11:34:05 Europe/Madrid] PHP Fatal error: Uncaught Error: Call to a member function rowCount() on null in ....

Y la función:

function FAF0011($a){
    $mys = new Mys();
//  error_log(print_r($a.'<br>'.$b,true)); //depurrador php
    $a = date ('Y-m-d 06:00:00' , strtotime($a));
    $b = date ('Y-m-d H:i:s' , strtotime('+24 hour', strtotime($a)));
    $res = $mys->FAC0014($a, $b);
    if ($res->rowCount() >= 1) {
        $row = $res->fetch();
        $items = array(
            'a_a'       => $row['Id_user'],
            'a_b'       => $row['Id_state']
        );
    } else {
    /* En caso de ir mal mostramos el mensaje de error */
    die(
        htmlspecialchars(
            implode(", ", $res->errorInfo())
        )
    );

    $this->FAF0002($items,1);
}

Reeditado

Ésta es una prueba de que me falta mucho por aprender. Con suerte el error te ayuda y ésta fue la solución:

  while ($row = $res->fetch()) {
        /* Agregamos un nuevo elemento */
        array_push($items, array(
            'a_a'       => $row['Id_user'],
            'a_b'       => $row['Id_state']
        ));
    }

Obvié el método correcto de cómo usar los resultados de MySQL:

   ->rowCount()

Se estaba usando mal. Gracias a @OscarGarcia por la solución.

8
  • en tu return has esto return return $query->debugDumpParams(); y comentanos que resultado obtienes
    – user128299
    Commented el 22 ene. 2021 a las 5:30
  • PHP Fatal error: Uncaught Error: Call to a member function rowCount() on null in .., tengo un contador que verifica si hay filas, y muestra ese error
    – XavRuss
    Commented el 22 ene. 2021 a las 5:38
  • No, antes del contador, prueba directamente la pura consulta que expones para que podamos ver si se le estan o no pasando los valores de las variables
    – user128299
    Commented el 22 ene. 2021 a las 5:47
  • sigo sorprendido colgue la respuesta editando la pregunta, probe una vez mas la consulta directamente en el gestor y si funciona.
    – XavRuss
    Commented el 22 ene. 2021 a las 5:54
  • Pues el debug nos indica que las cosas marchan bien, los valores si llegan, ¿seguro que si tienes programado el recorrido e impresión de los valores?
    – user128299
    Commented el 22 ene. 2021 a las 6:01

1 respuesta 1

1

Te propongo dos cambios para evitar problemas en tu código:

Es importante poner dos puntos delante del nombre de cada marcador. Además, al no comprobar el resultado de $query->execute() estás pasando por alto un fallo en la consulta SQL.

Te recomiendo hacerlo de la siguiente manera:

function FAC0014($a, $b){
    // echo $a; echo $b;
    $conexion = $this->connect();
    $query = $conexion->prepare('
        SELECT Id_user, Id_state
        FROM binplaza_colas.quotes
        WHERE Time_registry BETWEEN :in AND :to
    '); 
    /* Si la preparación no es emulada, comprobamos si fue bien */
    if ($query === false) {
        /* En caso de ir mal mostramos el mensaje de error */
        die(
            htmlspecialchars(
                implode(", ", $conexion->errorInfo())
            )
        );
    }
    $resultado = $query->execute([
        ':in' => $a,
        ':to' => $b,
    ]);
    /* Comprobamos si la consulta SQL fue bien */
    if ($resultado === false) {
        /* En caso de ir mal mostramos el mensaje de error */
        die(
            htmlspecialchars(
                implode(", ", $query->errorInfo())
            )
        );
    }
    return $query;
}

Además, estás usando PDOStatement::rowCount() de manera incorrecta:

PDOStatement::rowCount() devuelve el número de filas afectadas por la última sentencia DELETE, INSERT, o UPDATE ejecutada por el correspondiente objeto PDOStatement.

Si la última sentencia SQL ejecutada por el objeto PDOStatement asociado fue una sentencia SELECT, algunas bases de datos podrían devolver el número de filas devuelto por dicha sentencia. Sin embargo, este comportamiento no está garantizado para todas las bases de datos y no debería confiarse en él para aplicaciones portables.

Además, estás obteniendo únicamente un registro. Para obtenerlos todos debes realizar un bucle que los almacene todos en la variable deseada ($item).

Aquí está el código que te propongo:

function FAF0011($a) {
    $mys = new Mys();
    $a = date ('Y-m-d 06:00:00' , strtotime($a));
    $b = date ('Y-m-d H:i:s' , strtotime('+24 hour', strtotime($a)));
    $res = $mys->FAC0014($a, $b);
    /* Agregamos los registros obtenidos a $items */
    $items = array();
    while ($row = $res->fetch()) {
        /* Agregamos un nuevo elemento */
        array_push($items, array(
            'a_a'       => $row['Id_user'],
            'a_b'       => $row['Id_state']
        ));
    }
    $this->FAF0002($items,1);
}

Edición: Pruebas realizadas en:

9
  • Bueno, puse lo dos puntos pero sigue emitiendo error, además que todas mis consultas no las traen y funcionan perfectamente esta me ha salido como la oveja negra.
    – XavRuss
    Commented el 22 ene. 2021 a las 10:59
  • Puede ser un error en el prepare o en el execute. ¿Has probado un var_export($query) justo tras el connect()->prepare()? Commented el 22 ene. 2021 a las 10:59
  • var_export($query); $query = $this->connect()->prepare('SELECT Id_user, Id_state FROM binplaza_colas.quotes WHERE Time_registry BETWEEN :in AND :to'); lo acabo de intentar y esta igual
    – XavRuss
    Commented el 22 ene. 2021 a las 11:06
  • He modificado mi respuesta para gestionar una mala preparación de la consulta SQL. He tenido que separar la conexión en una variable para poder extraer de ella el mensaje de error. Commented el 22 ene. 2021 a las 11:06
  • En cuanto a tu último comentario, no, me refería al var_export después de la preparación. Todo apunta a que no se está realizando correctamente. Además, tratas de hacer rowCount() sobre una consulta que no ha actualizado registros, por lo que puede que no te funcione como esperas (ojo, algunas veces sí, otras veces no, depende del SGBD y el controlador MySQL). Commented el 22 ene. 2021 a las 11:06

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.