0

estoy usando la funcion global $con; y el error se fue pero no se si este bien que lo use en cada funcion y si en PHP5 funcione igual de bien que en PHP7

Error:

Warning: mysqli_query(): Couldn't fetch mysqli in C:\xampp\htdocs\new\trending.php on line 19

Warning: mysqli_num_rows() expects parameter 1 to be mysqli_result, null given in C:\xampp\htdocs\new\trending.php on line 21

codigo:

$sql = mysqli_query($con,"Select * From trending, ORDER BY top DESC LIMIT 13");

if (mysqli_num_rows($sql) == 0)
    {
        for ($n=1;$n<=12;$n++){
            echo'<div class="div_css_one">';
            echo'#</div>';
            echo'<div class="div_css_one_b">';
            echo'</div>';
        }
    }else{  

        while($data = mysqli_fetch_array($sql, MYSQLI_ASSOC)) {
            echo '<li><a href="hashtag?hashtag='.$data['hashtag'].'"><p class="font-weight">#'.$data['hashtag'].'</p></a><spam class="trending_post_top">'.$data['top'].' posts</spam><li>';
        }
    }
7
  • @abrahamhs ESE ES MI ERROR
    – Shareiv
    el 13 sep. 2017 a las 19:45
  • 1
    Error de sintaxis aquí: Select * From trending, ORDER BY top DESC LIMIT 13
    – A. Cedano
    el 13 sep. 2017 a las 19:50
  • 1
    Siempre es bueno en el código controlar todo lo que podría ser nulo o no funcionar. ¿Estás comprobando el estado de la conexión? Por ejemplo: if (!$con) { echo "Error: No se pudo conectar a MySQL." . PHP_EOL; echo "errno de depuración: " . mysqli_connect_errno() . PHP_EOL; echo "error de depuración: " . mysqli_connect_error() . PHP_EOL; }else{$sql = mysqli_query($con,"Select * From trending ORDER BY top DESC LIMIT 13");//Código de consulta y demás}
    – A. Cedano
    el 13 sep. 2017 a las 19:57
  • la conexion esta bien porque tiene un if 500 por si la conexion esta mal me avisa es otra cosa pero no se a cual
    – Shareiv
    el 13 sep. 2017 a las 20:01
  • 1
    Ese error indica casi siempre problemas con la conexión. ¿Puedes poner en la pregunta el código que hay antes? ¿No habrás cerrado $con o se llama de otra manera o algo así? ¿Esa consulta te trae datos si la pruebas en el manejador de base de datos directamente?
    – A. Cedano
    el 13 sep. 2017 a las 20:06

2 respuestas 2

1

Es necesario un control total del código para que puedas saber dónde está fallando:

//Controlar conexión
if (!$con) 
{ 
    echo "Error: No se pudo conectar a MySQL.";

}else{

    //Controlar consulta
    if (mysqli_query($con,"Select * From trending ORDER BY top DESC LIMIT 13")=== TRUE) 
    {

        if (mysqli_num_rows($sql) == 0)
        {
           for ($n=1;$n<=12;$n++)
           {
              echo'<div class="div_css_one">';
              echo'#</div>';
              echo'<div class="div_css_one_b">';
              echo'</div>';
            }
          }else{  

          while($data = mysqli_fetch_array($sql, MYSQLI_ASSOC)) 
          {
             echo '<li><a href="hashtag?hashtag='.$data['hashtag'].'"><p class="font-weight">#'.$data['hashtag'].'</p></a><spam class="trending_post_top">'.$data['top'].' posts</spam><li>';
           }
           //Cerrar recursos usados si fuere necesario
    }else{
           echo "Consulta errónea, revise la instrucción SQL: ".$sql
    }
}
7
  • i.imgur.com/RQLY6eg.png hablamos por un chat
    – Shareiv
    el 13 sep. 2017 a las 20:48
  • @Shareiv el mensaje de la imagen indica que hay un error en la consulta. O la tabla trending no existe, o no tiene una columna llamada top.
    – A. Cedano
    el 14 sep. 2017 a las 3:29
  • estoy usando la funcion ´global $con;´ y el error se fue pero no se si este bien que lo use en cada funcion y si en PHP5 funcione igual de bien que en PHP7
    – Shareiv
    el 14 sep. 2017 a las 15:10
  • 1
    El uso de globales no es una práctica recomendada, precisamente por lo que te está ocurriendo entre otras cosas: es difícil controlar y depurar cuando ocurren errores y fácilmente pierdes horas/días tratando de encontrar dónde está el problema. Quizá la conexión fue destruida en alguna parte del código y cuando la intentas usar tienes el error. Sugiero que empieces a usar el estilo orientado a objetos, que crees una clase que maneje la conexión y cuando la vayas a usar crees tu objeto conexión con new. Así tendrás control de tus objetos siempre y sabrás lo que pasa en cada instante.
    – A. Cedano
    el 14 sep. 2017 a las 15:35
1

Tu query esta incorrecto, tiene una , de más, al finalizar la lista de campos. Esto provoca que la función regrese un FALSE. Ese FALSE es lo que le pasas a la función mysqli_num_rows, que es lo que te genera el segundo warning.

Puedes ver más detalles de mysqli_query Y mysqli_num_rows

Tu Respuesta

Al pulsar en “Publica tu respuesta”, muestras tu consentimiento a nuestros términos de servicio, política de privacidad y política de cookies

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