对于令人困惑的标题道歉,我将尝试解释
我正在使用带有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){..})在我收到回调之前没有完成执行但是我不知道如何做到这一点,如果有的话。
编辑:纠正错误
答案 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
);
};
基本上,你需要在使用它们之前获取这些值(如果说这看起来很明显,但是异步编程往往会使显而易见的事情变得不那么明显)。
如果您需要检查多个异步项,那么您需要以某种方式将它们链接在一起。如有必要,异步库可以提供帮助。