0

El script que tengo aparentemente funciona, tengo el siguiente código en PHP:

$sql = "SELECT gcm_regid  FROM gcm_users WHERE papa1 = '$papa1'  AND  cel1 = '$cel1'"; 
echo $sql; 
$result = $conn->query($sql); 
if ($result->num_rows > 0) { 
    while($row = $result->fetch_assoc()) { 
        include_once '../gcm/GCM.php'; 
        $regId = $row["gcm_regid"]; 
        $message = $message; 
        echo $regId; 
        echo $message; 
        echo "entre"; 
        $gcm = new GCM(); 
        $registatoin_ids = array($regId); 
        $message = array("price" => $message); 
        $result = $gcm->send_notification($registatoin_ids, $message); 
        echo $result; 
    } 
} else { 
    echo "0 results";
}
$conn->close();

Pero me muestra el siguiente error:

Fatal error: Call to a member function fetch_assoc() on a non-object in /public_html/gcm/run.php on line 46r

¿Cómo podría solucionarlo?

5
  • 1
    ¿Cuál linea manda el fatal error? ¿Qué mensajes se muestran en el log de errores o en el navegador cuando se ejecuta este script?
    – user227
    Commented el 12 feb. 2016 a las 14:02
  • Justo en donde inicia el ciclo while, habla de un objeto, podría ser por $gcm = new GCM(); Commented el 12 feb. 2016 a las 14:03
  • 2
    $papa1 y $cel1 al parecer son variables ¿Qué valores tienes ahí? Commented el 12 feb. 2016 a las 14:07
  • 1
    ¿Puedes por favor editar tu pregunta y colocar el stacktrace que te arroja PHP? De esa manera será más facil entender y revisar tu problema.
    – user227
    Commented el 12 feb. 2016 a las 14:08
  • colocar esto como pregunta ? Fatal error: Call to a member function fetch_assoc() on a non-object in /public_html/gcm/run.php on line 46 Commented el 12 feb. 2016 a las 14:18

1 respuesta 1

2

El error "Call to a member function <función> on a non-object in" sucede porque la variable, en este caso $result, no es un objeto y por tanto no se puede llamar el método o función esperado. Esto se explica mejor acá (idioma inglés).

El problema está en esta línea dentro del while:

$result = $gcm->send_notification($registatoin_ids, $message);

Estás sobreescribiendo la variable $result. Es muy probable que el nuevo valor de $result no sea el de ejecutar $conn->query($sql), por ende el error. Te recomiendo que utilices otro nombre para esta variable local en el ciclo while.


Esto no es el problema real, pero mayormente es la causa. Lo dejo indicado por si alguien se topa con este problema por causa del query.

Cuando mysqli_query posee un error al procesar la sentencia SQL que posee, devuelve falso (énfasis mía):

Valores devueltos

Retorna FALSE en caso de error. Si una consulta del tipo SELECT, SHOW, DESCRIBE o EXPLAIN es exitosa, mysqli_query() retornará un objeto mysqli_result. Para otras consultas exitosas de mysqli_query() retornará TRUE.

Puede que el query retorne false. Es muy probable que el problema esté acá:

$sql = "SELECT gcm_regid  FROM gcm_users WHERE papa1 = '$papa1'  AND  cel1 = '$cel1'";

Esto puede suceder porque concatenas directamente los valores de $papa1 y $cel1 en tu query, y pueden poseer caracteres que rompan el query. Acá un ejemplo:

$papa1 = "papa";
$cel1 = "cel'"; //nota la comilla al final
$sql = "SELECT gcm_regid  FROM gcm_users WHERE papa1 = '$papa1'  AND  cel1 = '$cel1'";
echo $sql;

Que va a imprimir:

SELECT gcm_regid  FROM gcm_users WHERE papa1 = 'papa' AND cel1 = 'cel''

Y voalá, query destruido :D

Para evitar sanear los valores de estas variables en este y otros queries, te recomiendo ejecutar el query de manera preparada, tal como expliqué en esta otra respuesta.

7
  • 1
    Pienso lo mismo, ¿pero entonces no debería fallar en la fila anterior: if ($result->num_rows > 0) {?
    – Alvaro Montoro
    Commented el 12 feb. 2016 a las 14:30
  • 1
    @AlvaroMontoro inspeccionando el código, encontré el error. Mira la edición. A fin de cuentas, son cosas que pasan.
    – user227
    Commented el 12 feb. 2016 a las 14:33
  • 1
    No me había dado cuenta de eso. Entonces realmente falla en la segunda iteración y no en la primera. Buen ojo. Te daría otro +1 si el que tuvieras ahora mismo no fuera mío :)
    – Alvaro Montoro
    Commented el 12 feb. 2016 a las 14:42
  • 1
    @AlvaroMontoro gracias a ti. Si no me comentabas lo del num_rows no hubiese leido todo el código a más detalle.
    – user227
    Commented el 12 feb. 2016 a las 14:44
  • 1
    De nada @JESUSESPINOSA y la verdad que te seguiré insistiendo en queu utilices mysqli_prepare y sentencias preparadas :P pero fuera de bromas es una buena práctica usar sentencias preparadas, con eso tus sentencias SQL son más seguras.
    – user227
    Commented el 12 feb. 2016 a las 14:45

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.