是否有更好的依赖函数调用方法与堆叠它们相比?

时间:2015-06-21 20:00:34

标签: node.js mongodb function express dependencies

我在Express路由器代码中,使用Jade渲染并从MongoDB中提取数据以准备它。是否有更好的方法将所需的所有数据与下面的结构相结合?请注意,表餐馆,客户和驱动程序之间没有依赖关系,要求首先返回 - 我只需要在页面呈现之前执行所有三个。

在某些情况下,其中一些级联调用变得非常深入。必须有一个更好的结构,对吧?

router.get('/iframe_map/:restaurantid/:customerid', function(req, res, next) {
  Restaurant.findOne({'_id': req.params.restaurantid}, '-_id address name phone status lat lng LatLng marker', function(restaurantErr, restaurantDoc) {
    Customer.findOne({'_id': req.params.customerid}, '-_id address name phone status lat lng LatLng marker', function(customerErr, customerDoc) {
      Driver.find({}, '-_id letterOrdinal address firstName lastName phone status lat lng LatLng marker', {sort: {letterOrdinal: 1}}, function(driverErr, driverDocs) {
        res.render('iframe_map', {
          title:              "Google Map",
          defaultZoomLevel:   12,
          defaultCityState:   ", San Diego, CA",
          oRestaurant:        restaurantDoc,
          oCustomer:          customerDoc,
          oDrivers:           driverDocs
        });  // End of res.render()
      });    // End of Driver.find()
    });      // End of Customer.find()
  });        // End of Restaurant.find()
});          // End of router.get()

1 个答案:

答案 0 :(得分:2)

这是用Async Waterfall重写的代码:

router.get('/iframe_map/:restaurantid/:customerid', function(req, res, next) {
    var obj = {};
    async.waterfall([
        function(done) {
            Restaurant.findOne({'_id': req.params.restaurantid}, '-_id address name phone status lat lng LatLng marker', function(err, items) {
                if (err) done(err);
                obj.restaurant = items;
                done(null, obj);
            });
        },
        function(obj, done) {
            Customer.findOne({'_id': req.params.customerid}, '-_id address name phone status lat lng LatLng marker', function(err, items) {
                if (err) done(err);
                obj.customer = items;
                done(null, obj);
            });
        },
        function(obj, done) {
            Driver.find({}, '-_id letterOrdinal address firstName lastName phone status lat lng LatLng marker', {sort: {letterOrdinal: 1}}, function(err, items) {
                if (err) done(err);
                obj.driver = items;
                done(null, obj);
            });
        }
    ], function (err, result) {
        res.render('iframe_map', {
          title:              "Google Map",
          defaultZoomLevel:   12,
          defaultCityState:   ", San Diego, CA",
          oRestaurant:        result.restaurant,
          oCustomer:          result.customer,
          oDrivers:           result.driver
        });
    });
});