Javascript 如何避免巨大的嵌套,如果不是这样的话
这是我的登录代码Javascript 如何避免巨大的嵌套,如果不是这样的话,javascript,node.js,hapijs,Javascript,Node.js,Hapijs,这是我的登录代码 method: 'POST', path: '/api/login/sp', config: { auth: false }, handler: function (request, reply) { User.findOne({ phone: request.payload.phone }, function (err, user) { if (err) throw err
method: 'POST',
path: '/api/login/sp',
config: { auth: false },
handler: function (request, reply) {
User.findOne({ phone: request.payload.phone }, function (err, user) {
if (err) throw err;
if (user !== null) {
user.comparePassword(request.payload.password, function (err, isMatch) {
if (err) throw err;
if (isMatch) { // Login success
data = {
"statusCode": 200,
"token": generateJWT(user._id)
}
return reply(data);
}
else {
reply(Boom.unauthorized('Invalid Account'))
}
});
}
else { // Invalid User
reply(Boom.unauthorized('Invalid Account'))
}
});
}
它需要大量的代码,并且很难阅读。是否有更好的方法编写这部分代码,使其易于维护和阅读?您可以使用
return reply()
:
尝试使用早期返回模式:
这可能应该启用,而不是在此使用承诺。异步函数是否返回承诺?您可以使用
Promise
避免箭头结构:这是。
User.findOne({phone: request.payload.phone}, function (err, user) {
if (err) throw err;
if (user === null) return reply(Boom.unauthorized('Invalid Account'));
user.comparePassword(request.payload.password, function (err, isMatch) {
if (err) throw err;
if (!isMatch) return reply(Boom.unauthorized('Invalid Account'));
data = {
"statusCode": 200,
"token": generateJWT(user._id)
};
return reply(data);
});
})
User.findOne(..., {
// generic error
if (err) throw err;
// invalid user
if (user === null) {
reply(Boom.unauthorized('Invalid Account'));
return;
}
user.comparePassword(..., {
if (err) throw err;
if (!isMatch) {
reply(Boom.unauthorized('Invalid Account'));
return;
}
data = {
"statusCode": 200,
"token": generateJWT(user._id)
};
reply(data);
});
});