web-dev-qa-db-ja.com

ノードJS Promise.allとforEach

私は非同期メソッドを公開する構造のような配列を持っています。 asyncメソッド呼び出しは配列構造体を返すので、さらに多くの非同期メソッドが公開されます。この構造から取得した値を格納するために別のJSONオブジェクトを作成しているので、コールバックで参照を追跡することに注意する必要があります。

私はブルートフォースソリューションをコーディングしましたが、もっと慣用的な、あるいはクリーンなソリューションを学びたいと思います。

  1. このパターンは、nレベルのネストに対して繰り返し可能でなければなりません。
  2. 包含ルーチンをいつ解決するかを決定するには、promise.allまたは同様の手法を使用する必要があります。
  3. すべての要素が必ずしも非同期呼び出しを行う必要があるわけではありません。そのため、入れ子になったpromise.allでは、インデックスに基づいて単純にJSON配列要素に代入することはできません。それにもかかわらず、ネストされたforEachでpromise.allのようなものを使用して、すべてのプロパティの割り当てが、囲むルーチンを解決する前に行われるようにする必要があります。
  4. 私はbluebird promise libを使っていますが、これは必須ではありません

ここにいくつかの部分的なコードがあります - 

var jsonItems = [];

items.forEach(function(item){

  var jsonItem = {};
  jsonItem.name = item.name;
  item.getThings().then(function(things){
  // or Promise.all(allItemGetThingCalls, function(things){

    things.forEach(function(thing, index){

      jsonItems[index].thingName = thing.name;
      if(thing.type === 'file'){

        thing.getFile().then(function(file){ //or promise.all?

          jsonItems[index].filesize = file.getSize();
javascriptnode.jsasynchronouspromise
95
user3205931

いくつかの単純な規則を使えば、かなり簡単です。

  • あなたがthenで約束を作成するときはいつでも、それを返してください - あなたが返さない約束は、外部で待つことはできません。
  • あなたが複数のpromiseを作成するときはいつでも、それらの.all - それは全てのpromiseを待ち、それらのどれからのエラーも沈黙しません。
  • あなたがthensを入れ子にするときはいつでも、あなたは通常真ん中に戻ることができます - thenチェーンは通常せいぜい1レベルの深さです。
  • あなたがIOを実行するときはいつでも、それは約束を伴うべきです - それは約束の中にあるべきですかそれはその完了を知らせるために約束を使うべきです。

そしていくつかのヒント：

  • マッピングは.mapよりもfor/Pushのほうがうまくいきます - 関数を使って値をマッピングする場合、mapを使うとアクションを適用するという概念を簡潔に表すことができます。一つと結果を集約する。
  • 自由であれば並行性は逐次実行よりも優れています - 順番に実行するよりもPromise.allを実行するよりも並行して実行する方が良いです。次の前に。

それでは始めましょう。

var items = [1, 2, 3, 4, 5];
var fn = function asyncMultiplyBy2(v){ // sample async action
    return new Promise(resolve => setTimeout(() => resolve(v * 2), 100));
};
// map over forEach since it returns

var actions = items.map(fn); // run the function over all items

// we now have a promises array and we want to wait for it

var results = Promise.all(actions); // pass array of promises

results.then(data => // or just .then(console.log)
    console.log(data) // [2, 4, 6, 8, 10]
);

// we can nest this of course, as I said, `then` chains:

var res2 = Promise.all([1, 2, 3, 4, 5].map(fn)).then(
    data => Promise.all(data.map(fn))
).then(function(data){
    // the next `then` is executed after the promise has returned from the previous
    // `then` fulfilled, in this case it's an aggregate promise because of 
    // the `.all` 
    return Promise.all(data.map(fn));
}).then(function(data){
    // just for good measure
    return Promise.all(data.map(fn));
});

// now to get the results:

res2.then(function(data){
    console.log(data); // [16, 32, 48, 64, 80]
});
306
Benjamin Gruenbaum

これはreduceを使った簡単な例です。これはシリアルに実行され、挿入順序を維持し、Bluebirdを必要としません。

/**
 * 
 * @param items An array of items.
 * @param fn A function that accepts an item from the array and returns a promise.
 * @returns {Promise}
 */
function forEachPromise(items, fn) {
    return items.reduce(function (promise, item) {
        return promise.then(function () {
            return fn(item);
        });
    }, Promise.resolve());
}

そしてこれを次のように使います。

var items = ['a', 'b', 'c'];

function logItem(item) {
    return new Promise((resolve, reject) => {
        process.nextTick(() => {
            console.log(item);
            resolve();
        })
    });
}

forEachPromise(items, logItem).then(() => {
    console.log('done');
});

オプションのコンテキストをループに入れると便利です。コンテキストはオプションであり、すべての反復で共有されています。

function forEachPromise(items, fn, context) {
    return items.reduce(function (promise, item) {
        return promise.then(function () {
            return fn(item, context);
        });
    }, Promise.resolve());
}

あなたの約束の機能は次のようになります。

function logItem(item, context) {
    return new Promise((resolve, reject) => {
        process.nextTick(() => {
            console.log(item);
            context.itemCount++;
            resolve();
        })
    });
}
35
Steven Spungin

私は同じ状況を経験しました。 2つのPromise.All（）を使用して解決しました。

私は本当に良い解決策だったと思うので、私はnpmでそれを公開しました： https://www.npmjs.com/package/promise-foreach

あなたのコードはこのようなものになると思います

var promiseForeach = require('promise-foreach')
var jsonItems = [];
promiseForeach.each(jsonItems,
    [function (jsonItems){
        return new Promise(function(resolve, reject){
            if(jsonItems.type === 'file'){
                jsonItems.getFile().then(function(file){ //or promise.all?
                    resolve(file.getSize())
                })
            }
        })
    }],
    function (result, current) {
        return {
            type: current.type,
            size: jsonItems.result[0]
        }
    },
    function (err, newList) {
        if (err) {
            console.error(err)
            return;
        }
        console.log('new jsonItems : ', newList)
    })
1
saulsluz