MongoDB
 sql >> Base de Dados >  >> NoSQL >> MongoDB

Subdocumentos do MongoDB mongoose criados duas vezes


O loop forEach em sua tentativa não reconhece a conclusão de retorno de chamada do findById() método assíncrono antes da próxima iteração. Você precisa usar qualquer um dos async métodos de biblioteca async.each , async.whilst , ou async.until que são equivalentes a um loop for e aguardarão até que o retorno de chamada do async seja invocado antes de passar para a próxima iteração (em outras palavras, um loop for que renderá).

Por exemplo:
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);
});

Para toda a operação, você pode usar o async.waterfall() método que permite que cada função passe seus resultados para a próxima função.

A primeira função no método cria o novo artigo.

A segunda função usa o async.each() função utilitária para iterar na lista de plataformas, execute uma tarefa assíncrona para cada id para atualizar a plataforma usando findByIdAndUpdate() , e quando tudo estiver pronto, retorne os resultados da consulta de atualização em uma variável de objeto para a próxima função.

A função final atualizará o artigo recém-criado com os IDs de plataforma do pipeline anterior.

Algo como o exemplo a seguir:
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);
});