Ando estudando a biblioteca (Express) e a linguagem (JavaScript), mas esse tipo de repetição abaixo acaba me incomodando e. Gostaria de saber o que vocês fariam na situação para reduzir a repetição de código.
Este é um middleware que estou fazendo para uma rota e, se reparar, há uma seção em que verifico o username
, password
, email
, fullName
e birthDate
, que retornam exatamente a mesma coisa, porém com o nome do modelo alterado. Acabei gastando 5 linhas para fazer isso.
A questão é: Existe alguma forma para se reduzir essa repetição de código?
module.exports = {
register: async (req, res) => {
const {
username,
password,
email,
fullName,
birthDate
} = req.body
if (!username) return res.json({error: 'Username not found.'})
if (!password) return res.json({error: 'Password not found.'})
if (!email) return res.json({error: 'Email not found.'})
if (!fullName) return res.json({error: 'Full name not found.'})
if (!birthDate) return res.json({error: 'Birthdate not found.'})
const userEmailExists = await userService.fetch( { email } ) ? true : false
const userNameExists = await userService.fetch( { username } ) ? true : false
if (userEmailExists) return res.json({error: 'User email already exists.'})
if (userNameExists) return res.json({error: 'Username already exists.'})
const user = await userService.create({ username, password, email, fullName, birthDate })
user.password = undefined
return res.status(200).json(user)
}
}