在 node.js 的 Restify 模块中正确放置 return next() 的位置？

Question

这是一个从数据库中查询用户的示例，如果用户存在，则另一个查询电话：

module.exports = function (username, req, res, next) {

    var query = User.where('username', new RegExp('^' + username + '$', 'i'));
    query.findOne(function (err, user) {
        if (err) {
            res.send("error");
            //## position D
        } else if (!user) {
            res.send("user not found");
            //## position D
        } else {
            //user exists
            //search for phone
            var query2 = Phone.where('phone', new RegExp('^' + user.phone + '$', 'i'));
            query2.findOne(function (err, phone) {
                if (err) {
                    res.send("error");
                    //## position D
                } else if (!phone) {
                    res.send("phone not found");
                    //## position D
                } else {
                    res.send(phone);
                    //## position D
                }
                //## position C
            });
        }
        //## position B
    });
    //## position A
}

我的问题是：根据这段代码，正确的设置位置在哪里 返回下一个(); ?为什么？

我提出了一些建议：

• 位置 A：@ 函数底部。在这里它甚至会被称为 早于从数据库中查询数据

• 位置B：@第一个回调函数的底部

• 位置C：@最后一个回调函数的底部

• 位置 D：在 res.send(..) 之后立即

P.S：我知道我可以将错误作为对象传递到 next.. 我只是想简化代码以专注于主要问题。

谢谢yokefellows，'。

Answer 1

当然是位置 D，但你可以通过回调获得“更干净”的东西。

module.exports = function (username, req, res, next) {

    var query = User.where('username', new RegExp('^' + username + '$', 'i'));
    query.findOne(function (err, user) {
        if (err) {
            return next(err);
        }
        if (!user) {
          res.send("user not found");
          return next();
        }
        //user exists
        //search for phone
        var query2 = Phone.where('phone', new RegExp('^' + user.phone + '$', 'i'));
        query2.findOne(function (err, phone) {
            if (err) {
                return next(err);
            }
            if (!phone) {
                res.send("phone not found");
            } else {
                res.send(phone);
            }
            next();
        });
    });
}