ノードpromises.all（）が正常に動作しません

Question

promise.all()を使用してループの後に関数のリストを同期して実行しようとしていますが、コードが期待通りに機能しません。

誰かが私が間違っていたことを知っていますか？

私のコードは次のとおりです。私は

push promise 0 
push promise 1 
push promise 2 
---> foo i 0 
---> foo i 1 
---> foo i 2 
done 

を期待するところ

var Promise = require('promise'); 
var promises = []; 

for (var i = 0; i < 3; i++) { 
    console.log('push promise ' + i); 
    promises.push(foo(i)); 
} 

Promise.all(promises) 
     .then(function (data) { 
      console.log("done"); 
     }); 

function foo(i) { 
    return new Promise(function (resolve, reject) { 
     console.log('---> foo i ' + i); 
     resolve('done foo ' + i); 

    }); 
} 

が、出力は

push promise 0 
---> foo i 0 
push promise 1 
---> foo i 1 
push promise 2 
---> foo i 2 
done 

で事前にありがとうございます。

Answer 1

を与えるべきではあなたが約束コンストラクタを呼び出すときに、それに渡される関数は、建設通話中に呼び出されていることです。 プロキシの内部に非同期呼び出しがある場合（意図したユースケース）、期待していた動作が作成されます。

私のコード例は、setTimeoutを非同期呼び出しとして使用し、resolve呼び出しが非同期コールバック内でどのように行われるかに注意してください。

var promises = []; 

for (var i = 0; i < 3; i++) { 
    console.log('push promise ' + i); 
    promises.push(foo(i)); 
} 

Promise.all(promises) 
     .then(function (data) { 
      console.log("done"); 
     }); 

function foo(i) { 
    return new Promise(function (resolve, reject) { 
     // The async call is made inside the Promise 
     setTimeout(()=> { 
      console.log('---> foo i ' + i); 
      // And the resolve call is made inside the callback function 
      resolve('done foo ' + i); 
     }); 
    }); 
} 

これを実行すると、あなたは所望の出力（クロームでテスト）を取得する必要があります

push promise 0 
push promise 1 
push promise 2 
---> foo i 0 
---> foo i 1 
---> foo i 2 
done 

追記：こののみ動作（foo i 0, ... 1 ... 2が順になっている）要求は全て即座に行われているので1つずつそれがajax呼び出しだった場合、fooステートメントは完了の順に表示されます。したがって、要求2が0または1より前に終了した場合、最初にfoo i 2が表示されます。しかし、は、すべての要求が解決された場合にのみ表示されます。

Answer 2

resolveメソッドの外にあり、代わりにPromiseコンストラクタに指定したエグゼキュータ関数の中に、console.log('---> foo i ' + i);という行が約束されて実行されています。

これは、ここで何が起こっている、あなたの期待出力

var Promise = require('promise'); 
var promises = []; 

for (var i = 0; i < 3; i++) { 
    console.log('push promise ' + i); 
    promises.push(foo(i)); 
} 

Promise.all(promises) 
     .then(function (data) { 
      for (var i = 0; i < data.length; i++) { 
      console.log(data[i]); 
      }   
      console.log("done"); 
     }); 

function foo(i) { 
    return new Promise(function (resolve, reject) { 
     resolve('--> foo i ' + i); 
    }); 
} 

Answer 3

私はpromise.all（）を使用してループの後に関数のリストを同期して実行しようとしていますが、コードが期待通りに機能しません。

あなたは機能のリストを同期して実行したいと思います。その場合は、Bluebird Promise.each()を使用することをお勧めします。

var Promise = require('bluebird'); 
var promises = []; 

for (var i = 0; i < 3; i++) { 
    console.log('push promise ' + i); 
    promises.push(foo(i)); 
} 

Promise.each(promises, function(result) { 
    console.log(result); 
}); 

function foo(i) { 
    return new Promise(function (resolve, reject) { 
     resolve('done foo ' + i); 
    }); 
} 

結果：

push promise 0 
push promise 1 
push promise 2 
done foo 0 
done foo 1 
done foo 2