节点js函数需要返回值,但值来自回调,因此无法返回

时间:2013-10-12 21:08:07

标签: javascript node.js express

对于令人困惑的标题道歉,我将尝试解释

我正在使用带有express-form的node.js和自定义验证函数,如下所示:

app.post('/register', form(
    field("username").trim().required().custom(function(value) {
       //Works correctly
       if (!user.validUsername(value)) {
           throw new error("Invalid username");
       }

       //Does not work due to callback
       user.uniqueUsername(value, function(uniqueUsername) {
            if (!uniqueUsername) {
                //Not unique, throw error
                throw new Error("username is already taken");
                //Could be subsituted for return "whatever"; and it would still be the same problem
            }
        });
    }),
    registerAttempt);
};

// ...
//Example function for brevity  
user.uniqueUsername = function(username, callback) {
    User.findOne({username: username}, function(err, user) {
        if (err) { throw err; }
        callback(user === null);
    });
}

我需要一种方法来重构这个,所以.custom(function(value){..})在我收到回调之前没有完成执行但是我不知道如何做到这一点,如果有的话。

编辑:纠正错误

2 个答案:

答案 0 :(得分:2)

看起来快递形式不支持异步验证,所以唯一的方法就是实现自定义中间件。它看起来像这样:

app.post('/register',
    function (req, res, next) { //here it goes
        user.uniqueUsername(req.body.username, function(uniqueUsername) {
            req.body.uniqueUsername = uniqueUsername; //we can't do anything about it yet
            next();
        });    
    },

   form(field('username').trim().required().custom(function(value) {
        if (!user.validUsername(value)) {
             throw new error("Invalid username");
        }   
   }),
   field('uniqueUsername').custom(function(value) {
        if (!value) {
             throw new Error("username is already taken");
        }   
   }),

   registerAttempt);

它可以用作表达形式无法进行异步验证的通用解决方法:我们在单独的中间件中提取所需的所有数据,并将其作为新字段注入,然后通过快速形式进行验证。

答案 1 :(得分:-1)

对于这种情况,您需要反转呼叫顺序。类似的东西:

User.findOne({username: username}, function(err, uniqueUsername) {
    if (err) { throw err; }

    app.post('/register', form(
        field("username").trim().required().custom(function(value) {
            if (!user.validUsername(value)) {
                throw new error("Invalid username");
            }
            if (!(uniqueUsername === null)) {
                //Not unique, throw error
                throw new Error("username is already taken");
            }
        }),
        registerAttempt
    );
};

基本上,你需要在使用它们之前获取这些值(如果说这看起来很明显,但是异步编程往往会使显而易见的事情变得不那么明显)。

如果您需要检查多个异步项,那么您需要以某种方式将它们链接在一起。如有必要,异步库可以提供帮助。