Vamos por partes.
Primero, evitar usar return
a diestra y siniestra.
Si el método getByEmail
es asíncrono y provee then()
y catch()
, ¿porqué usar return
seguido de then()
(y de catch
)?
En todo caso, si vas a devolver getByEmail()
, usarías then()
y catch()
sobre registerUSer()
, y toda tu lógica la escribirías a partir de allí.
Supongamos que tengo una función que devuelve una Promesa, por lo tanto, implementa then()
y catch()
, y la misma me devolverá un resultado o producirá un error. Esta sería nuestra función getByEmail()
. La usaremos para determinar si el usuario existe. (No abordaré la lógica de la BD, sólo ilustraré lo que ocurre).
const emails = ['[email protected]','[email protected]']
const getByEmail = (email) => {
return new Promise((resolve, reject) => {
if(!email) {
error = new Error('No se especificó un correo');
return reject(error);
}
let existe = false;
if(emails.includes(email)) {
existe = true;
}
return resolve(existe);
});
}
En la función anterior, dado un email
determinamos si existe en nuestra lista de correos. La función devolverá: true
si el correo ya existe, false
si el correo no existe, y arrojará un error si no se le indica un correo para buscar. Como puedes observar, en la función sólo uso una sentencia return
, y la misma la uso para indicar que la promesa sea rechazada con el objeto de error y de esta forma el código que viene después no se ejecuta (no hace falta ejecutarlo). Si no hay errores, puedo ejecutar la comprobación, y devuelvo el resultado con el método resolve()
.
Veamos esto en acción:
const emails = ['[email protected]','[email protected]']
const getByEmail = (email) => {
return new Promise((resolve, reject) => {
if(!email) {
error = new Error('No se especificó un correo');
return reject(error);
}
let existe = false;
if(emails.includes(email)) {
existe = true;
}
return resolve(existe);
});
}
// la siguiente llamada mostrará que no existe el email
getByEmail('[email protected]')
.then(result => {
console.log(`Existe el email? ${result}`);
})
.catch(error => console.error);
//la siguiente llamada mostrará que sí existe el email
getByEmail('[email protected]')
.then(result => {
console.log(`Existe el email? ${result}`);
})
.catch(error => console.error);
//la siguiente llamada arrojará un error
getByEmail()
.then(result => {
console.log(`Existe el email? ${result}`);
})
.catch(console.error);
Ahora, esta función es parte de la lógica del método registerUser()
, y aquí es donde se pone bonita la cosa.
const emails = ['[email protected]','[email protected]'];
const getByEmail = (email) => {
return new Promise((resolve, reject) => {
if(!email) {
error = new Error('No se especificó un correo');
return reject(error);
}
let existe = false;
if(emails.includes(email)) {
existe = true;
}
return resolve(existe);
});
}
const registerUser = ({ name, email }) => {
return getByEmail(email)
.then(result => {
console.log(`Existe el email? ${result}`);
})
.catch(error => console.error);
}
const middleware = async (req, res) => {
try {
const resultado = await registerUser(req.user);
console.log(resultado);
} catch (e) {
console.error(e);
}
}
// si bien mostrará que el usuario no existe, el resultado devuelto por await no es ni true ni false.
middleware({user: {name: 'Sansa', email: '[email protected]'}});
Podemos ver que no se devuelve nada a la función middleware
, a menos que usemos específicamente return
dentro del método then()
en nuestra función getByEmail
.
Si quitamos el return
que hay delante del llamado a getByEmail
el resultado será el mismo:
const emails = ['[email protected]','[email protected]'];
const getByEmail = (email) => {
return new Promise((resolve, reject) => {
if(!email) {
error = new Error('No se especificó un correo');
return reject(error);
}
let existe = false;
if(emails.includes(email)) {
existe = true;
}
return resolve(existe);
});
}
const registerUser = ({ name, email }) => {
//eliminamos el return aquí y el resultado es el mismo:
getByEmail(email)
.then(result => {
console.log(`Existe el email? ${result}`);
})
.catch(error => console.error);
}
const middleware = async (req, res) => {
try {
const resultado = await registerUser(req.user);
console.log(resultado);
} catch (e) {
console.error(e);
}
}
middleware({user: {name: 'Sansa', email: '[email protected]'}});
Incluso hagamos otra cosa divertida, en vez de llamar al método then()
de nuestra promesa, devolvamos sólo la ejecución de la función, y el resultado será el esperado: (result contendrá un booleano).
const emails = ['[email protected]','[email protected]'];
const getByEmail = (email) => {
return new Promise((resolve, reject) => {
if(!email) {
error = new Error('No se especificó un correo');
return reject(error);
}
let existe = false;
if(emails.includes(email)) {
existe = true;
}
return resolve(existe);
});
}
const registerUser = ({ name, email }) => {
//devolvemos la ejecución (sin 'then' o 'catch') y el resultado si que se imprime, pero no el mensaje (obvio):
return getByEmail(email);
}
const middleware = async (req, res) => {
try {
const resultado = await registerUser(req.user);
console.log(resultado);
} catch (e) {
console.error(e);
}
}
// mostrará false
middleware({user: {name: 'Sansa', email: '[email protected]'}});
// mostrará error
middleware({user: {name: 'Joffrey', email: undefined}});
Seamos más creativos, y no hagamos la función middleware
tipo async
. En vez de eso, como estamos devolviendo una Promesa (la función getByEmail
devuelve una Promesa) podemos decir que registerUser
también devuelve una Promesa (return getByEmail()
) y por lo tanto, se puede implementar then()
y catch()
sobre registerUser
:
const emails = ['[email protected]','[email protected]'];
const getByEmail = (email) => {
return new Promise((resolve, reject) => {
if(!email) {
error = new Error('No se especificó un correo');
return reject(error);
}
let existe = false;
if(emails.includes(email)) {
existe = true;
}
return resolve(existe);
});
}
const registerUser = ({ name, email }) => {
//devolvemos la ejecución (sin 'then' o 'catch') y el resultado si que se imprime, pero no el mensaje (obvio):
return getByEmail(email);
}
// ahora middleware no es async, pero da igual porque registerUser implementa then y catch
const middleware = (req, res) => {
registerUser(req.user)
.then(result => {
console.log(`Existe el correo? ${result}`);
})
.catch (error => {
console.error(error);
});
}
// mostrará false
middleware({user: {name: 'Sansa', email: '[email protected]'}});
// mostrará error
middleware({user: {name: 'Joffrey', email: undefined}});
Segundo: Uso sensato de then
y catch
En la primera parte vimos lo que significa usar return
al momento de devolver resultados de nuestros métodos, ahora vamos a ver el uso sensato de los métodos then()
y catch
y lo que debemos hacer para evitar futuros errores.
Lo primero es entender la lógica de nuestro programa. El mismo trata de registrar un usuario sí y sólo sí, el mismo no existe en la BD. Asumimos que el usuario existe si ya su correo se encuentra en la lista de correos.
Entonces, vamos a usar un middleware tipo async
y usaremos las funciones ya conocidas. Y vamos a completar la función de registro de usuario para que realice o rechace la solicitud.
Una forma de hacerlo (usando then
y catch
) sería la siguiente:
const emails = ['[email protected]','[email protected]'];
// esta función devuelve un booleano o lanza un error
const getByEmail = (email) => {
return new Promise((resolve, reject) => {
if(!email) {
error = new Error('No se especificó un correo');
return reject(error);
}
let existe = false;
if(emails.includes(email)) {
existe = true;
}
return resolve(existe);
});
}
// esta función devuelve el resultado de la solicitud de registro.
const registerUser = ({ name, email }) => {
return getByEmail(email)
.then(result => {
if(result) {
// aquí devolvemos el resultado.
return {message: 'Usuario existe', inserted: false};
}
// si el usuario no existe, lo agregamos y devolvemos el resultado
emails.push(email);
return {message: 'Usuario registrado con éxito', inserted: true};
})
.catch(error => {
//si hay algún error, devolvemos el mismo
// notemos que este es el punto de captura
// no deberíamos seguir propagando el error
// con un 'throw' (lo he visto antes)
return {message: `Error al registrar usuario: ${error.message}`, inserted: false};
});
}
const middleware = async (req, res) => {
try {
const result = await registerUser(req.user);
console.log(result.message);
console.log(emails);
// lógica para continuar con la respuesta al usuario
} catch (e) {
console.error(e.message);
//lógica en caso de error
}
}
// mostrará que se agregó usuario correctamente
middleware({user: {name: 'Sansa', email: '[email protected]'}});
// mostrará error
middleware({user: {name: 'Joffrey', email: undefined}});
// mostrará que usuario ya existe y no lo agregará
middleware({user: {name: 'Jon', email: '[email protected]'}});
Como podemos observar, usar then
y catch
puede ser un poco enredoso, y es que cuando tenemos mucho código (o el código es denso) podemos perder de vista el objetivo final de lo que deben devolver nuestras funciones.
Mi comentario incial: Si el método getByEmail es asíncrono y provee then() y catch(), ¿porqué usar return seguido de then() (y de catch)? se refiere que no veo particularmente útil tu función registerUser
ya que sólo es un envoltorio para getByMail
. Y la conclusión a la que puedo llegar es que puedes escribir toda tu lógica usando directamente getByMail
:
const emails = ['[email protected]','[email protected]'];
// esta función devuelve un booleano o lanza un error
const getByEmail = (email) => {
return new Promise((resolve, reject) => {
if(!email) {
error = new Error('No se especificó un correo');
return reject(error);
}
let existe = false;
if(emails.includes(email)) {
existe = true;
}
return resolve(existe);
});
}
const middleware = async (req, res) => {
try {
const existe = await getByEmail(req.user.email);
if(existe) {
// aquí devolvemos el resultado al cliente
console.log('Usuario existe');
return {message: 'Usuario existe', inserted: false};
}
//insertamos si no existe:
emails.push(req.user.email);
console.log('Usuario agregado');
console.log(emails);
return {message: 'Usuario registrado con éxito', inserted: true};
} catch (e) {
console.error(e.message);
//lógica en caso de error
}
}
// mostrará que se agregó usuario correctamente
middleware({user: {name: 'Sansa', email: '[email protected]'}});
// mostrará error
middleware({user: {name: 'Joffrey', email: undefined}});
// mostrará que usuario ya existe y no lo agregará
middleware({user: {name: 'Jon', email: '[email protected]'}});
Dicho esto, si lo que deseas es tener tu lógica dentro de una función llamada registerUser
y que la misma devuelva realmente un resultado basado en lo que devuelve tu función getByEmail
, mi sugerencia es que conviertas tu función registerUser
en tipo async
y captures el resultado de getByEmail
usando await
. Con eso, tu código se verá incluso mejor.
Espero que esto aclare las dudas que tengas al respecto.
Edición
Como te digo en mi comentario, existen muchas formas de realizar la tarea que requieres, todo dependerá de tu configuración y de la forma en que realizas las cosas con respecto a los datos que almacenas en la Base de Datos.
En tu caso, veo que trabajas con SQL, por lo tanto lo ideal (aunque esto puede variar de aplicación en aplicación) es crear una restricción sobre el campo correo
. De esta forma, dejas que sea la base de datos la que te informe o la que se encargue de rechazar una inserción cuando un correo ya está almacenado en otro registro. Eso evitará el paso adicional de verificar antes de insertar.
Ahora, lo que deseas es poder realizar el proceso para entender el trabajo con Promesas. Tomaré mi ejemplo y crearé un proceso que devuelva una Promesa al momento de insertar el usuario.
Una forma de trabajarlo sería la siguiente:
const emails = ['[email protected]','[email protected]'];
// esta función devuelve un booleano o lanza un error
const getByEmail = (email) => {
return new Promise((resolve, reject) => {
if(!email) {
error = new Error('No se especificó un correo');
return reject(error);
}
let existe = false;
if(emails.includes(email)) {
existe = true;
}
return resolve(existe);
});
}
const insert = (email) => {
// la siguiente función siempre inserta un correo en el array
return new Promise((resolve, reject) => {
//simulamos un proceso que dura 1 segundo
setTimeout(() => {
emails.push(email);
return resolve(true);
}, 1000);
});
}
const registrarUsuario = async (user) => {
try {
const existe = await getByEmail(user.email);
if(existe) {
// lógica en caso de que el usuario exista
// en este caso devolveré un booleano
return false;
}
// aqui insertamos el usuario ya que no existe
return await insert(user.email);
} catch(e) {
// lógica para capturar el error
console.error(e.message);
// no se insertó el usuario, se devuelve false
return false;
}
}
// podemos usar async/await, ya que insertarUsuario es async
// o podemos usar then/catch.
const middleware = (req, res) => {
registrarUsuario(req.user)
.then(inserted => {
if(!inserted) {
// lógica para indicar que no se insertó el usuario
console.log(`Usuario ${req.user.email} no insertado`);
console.log(emails);
return;
}
// lógica para indicar que se insertó
console.log(`Usuario ${req.user.email} insertado correctamente`);
console.log(emails);
})
.catch(error => {
// lógica en caso de error
console.error(error.message);
});
}
// mostrará que se agregó usuario correctamente
middleware({user: {name: 'Sansa', email: '[email protected]'}});
// mostrará error
middleware({user: {name: 'Joffrey', email: undefined}});
// mostrará que usuario ya existe y no lo agregará
middleware({user: {name: 'Jon', email: '[email protected]'}});
De esta forma podrías hacer la inserción. Sin embargo, mi recomendación es dejar que la Base de Datos se encargue de la verificación de unicidad del registro.
false
cuando debería devolvertrue
en el momento de insertar.getByEmail
, como tu función devuelve untrue
,false
o si pasa algo un error, te va imprimir eso primero y después mostrará loslog
que haces en la función.