Veamos los errores que tienes:
- Nunca llamas a
resolve
o a reject
dentro de tu Promesa.
- Intentas usar el valor de tu Promesa inmediatamente sin usar
await
.
PROBLEMA
Se desea enviar el resultado de una consulta a una BD a la vista, usando Express y NodeJS.
SOLUCIÓN
Vas encaminado en la dirección correcta: usar Promesas para envolver una consulta, pero la ejecución falla.
Tienes una función llamada respuesta
, y la misma devuelve una Promesa.
Para que la Promesa devuelva un resultado (o un error) debes ejecutar los métodos resolve
o reject
de acuerdo al resultado de la consulta.
Podrías hacerlo así:
function respuesta() {
return new Promise((resolve, reject) => {
con.query("SELECT texto FROM respuestas", function (err, result, fields) {
// si ocurre error en la consulta, rechazamos la Promesa
if (err) {
console.error(error.message); // <= mostramos el mensaje en consola
return reject(error); // devolvemos una Promesa rechazada pasando como argumento el error recibido
}
// Si la consulta devuelve resultados, resuelvo la promesa con los mismos
console.log(result);
return resolve(result);
// Si necesito los campos (fields) puedo devolver un objeto, dependerá de lo que necesites hacer
// return resolve({result, fields});
});
});
}
Ahora, para usar este método tienes 2 opciones:
- Usar
then()
y catch()
y escribir tu lógica usando este estilo.
- Usar tu mñetodo dentro de una función tipo
async
para esperar el resultado de la Promesa usando await
:
Usando then()
y catch
:
// En NodeJS, evita usar 'var', es preferible 'let' y 'const'
let respuesta = bbdd.respuesta();
respuesta
.then(result => {
// aqui manejas el resultado de la consulta
return res.status(200).json({
text: result, // <= yo escogería otro nombre en vez de text.
type: 'message'
});
})
.catch(error => {
// aqui manejas si hubo error en la consulta
return res.status(500).json({ error:<mensaje de error> }
});
Usando una función async
:
Suponiendo que este método lo llamas dentro de un middleware de Express, puedes declararlo como async
:
async (req, res, nex) => {
//...
// uso await para esperar el resultado de la promesa, y debo llamar a la misma
// dentro de un bloque try catch
try {
let respuesta = await bbdd.respuesta();
return res.status(200).json({ ... });
} catch(e) {
//manejo de error
console.log(e);
return res.status(500).json({ ... });
}
}
Esa sería la lógica a implementar en este caso.
resolve
oreject
. Adicionalmente, para usar el valor de tu función como intentas hacerlo, deberías usarawait
dentro de una funciónasync
, o usar los métodosthen()
ycatch()
de las Promesas. Saludos