Question

所以我使用Node.js + Mongoose + Lie作为promise库。代码如下：

var User = require('../models/user'),
    Promise = require('lie'),
    jwt = require('jsonwebtoken'),
    config = require('../../config');

module.exports = function(express) {
    var router = express.Router();

    router.post('/', function(req, res) {
        if (!req.body.username) return res.json({ success: false, reason: 'Username not supplied.' });
        if (!req.body.password) return res.json({ success: false, reason: 'Password not supplied.' });

        var findUser = new Promise(function(resolve, reject) {
            User.findOne({ username: req.body.username }, function(err, user) {
                if (err) reject (err);
                if (!user) reject({ success: false, reason: 'User not found or password is incorrect.' });
                if (!user.validPassword(req.body.password)) reject({ success: false, reason: 'User not found or password is incorrect.' });
                resolve(user);
            });
        });

        var sendToken = function(user) {
            var token = jwt.sign({ username: user.username }, config.secret, { expiresIn: 2 * 60 * 60 });
            res.json({ success: true, token: token });
        };

        findUser.then(function(value) {
            sendToken(value);
        }).catch(function(reason) {
            res.send(reason);
        });

    return router;
};

所以基本上它是一个身份验证路由，如果一切正常，它会发送签名的jwt。 奇怪的行为是，如果我发送错误的用户名，服务器会抛出错误说

TypeError: Cannot read property 'validPassword' of null

因此，它表示user === null在达到密码有效性检查时，但是，在此有效性检查之前，检查是否找到了用户，主要是

if (!user) reject({ success: false, reason: 'User not found or password is incorrect.' });

此处服务器已经知道user === null，因此它应该使用success: false拒绝此承诺，并且这段代码应该被抛入catch部分，但它不会发生。

请问任何想法？

我应该补充一点，如果我将if (!user.validPassword...部分更改为else if而不是if，那么它的工作原理应该如此。但是我无法理解为什么在承诺被拒绝后仍然执行任何代码。

修改

在学习节点时，我主要使用MEAN Machine book，这是一个很好的帮助，他们使用这种语法：

if (!user) {
    //
} else if (user) {
    //
}

但我想它也应该按照我的方式运作。

Answer 1

您并未停止在reject上执行您的功能，因此如果findOne导致结果为空validPassword仍然会被调用。

您可能需要添加return：

 User.findOne({ username: req.body.username }, function(err, user) {
   if (err)
     return reject (err);
   if (!user)
     return reject({ success: false, reason: 'User not found or password is incorrect.' });
   if (!user.validPassword(req.body.password))
     return reject({ success: false, reason: 'User not found or password is incorrect.' });
   resolve(user);
 });

或使用else if：

 User.findOne({ username: req.body.username }, function(err, user) {
   if (err) { 
     reject (err);
   } else if (!user) {
     reject({ success: false, reason: 'User not found or password is incorrect.' });
   } else if (!user.validPassword(req.body.password))
     reject({ success: false, reason: 'User not found or password is incorrect.' });
   } else {
     resolve(user);
   }
 });

具有Mongoose和promises的Node中的奇怪异步行为

修改

1 个答案: