然后承诺不等待结果返回

Question

以下是node.js中使用promise的函数。

exports.getDetailsByUserId = function(req,res) {
     getUserDetails(req, res) // function to get user details
    .then(function (Details) {
        var allpromises = Details.map(getStatus); //get and update the status of user
        return allpromises
    })
    .then(function (promises) {
        var orderresult =[];
        Q.allSettled(promises)
        .then(function (allpromises) {
            orderresult.push(allpromises);
        })
        return orderresult;
    })
    .then(function (result) {
        getUserDetails(req, res)//again calling get user details so that i will get modified result
       .then(function (data) {
            res.json({                             
                "User" : data
            })
        })
    })
    .catch(function (error) {
        console.log(error);
        res.status(200).json({
            "Message": error
        })
    })
}

var getStatus = function (data,req) {
    var deferred = Q.defer();
        requestify.post('http://example.com', {
            "data": [{ "Number": No }]
        })
    .then(function (response) {
            response.getBody();
            response.body;
            var Status = JSON.parse(response.body);
          var Details = { "_id": data._id, "Status": Status.status[0] };
          return OrderDetails;
    })
    .then(function (DetailsData){
        var req = { body : { Details :DetailsData} };
       updateDetails(req).then(function (data) { 
          deferred.resolve;
        })
    })
   .catch(function (error) {                     
        deferred.resolve(error);
   });    
    return deferred.promise;
}

getUserDetails 函数第一次获取用户的详细信息，然后getStatus 调用另一个 API 并更新相关的用户状态。至此，一切正常。

我需要再次调用getUserDetails 以获取更新结果，但它没有给出更新结果。也许它在更新状态之前被调用？代码可能有什么问题，以便在Q.all 之后执行？

谢谢。

Answer 1

var orderresult =[];
Q.allSettled(promises)
.then(function (allpromises) {
    orderresult.push(allpromises);
})
return orderresult;

你爱上了经典的mutation in asynchronous callback 问题。 then 回调是异步的，orderresult 在返回时不会包含 allpromises 数组。要让某些东西等待承诺，你必须return它。

无论如何，您都应该将代码简化为

getUserDetails(req, res)
.then(function (details) {
    var allpromises = details.map(getStatus);
    return Q.allSettled(allpromises); // just call it here already and return a promise
})
.then(function (allpromiseresults) {
    var orderresult = [allpromiseresults]; // not sure what this is good for
    return orderresult;
})
.then(function (result) { // given that you don't use the `result` anyway
    return getUserDetails(req, res);
}).then(function (data) {
    res.json({                             
        "User" : data
    })
}, function (error) {
    console.log(error);
    res.status(200).json({
        "Message": error
    })
});

在getStatus 中，您应该避免使用deferred antipattern：

function getStatus(data,req) {
    return requestify.post('http://example.com', {
        "data": [{ "Number": No }]
    })
    .then(function (response) {
        response.getBody(); // ???
        var status = JSON.parse(response.body);
        var details = { "_id": data._id, "Status": Status.status[0] };
        // no `OrderDetails`, no `DetailsData`

        var req = {body: {Details: details}};
        return updateDetails(req) // probably "getStatus" should be called "updateStatus"
    });
}