Обратные вызовы внутри forEach()

Question

При загрузке одного изображения, то есть uploadResults имеет только один объект - нижеследующие работы отлично.

При загрузке нескольких изображений это не так. Объект изображения создается правильно в db, но объект альбома имеет повторяющиеся изображения. Например, если я загружу image1 и image2, album.Images должен быть просто [image1, image2], но вместо этого он будет [image1, image2, image2]. Я считаю, что вопрос о сроках обратных вызовов внутри Еогеасп, но не совсем точно вопрос ..

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). Используйте 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 
     }); 
    });