I’m wondering if there’s a way to clean this up or if I need to put a separate resolve() within each if/else like it is now. When I put it outside the three it says message is not defined.
function checkAddUser(input) {
return new Promise((resolve,reject) =>{
if(input.email === '') {
const message = 'Please enter an email address.';
resolve(message);
} else if (input.first_name === '') {
const message = 'Please enter your first name.';
resolve(message);
} else {
const message = false;
resolve(message);
}
});
}
>Solution :
There is no need to return a promise, since no asynchronous calls are made. To clean up the code I’d suggest this:
function checkAddUser(input) {
if(input.email === '') {
return 'Please enter an email address.';
}
if (input.first_name === '') {
return 'Please enter your first name.';
}
return false;
}