如何解决NodeJS方法优先级问题

时间:2018-11-10 21:32:50

标签: javascript node.js mongodb mongoose

在这种情况下,方法3首先起作用,但出现错误。它的优先级必须类似于方法1,方法2和方法3。并保证异步工作。

我要检查是否正在使用新用户的用户名和电子邮件。如果未使用用户名或电子邮件,请进行注册。

我该如何解决这个问题?我是nodejs的新手。

module.exports.addUser = function(newUser, callback) {

    // method 1
    User.countDocuments({username: newUser.username}).then(count => {
        if(count > 0) {
            console.log("username in use");
            callback("username in use", null);
            return;
        }});

    // method 2
    User.countDocuments({email: newUser.email}).then(count => {
        if(count > 0) {
            console.log("email in use");
            callback("email in use", null);
            return;
        }});

    // method 3 , this method works first
    bcrypt.genSalt(10, (err, salt) => {
        console.log("salt here");
        bcrypt.hash(newUser.password, salt, (err, hash) => {
            if(err) throw err;
            newUser.password = hash;
            newUser.save(callback);
        });
    });
};

输出:

salt here
username in use
email in use

(node:7972) UnhandledPromiseRejectionWarning: Error: Can't set headers after they are sent.
    at validateHeader (_http_outgoing.js:491:11)
    at ServerResponse.setHeader (_http_outgoing.js:498:3)
    at ServerResponse.header (C:\Users\cyclone\Desktop\my_auth\node_modules\express\lib\response.js:767:10)
    at ServerResponse.send (C:\Users\cyclone\Desktop\my_auth\node_modules\express\lib\response.js:170:12)
    at ServerResponse.json (C:\Users\cyclone\Desktop\my_auth\node_modules\express\lib\response.js:267:15)
    at User.addUser (C:\Users\cyclone\Desktop\my_auth\routes\users.js:20:17)
    at User.countDocuments.then.count (C:\Users\cyclone\Desktop\my_auth\models\user.js:48:13)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:189:7)
(node:7972) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:7972) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled
will terminate the Node.js process with a non-zero exit code.
events.js:183
      throw er; // Unhandled 'error' event
      ^

Error: Can't set headers after they are sent.
    at validateHeader (_http_outgoing.js:491:11)
    at ServerResponse.setHeader (_http_outgoing.js:498:3)
    at ServerResponse.header (C:\Users\cyclone\Desktop\my_auth\node_modules\express\lib\response.js:767:10)
    at ServerResponse.send (C:\Users\cyclone\Desktop\my_auth\node_modules\express\lib\response.js:170:12)
    at ServerResponse.json (C:\Users\cyclone\Desktop\my_auth\node_modules\express\lib\response.js:267:15)
    at User.addUser (C:\Users\cyclone\Desktop\my_auth\routes\users.js:22:17)
    at C:\Users\cyclone\Desktop\my_auth\node_modules\mongoose\lib\model.js:4518:16
    at model.$__save.error (C:\Users\cyclone\Desktop\my_auth\node_modules\mongoose\lib\model.js:422:7)
    at C:\Users\cyclone\Desktop\my_auth\node_modules\kareem\index.js:315:21
    at next (C:\Users\cyclone\Desktop\my_auth\node_modules\kareem\index.js:209:27)
    at C:\Users\cyclone\Desktop\my_auth\node_modules\kareem\index.js:182:9
    at process.nextTick (C:\Users\cyclone\Desktop\my_auth\node_modules\kareem\index.js:499:38)
    at _combinedTickCallback (internal/process/next_tick.js:132:7)
    at process._tickCallback (internal/process/next_tick.js:181:9)
[nodemon] app crashed - waiting for file changes before starting...

1 个答案:

答案 0 :(得分:2)

countDocuments之后使用“ then”的事实确实表明这是一个承诺,因此是异步的。

这时最简单的解决方案是将addUser函数定义为async

module.exports.addUser = async function(newUser, callback) {

  // method 1
  const count1 = await User.countDocuments({
    username: newUser.username
  });

  if (count1 > 0) {
    console.log("username in use");
    callback("username in use", null);
    return;
  };

  // method 2
  const count2 = await User.countDocuments({
    email: newUser.email
  });

  if (count2 > 0) {
    console.log("email in use");
    callback("email in use", null);
    return;
  };

  // method 3 , this method works first
  bcrypt.genSalt(10, (err, salt) => {
    console.log("salt here");
    bcrypt.hash(newUser.password, salt, (err, hash) => {
      if (err) throw err;
      newUser.password = hash;
      newUser.save(callback);
    });
  });
};

但是,现在有一个addUser回调函数是没有意义的,因为异步函数会自动返回一个Promise。我建议做这样的事情...

module.exports.addUser = async function(newUser) {

  // method 1
  const count1 = await User.countDocuments({
    username: newUser.username
  });

  if (count1 > 0) {
    throw Error("username is in use");
  };

  // method 2
  const count2 = await User.countDocuments({
    email: newUser.email
  });

  if (count2 > 0) {
    throw Error("email in use");
  };

  let result = null;
  // method 3 , this method works first
  bcrypt.genSalt(10, (err, salt) => {
    console.log("salt here");
    bcrypt.hash(newUser.password, salt, (err, hash) => {
      if (err) throw err;
      newUser.password = hash;
      result = await newUser.save(callback);
    });
  });

  return result;

};

并在使用中看起来像:

addUser(someUserObject).then(result=>console.log(result)).catch(error=>{
  //Example: username in use
  console.log(error.message)
});