Estás cometiendo el error de intentar acceder a la propiedad de un objeto sin verificar el estado de dicho objeto.
Dado que fetch()
devuelve false
si no encuentra datos o si hubo algún error, puedes hacer la verificación de este modo:
if($cancion) {
// Hay fiesta
} else {
// No hay fiesta, lo siento
}
Más cosas:
Podrías retornar directamente el resultado de fetch()
, no es necesario en ese contexto reservar espacio de memoria para una variable.
Deberías evitar la forma abreviada de PHP. El Manual dice claramente que esa forma no es recomendable, primero, porque para que funcione debes modificar el archivo php.ini
y segundo, por problemas de compatibilidad:
PHP también permite la etiqueta de apertura abreviada <?
(la cual
está desaconsejada debido a que sólo está disponible si se habilita
con la directiva short_open_tag
del fichero de configuración
php.ini
, o si PHP se configuró con la opción --enable-short-tags
).
Aplicando las mejoras sugeridas, tu código puede quedar así, nótese que el uso de printf()
o sprintf()
según el caso, son una alternativa mucho más limpia que usar etiquetas de apertura abreviada.
Método
function getCancion($id)
{
$query = $this->db->prepare('SELECT a.*, b.* FROM canciones a INNER JOIN artistas b ON a.nombre_artista = b.nombre_artista WHERE id_cancion = ?');
$query->execute([$id]);
return $query->fetch(PDO::FETCH_OBJ);
}
Donde consumes la respuesta
<?php
//Lo que sea de código
$id=1;
$cancion=getCancion($id);
if($cancion)
{
printf('<h1>%s</h1>',$cancion->nombre_cancion);
}
else
{
echo "No hay canción";
}
O si quieres, para algo sencillo como esto, en lugar de printf
, puedes simplemente usar comillas dobles y meter dentro la llamada al objeto y su propiedad, algo así:
echo "<h1>{$cancion->nombre_cancion}</h1>";
Cualquiera de las dos es más limpia y más compatible que usar la forma abreviada.
false
. Verifica si la consulta está correcta o si no está trayendo registros porque elid
que no existe. En ese caso tendrás que manejar la excepción [no hay canciones con ese id]. Por otro lado elJOIN
me parece extraño, porque ¿tienes el nombre del artista en ambas tablas? Debería serid_artista
el cual es un dato numérico entero.