[SOLUCIONADO] Consulta mysqli me da error

[diegoguerrero]

Código PHP:

  <?php
$mysqli = new mysqli('localhost','root','','grupo');
 
if (mysqli_connect_errno()){
    printf("Falló la conexión: %s\n", mysqli_connect_error());
}
 
$consulta = "SELECT materia,curso FROM libros WHERE user=".$_GET['user'];
 
if ($sentencia = $mysqli->prepare($consulta)){
    /* Ejecutar la sentencia */
    $sentencia->execute();
    /* Vincular las variables de los resultados */
    $sentencia->bind_result($mysql_materia,$mysql_curso);
}
 
?>
<div id="h3-tec-bsm">
    Mis libros
</div>
<div id="tec-caja">
        <ul>
            <?php
                while ($sentencia->fetch()){
                    echo '
            <li type="none">
                '.$mysql_materia.' - '.$mysql_curso.'
            </li>
            ';
                }?>
        </ul>
    <a href="paginas/libros/mislibros.php">
        <p align="right">
            Ver más
            <i class="fa fa-arrow-right"></i>
        </p>
    </a>
</div>  

Me da error Call to a member function fetch() on boolean in /Applications/XAMPP/xamppfiles/htdocs/grupo/paginas/libros/MisLibros-resumen.php on line 18

Estoy probando con mysqli (siempre he sido fan de mysql) pero mi primer intento siempre me sale mal. ¿Cómo se puede solucionar?

[pateketrueke]

Probablemente el método prepare() no es correcto y devuelve FALSE, y pues, al usar $sentencia sin validar que efectivamente sean un objeto produce el error.

[diegoguerrero]

Iniciado por pateketrueke

Probablemente el método prepare() no es correcto y devuelve FALSE, y pues, al usar $sentencia sin validar que efectivamente sean un objeto produce el error.

¿Y cómo podría solucionar eso?

[pateketrueke]

Iniciado por diegoguerrero

¿Y cómo podría solucionar eso?

Pues, primero revisa el manual para que entiendes bajo qué circunstancias prepare() devuelve FALSE.

Segundo, siempre debes validar antes de usar cualquier cosa: siempre.

Lo ḿas simple sería comprobar con un if() al que menos, que $consulta no sea FALSE.

Código PHP:

if ($consulta) {
   // acá usas la variable
}
 

[gnzsoloyo]

Si el campo "user" es un VARCHAR, te debe estar disparando un error de ejecución por columna desconocida.
Estas tratando el valor recibido por GET como número, si es un string se dará un error, y OBVIAMENTE no estas capturando errores.
No asumas que algo funciona bien solo porque este aparentemente bien escrito.

[mortiprogramador]

<saludo>
Pues el tema de conexión a base de datos y muestra de datos
funciona perfecto.

El tema es más bien de preguntarse, que debe pasar sí
no viene el user por get?

Obviamente, sí no viene, la consulta quedará sin dato
probable a buscar, y por ende el fetch no tendrá data para
mostrar, y por consecuencia muestra el mensaje que ya viste.

Así que es solo cuestión de mejorar la lógica a la hora de armar
la consulta, pues sí el dato user viene, todo funcionará
(Acabo de probar copiando el código y funciona)

</saludo>

[diegoguerrero]

Iniciado por mortiprogramador

<saludo>
Pues el tema de conexión a base de datos y muestra de datos
funciona perfecto.

El tema es más bien de preguntarse, que debe pasar sí
no viene el user por get?

Obviamente, sí no viene, la consulta quedará sin dato
probable a buscar, y por ende el fetch no tendrá data para
mostrar, y por consecuencia muestra el mensaje que ya viste.

Así que es solo cuestión de mejorar la lógica a la hora de armar
la consulta, pues sí el dato user viene, todo funcionará
(Acabo de probar copiando el código y funciona)

</saludo>

Este archivo está insertado en otro PHP en el que sí está declarado.

[diegoguerrero]

He probado pasándolo a 'mysql' pero ahora sí que no me muestra nada. Además he declarado también aquí el $user y tampoco.

Código PHP:

<?php
$sql = mysql_connect('localhost','root',''); mysql_select_db('grupo',$sql);
echo mysql_error();
$user = $_GET['user'];
require ('conexion.php');
$sql_tarea = "SELECT * FROM MisLibros WHERE 'user=".$user."'";
$sql_execute = mysql_query($sql_tarea,$sql);
echo mysql_error();
?>
<div id="h3-tec-bsm"> 
    Mis libros 
</div> 
<div id="tec-caja"> 
        <ul>
                <?php
                    while($import = mysql_fetch_assoc($sql_execute)){
                        ?>
                            <li type="none">
                                <?php echo $import['materia'].' - '.$import['curso']; ?>
                            </li>
                        <?php
                    }
echo mysql_error();
                ?>
        </ul> 
    <a href="paginas/libros/mislibros.php"> 
        <p align="right"> 
            Mis libros 
            <i class="fa fa-arrow-right"></i> 
        </p> 
    </a> 
</div>
 

[gnzsoloyo]

Vuelvo al punto: Tienes que verificar si la sentencia queda bien escrita para MySQL. No te olvides que PHP no interpreta SQL, y MySQL tampoco lo hace con PHP.

Mira, el error básico es simplísimo, ´suponiendo que hayas ingresado tu user como "Antonio", tu SQL queda así:

Código MySQL:

SELECT * FROM MisLibros WHERE 'user=antonio'
 

¿Notas como todo lo del WHERE ha quedado resaltado como una cadena de texto?
Bueno, eso es porque lo UNICO que debe ir como cadena de texto es lo que estás recibiendo por GET:

Código MySQL:

SELECT * FROM MisLibros WHERE user='Antonio'
 

¿Notas la diferencia?

Ahora bien, otra cosa que te conviene hacer SIEMPRE, mientras estás aprendiendo a programar y todavía estás en etapa de desarrollo es visualizar lo que el SQL genera:

Código PHP:

  $sql_tarea = "SELECT * FROM MisLibros WHERE user='".$user."'";
echo $sql_tarea; 
   

de ese modo podrás asegurarte de qué es lo que estás enviando a la base, porque si el usuario no está llegando, obtendrías esto:

Código MySQL:

SELECT * FROM MisLibros WHERE user=''
 

Y eso no genera un error, pero tampoco devuelve datos.... cosa que es lo que no estás verificando en NINGUNA PARTE. Tu asumes que siempre obtienes datos, y eso no es necesariamente cierto.

[diegoguerrero]

Iniciado por gnzsoloyo

Vuelvo al punto: Tienes que verificar si la sentencia queda bien escrita para MySQL. No te olvides que PHP no interpreta SQL, y MySQL tampoco lo hace con PHP.

Mira, el error básico es simplísimo, ´suponiendo que hayas ingresado tu user como "Antonio", tu SQL queda así:

Código MySQL:

SELECT * FROM MisLibros WHERE 'user=antonio'
 

¿Notas como todo lo del WHERE ha quedado resaltado como una cadena de texto?
Bueno, eso es porque lo UNICO que debe ir como cadena de texto es lo que estás recibiendo por GET:

Código MySQL:

SELECT * FROM MisLibros WHERE user='Antonio'
 

¿Notas la diferencia?

Ahora bien, otra cosa que te conviene hacer SIEMPRE, mientras estás aprendiendo a programar y todavía estás en etapa de desarrollo es visualizar lo que el SQL genera:

Código PHP:

  $sql_tarea = "SELECT * FROM MisLibros WHERE user='".$user."'";
echo $sql_tarea; 
   

de ese modo podrás asegurarte de qué es lo que estás enviando a la base, porque si el usuario no está llegando, obtendrías esto:

Código MySQL:

SELECT * FROM MisLibros WHERE user=''
 

Y eso no genera un error, pero tampoco devuelve datos.... cosa que es lo que no estás verificando en NINGUNA PARTE. Tu asumes que siempre obtienes datos, y eso no es necesariamente cierto.

Estupenda respuesta. ¡Muchas gracias!

[mortiprogramador]

Iniciado por diegoguerrero

Este archivo está insertado en otro PHP en el que sí está declarado.

<saludo>
Pues debe funcionar sin problema entonces
</saludo>