Javascript for循环承诺

时间:2015-09-21 18:12:19

标签: javascript promise

我有一系列像这样的网址

var urls = ["www.google.com", "www.yahoo.com"];

我想循环访问url并在循环内执行异步任务,而不是继续执行下一项,直到异步任务完成。我知道你可以用承诺做到这一点,但我遇到了一些麻烦。我在这里有什么

var xmlReader = require('cloud/xmlreader.js');

function readResponse_async(xlmString) {
    var promise = new Parse.Promise();
    xmlReader.read(xlmString, function (err, res) {
        if(err) {
            promise.reject(err);
        } else {
            promise.resolve(res);
        }
    });
    return promise;
}

for (i = 0; i < urls.length; i++) {

    Parse.Cloud.httpRequest({
        url: unionUrls[i],
    }).then(function(httpResponse) {
            try {
//              console.log(httpResponse.text)
                return readResponse_async(httpResponse.text)
            } catch (e) {console.log(e)}

}

但是现在它不等待readResponse_async完成,我怎么能让它等待呢?

由于

修改

阅读完回复后,我将保存到我的数据库,我有另一个这样的数组

var location = ['USA', 'England'];

我像这样进行保存

function saveLoc_async(data, location) {
var i3, i4, i5, m,
            TestItem = Parse.Object.extend("TestItem"),//can be reused within the loops?
            promise = Parse.Promise.as();//resolved promise to start a long .then() chain
        for (i3 = 0; i3 < data.count(); i3++) {
             (function(testItem) {
                        testItem.set("item", data.at(i));
                        testItem.set("location", location);
                        //build the .then() chain
                        promise = promise.then(function() {
                            return testItem.save();
                        });
                    })(new TestItem());
//************************
//CALL  retry(); here?
//**************************

}

因为我的回答是

function retry() {
if (urlsUnion.length > 0) {
    var nextUrl = urlsUnion.pop();
    //********** ADDED LINE
    var nextLoc = location.pop();

    Parse.Cloud.httpRequest({
        url: nextUrl,
    }).then(function(httpResponse) {
        xmlReader.read(httpResponse.text, function (err, res) {
            if(err) {
                // show an error
            } else {
                //********** ADDED LINE
                saveLoc_async(res, nextLoc);
                retry();
            }
        });
    });
}
}

那么retry();应该去哪里,因为现在保存有时会将第二个位置与第一个项目之一放在一起?为什么会这样?

3 个答案:

答案 0 :(得分:2)

我为动画做了类似的事情。

var actions = [drawXXX, fadeOutYYY, drawXYZ];


this.startAnimation = function () {
    actions.reduce(function (previousAction, nextAction) {
        return previousAction.then(nextAction)
    }, $.when());
}

答案 1 :(得分:1)

您的代码会立即触发两个网址,而不会在中间等待。

您需要做的是从阵列中删除第一个网址并将其解雇。在'then'分支中检查你是否仍然在数组中有url并重复。

像这样(未经测试,编辑以使代码再次清洁):

var xmlReader = require('cloud/xmlreader.js');

function readResponse_async(xlmString) {
    xmlReader.read(xlmString, function (err, res) {
        if(err) {
            // show an error
        } else {
            readFirstUrl();
        }
    });
}

function readFirstUrl() {
    if (urlsUnion.length == 0) {
        return;
    }
    var url = urlsUnion.pop();
    Parse.Cloud.httpRequest({
        url: url,
    }).then(function(httpResponse) {
        readResponse_async(httpResponse.text);
    });
}

readFirstUrl();

答案 2 :(得分:0)

我不确定我是否理解您对unionUrls数组的使用,但如果您的网址在urls数组中,我认为这很干净:

function getUrl(url) {
      return Parse.Cloud.httpRequest(url)
                  .then( function(httpResponse) {
                     return readResponse_async(httpResponse.text);
                  });
}

urls.reduce( function(prev, url) {
   return prev ? prev.then( function() { getUrl(url); }) : getUrl(url);
 }, null);