Question

我正在尝试使用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});
    })
}

有最好的方法吗？

顺便说一句，我不能使用验证库，也不能使用猫鼬。

感谢。

Answer 1

  // 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) {  
         return  db.collection(COLLECTION_NAME).insertOne(userData)
     }else{ 
         return new Promise.reject(0);//throw the error
     }
 }).then(function (data) {
      res.status(201).json("success");
 }, function (err) {
      res.status(400).json({errors: ["DB error: cannot create user"]
});

你可以将承诺链接起来......

如何避免级联承诺和捕获？

1 个答案: