MongoDB mongoose underdokument skapade två gånger

ForEach-slingan i ditt försök känner inte igen återuppringningen av findById() asynkroniseringsmetod före nästa iteration. Du måste använda någon av async biblioteksmetoder async.each , async.whilst , eller async.until som är ekvivalenta med en for-loop, och kommer att vänta tills async:s callback anropas innan du går vidare till nästa iteration (med andra ord, en for-loop som ger).

Till exempel:

var platform_docs = [];
async.each(platforms, function(id, callback) {
    Platform.findById(id, function(err, platform) {
        if (platform) 
            platform_docs.push(platform);
        callback(err);
    });
}, function(err) {
   // code to run on completion or err
   console.log(platform_docs);
});

För hela operationen kan du använda async.waterfall() metod som låter varje funktion föra sina resultat vidare till nästa funktion.

Den första funktionen i metoden skapar den nya artikeln.

Den andra funktionen använder async.each() verktygsfunktion för att iterera över plattformslistan, utför en asynkron uppgift för varje id för att uppdatera plattformen med findByIdAndUpdate() , och när de är klara returnerar resultaten av uppdateringsfrågan i en objektvariabel till nästa funktion.

Den sista funktionen kommer att uppdatera den nyskapade artikeln med plattforms-ID från den tidigare pipeline.

Något i stil med följande exempel:

var newArticle = {},
    platforms            = req.body.platforms,
    date                 = req.body.date,
    split                = date.split("/");

newArticle.title         = req.body.title;
newArticle.description   = req.body.description;
newArticle.date          = split[2]+'/'+split[0]+'/'+split[2];
newArticle.link          = req.body.link;
newArticle.body          = req.body.body;
console.log(platforms);

async.waterfall([

    // Create the article
    function(callback) {
        var article = new Article(newArticle);
        article.save(function(err, article){
            if (err) return callback(err);                  
            callback(null, article);
        });
    },

    // Query and update the platforms 
    function(articleData, callback) {
        var platform_ids = [];
        async.each(platforms, function(id, callback) {
            Platform.findByIdAndUpdate(id, 
                { "$push": { "articles": articleData._id } },
                { "new": true },
                function(err, platform) {
                    if (platform) 
                        platform_ids.push(platform._id);
                    callback(err);
                }
            );
        }, function(err) {
            // code to run on completion or err
            if (err) return callback(err);                  
            console.log(platform_ids);
            callback(null, {
                "article": articleData,
                "platform_ids": platform_ids
            });
        });         
    },

    // Update the article
    function(obj, callback) {
        var article = obj.article;
        obj.platform_ids.forEach(function(id){ article.platforms.push(id); });
        article.save(function(err, article){
            if (err) return callback(err);                  
            callback(null, article);
        });
    }   

], function(err, result) { 
/*
    This function gets called after the above tasks 
    have called their "task callbacks"
*/
    if (err) return next(err);
    console.log(result);
    res.redirect('articles/' + result._id);
});