【发布时间】:2017-10-24 20:44:33
【问题描述】:
我正在尝试使用 node 和 express 制作 API。 这是创建用户的函数。
我不确定我是否正确处理错误,因为异步 mongodb 函数,我感觉自己正处于“承诺地狱”中。接下来我要做的是获取插入用户的 id,我想这将是另一个承诺,另一个需要处理的错误......
exports.create = function(req, res, next) {
var errors = [];
var userData = req.body;
// exit if the user didn't fill all fields
var requiredFields = ['first_name',
'last_name',
'login',
'email',
'password',
'sex'];
requiredFields.forEach(function(elem) {
if (!userData.hasOwnProperty(elem))
errors.push('The field \'' + elem + '\' is missing');
});
if (errors.length !== 0)
res.status(400).json({errors: errors});
// check if the user or the login are already in use
db.connection.collection(COLLECTION_NAME).findOne({ $or: [
{ email: userData.email },
{ login: userData.login }
]})
.then(function(data) {
// if there is no user (null) we can create it
if (data === null) {
db.collection(COLLECTION_NAME).insertOne(userData).then(function (data) {
res.status(201).json("success");
}, function (err) {
res.status(400).json({errors: ["DB error: cannot create user"]});
})
} else {
errors.push('An user is already registered with this email or this login.');
if (errors.length !== 0)
res.status(400).json({errors: errors});
}
}, function (err) {
res.status(400).json({errors: errors});
})
}
有没有最好的方法来做到这一点?
顺便说一句,我不能使用验证库和猫鼬。
谢谢。
【问题讨论】:
-
你可以在 then 语句中返回一个新的 promise...
标签: javascript node.js mongodb error-handling promise