forEach（）内のコールバック

Question

1つの画像をアップロードする場合、uploadResultsには1つのオブジェクトしかありません。

複数の画像をアップロードする場合は、アップロードしません。イメージオブジェクトはdb内で正しく作成されますが、アルバムオブジェクトには繰り返しイメージがあります。たとえば、image1とimage2をアップロードした場合、album.Imagesは[image1、image2]になりますが、代わりに[image1、image2、image2]になります。問題はforeach内のコールバックのタイミングだと思うが、問題を正確に特定できない。

uploadResults.forEach(function(uploadedItem) { 
    var image = new Images({ 
     imageUrl: uploadedItem.url, 
    });  
    image.save(function (err, doc) {  
     album.Images.push(doc.id); 
     album.save(function(err, doc) { 
      //err handling... 
     }); 
    }); 
}); 

Answer 1

私は最後の反復にalbum.save（）を実行することによって動作するようにそれを得ています。

uploadResults.forEach(function(uploadedItem, idx, array) { 
    var image = new Images({ 
     imageUrl: uploadedItem.url, 
    });  
    image.save(function (err, doc) {  
     album.Images.push(doc.id); 
     if (idx === array.length - 1) { 
      album.save(function(err, doc) { 
       //err handling... 
      }); 
     } 
    }); 
}); 

Answer 2

なぜ各繰り返しでアルバムを保存していますか？ Promiseを使用してください（多分allのpolyfillが必要です）。 es6-promiseを使用します。

const Promise = require('es6-promise').Promise; 

let promises = []; 
uploadResults.forEach(uploadedItem => { 
    let promise = new Promise((resolve, reject) => { 
     let image = new Images({ 
      imageUrl: uploadedItem.url, 
     }); 
     image.save((err, doc) => { 
      album.Images.push(doc.id); 
      // No saving here 
     }); 
    }); 
    promises.push(promise); 
}); 

Promise 
    .all(promises) 
    .then(() => { 
     album.save((error, doc) => { 
      // rest of code 
     }); 
    });