Question

我正在Parse的Cloud Code中编写一个函数，它用5张扑克牌填充服务器。这个想法是服务器应该等到所有5张卡都在数据库中，然后再转到函数的下一部分（目前只是向控制台输出一条消息。

我所看到的是，所有5张卡都被添加到数据库中。它们也不一定按顺序出现在数据库中，这使我相信它们是正确地异步添加的。对于大多数人而言，我的日志记录显示了预期的行为。

I2015-01-03T17:08:01.244Z] Attempting to create cards x5
I2015-01-03T17:08:01.245Z] Attempting to save card0
I2015-01-03T17:08:01.247Z] Attempting to save card1
I2015-01-03T17:08:01.248Z] Attempting to save card2
I2015-01-03T17:08:01.249Z] Attempting to save card3
I2015-01-03T17:08:01.250Z] Attempting to save card4
I2015-01-03T17:08:01.352Z] Card saved:4
I2015-01-03T17:08:01.353Z] Card saved:4
I2015-01-03T17:08:01.354Z] Card saved:4
I2015-01-03T17:08:01.355Z] Card saved:4
I2015-01-03T17:08:01.356Z] Card saved:4
I2015-01-03T17:08:01.357Z] ALL 5 Promises theoretically fulfilled

但是，请注意，当实际保存卡时，每个卡的日志使用相同的编号 - 在这种情况下“卡已保存：4”。

问题是......我的承诺是否符合预期？以及如何修复我的错误以显示已保存的实际卡号？

这是我的代码：

Parse.Cloud.define("populateServer", function(request, response)
{
    console.log("Attempting to create cards x5");
    var promises = createCards(5);

    Parse.Promise.when(promises).then(function(result)
    {
        console.log("ALL 5 Promises theoretically fulfilled");
    });
});

function createCards(qty)
{
    var promises = [];

    for (i=0;i<qty;i++)
    {
        var Card = Parse.Object.extend("Card");
        var card = new Card();

        card.set("name", "test");
        card.set("number", i);

        console.log("Attempting to save card" +i);

        var promise = card.save();

        promises.push(promise);

        promise.then(function() {
            console.log("Card saved:" +i);
        }, function(error) {
            console.log("Uh oh, something went wrong.");
        });
    }
    return promises;
}

Answer 1

你实际上正在获得你想要的行为，但i在你的承诺结束时总是4，因为它在任何then处理程序执行之前递增（你的for循环是同步的，但你的处理程序是异步的）。

通常传递给all的承诺不会同步解决，但由于您通过then每个承诺创建一个链，您的调用只会在前一个承诺完成后才会触发。

旁注：我对Parse的承诺实现并不是非常熟悉，但你可以保存自己创建一个额外的数组并通过从函数返回promise链来调用Parse.Promise.when： / p>

function createCards(qty) {
      // create a resolved promise that starts the "chain"
      // for Parse prommises use Parse.Promise.as();
      var promiseChain = Parse.Promise.as();

      // start the for loop
      for (i=0;i<qty;i++) {
        // add another promise onto our chain
        // you can `then` or `catch` off of this
        // now if you want
        promiseChain = promiseChain.then(function () {
          // ... any additional logic
          card.save();
        });
      }

      return promiseChain;
}

// usage
createCards(5)
  .then(function () {
    console.log('finished');
  });

Answer 2

正如尼克所说，你的计数器（i）在任何then处理程序执行之前增加到4，所以所有处理程序自然会使用i的最终值（这是4）。

围绕此问题的最快解决方法可能是将then处理程序的赋值括在一个闭包中，在该闭包中，您可以使用变量来记住闭包时i的值。调用。

例如，替换它：

promise.then(function() {
    console.log("Card saved:" +i);
}, function(error) {
    console.log("Uh oh, something went wrong.");
});

有了这个：

(function(){
    var myIndex = i;
    promise.then(function() {
        console.log("Card saved:" + myIndex);
    }, function(error) {
        console.log("Uh oh, something went wrong.");
    });
})();

我做了几个小动作，看看行动的不同之处：Without closure - With closure

我的承诺是否符合预期？

2 个答案: