Knoten For-Loop-Rückruf erforderlich

Question

Ich möchte die Tags Format ändern, die ich aus einer der Sammlungen abholen. Stichworte Daten enthalten einig KC ids in einer Anordnung, die ich KC Daten und stecken in TagUnit zu bekommen endgültiges Antwort-Format zu bekommen verwenden.

var newTags = Tags.map(function(TagUnit) { 
    for (var i = 0; i < TagUnit.kcs.length; i++) { 
     KCArray = []; 
     KC.findById(TagUnit.kcs[i], function(error, data) { 
      KCMap = {}; 
      KCMap['kc_id'] = data._id; 
      KCMap['kc_title'] = data.title; 
      KCArray.push(KCMap); 
      if (KCArray.length == TagUnit.kcs.length) { 
       TagUnit.kcs = KCArray; 
      } 
     }); 
    } 
    return TagUnit; 
}); 

response.send(JSON.stringify(newTags)); 

Aber ich bekomme nicht das gewünschte Ergebnis. Antwort gibt Tag Daten in initial für statt formatierter Form aus. Ich denke, es ist aufgrund der Ereignisschleife. Ich wäre dankbar, wenn mir jemand dabei helfen könnte.

** Bearbeiten: ** Ich verwende MongoDB als Datenbank und Mungo als ORM.

Answer 1

Ich würde vorschlagen, Versprechen zu verwenden, um Ihre asynchronen Operationen zu verwalten, die jetzt der Standard in ES6 ist. Sie sagen nicht, welche Datenbank Sie verwenden (möglicherweise verfügt sie bereits über eine auf Versprechen basierende Benutzeroberfläche). Wenn dies nicht der Fall, dann manuell wir KC.findById() promisify:

function findById(key) { 
    return new Promise(function(resolve, reject) { 
     KC.findById(key, function(err, data) { 
      if (err) return reject(err); 
      resolve(data); 
     }); 
    }); 
} 

Dann, vorausgesetzt, Sie tun können, all diese Vorgänge parallel zu finden, können Sie Promise.all() verwenden beide im Auge zu behalten, wenn sie alle fertig sind, und sie zu bestellen für dich.

var allPromises = Tags.map(function(TagUnit) { 
    var promises = TagUnit.kcs.map(function(key) { 
     return findById(key).then(function(data) { 
      // make resolved value be this object 
      return {kc_id: data._id, kc_title: data.title}; 
     }); 
    }); 
    // this returns a promise that resolves with an array of the kc_id and kc_title objects 
    return Promise.all(promises).then(function(results) { 
     return { 
      _id: TagUnit._id, 
      kcs: results 
     }; 
    }); 
}); 

// now see when they are all done 
Promise.all(allPromises).then(function(results) { 
    response.send(JSON.stringify(results)); 
}).catch(function(err) { 
    // send error response here 
}); 

Answer 2

Sie verspricht verwenden können oder Async Modul

var async = require('async'); 
... 
function getKC (kc, callback) { 
    KC.findById(kc, function(err, data) { 
     if (err) 
      return callback(err); 

     callback(null, {kc_id: data._id, kc_title: data.title}) 
    }); 
} 

function getKCs (tag, callback) { 
    async.map(tag.kcs, getKC, callback); 
} 

async.map(Tags, getKCs, function(err, results){ 
    if (err) 
     return console.log(err.message); 

    res.json(results); // or modify and send 
}); 

P. S. Vielleicht enthält der Code Fehler. Ich kann es nicht testen.