ForEach aktualisiert Hauptarray nicht

Question

Ich versuche, jedem Element eines Arrays einige zusätzliche Werte hinzuzufügen. Also habe ich ein Array mit Objekten und sie haben: x, y und z Felder. Ich möchte dann zu jedem Objekt im Array zusätzliche Elemente basierend auf der Antwort eines http.get-Aufrufs hinzufügen.

Haupt Array ist: Beiträge

Siehe Code unten:

router.get('/api/posts', function(req, res){ 

    postModel.find({}) 
     .limit(10) 
     .exec(function(err, posts) { 
      var options = { 
       host: 'localhost', 
       port: 3000, 
       path: '/user?id=12345678', 
       method: 'GET' 
      }; 
      if(posts){ 
       posts.forEach(function(post) { 

        var req = http.get(options, function(res) { 
         var bodyChunks = []; 
         res.on('data', function(chunk) { 
          bodyChunks.push(chunk); 
         }).on('end', function() { 
          var body = Buffer.concat(bodyChunks); 
          var parsedBody = JSON.parse(body); 
          post.fullname = parsedBody.user.fullname; 
          post.profilePic = parsedBody.user.profilePic; 
         }); 
        });  
       }); 
      res.json({ 
       posts  : posts 
      }); 
      } else { 
       res.send('Post does not exist'); 
      } 
     }); 
}); 

Zum Zeitpunkt des post.profilePic = parsedBody.user.profilePic - die profilePic Variable ist aber wenn ich eine Antwort vom Knoten über res.json bekomme, sind die zusätzlichen Werte nicht.

Was fehlt mir hier? Ich benutze diesen Ansatz mit meinem Angular Frontend die ganze Zeit ohne ein Problem.

Dank

Answer 1

sein sollten Dies ist ein unglaublich weit verbreitetes Problem, Sie behandeln asynchronen Code, als ob es synchron waren. http.get wird nicht sofort abgeschlossen, noch wird der Code daran gehindert, weiterzumachen, daher wird res.json aufgerufen, bevor Ihre Anforderungen abgeschlossen sind. Es gibt eine Menge Möglichkeiten, dies zu beheben, werde ich meinen Favoriten - Javascript Promises.

// use map instead of forEach to transform your array 
// of posts into an array of promises 
var postPromises = posts.map(function(post) { 
    return new Promise(function(resolve) { 
    var req = http.get(options, function(res) { 
     var bodyChunks = []; 
     res.on('data', function(chunk) { 
      bodyChunks.push(chunk); 
     }).on('end', function() { 
      var body = Buffer.concat(bodyChunks); 
      var parsedBody = JSON.parse(body); 
      post.fullname = parsedBody.user.fullname; 
      post.profilePic = parsedBody.user.profilePic; 
      // resolve the promise with the updated post 
      resolve(post); 
     }); 
    }); 
    }); 
}); 

// once all requests complete, send the data 
Promise.all(postPromises).then(function(posts) { 
    res.json({ 
    posts: posts 
    }); 
}); 

Answer 2

Knoten arbeiten nach Rückrufe. Sie Callbacks, die nicht in der forEach-Schleife abgeschlossen werden, und Sie geben Antwort auf Benutzer. Das ist das Problem.

Schreiben Sie Code für das kann ich Lösung vorschlagen.

router.get('/api/posts', function(req, res){ 

    postModel.find({}) 
     .limit(10) 
     .exec(function(err, posts) { 
      var options = { 
       host: 'localhost', 
       port: 3000, 
       path: '/user?id=12345678', 
       method: 'GET' 
      }; 
      if(posts){ 
       var EventEmitter = require('events'); 
       var HttpEvent = new EventEmitter(); 
       let counts = 0; 
       let length = posts.length; 

       posts.forEach(function(post) { 

        var req = http.get(options, function(res) { 
         var bodyChunks = []; 
         res.on('data', function(chunk) { 
          bodyChunks.push(chunk); 
         }).on('end', function() { 
          var body = Buffer.concat(bodyChunks); 
          var parsedBody = JSON.parse(body); 
          posts.fullname = parsedBody.user.fullname; 
          posts.profilePic = parsedBody.user.profilePic; 
          HttpEvent.emit('done'); 
         }); 
        });  
       }); 

       HttpEvent.on('done',()=>{ 
        counts += 1; 
        if(counts == length){ 
         res.json({ 
          posts  : posts 
         }); 
        } 
       }) 

      } else { 
       res.send('Post does not exist'); 
      } 
     }); 
}); 

Noch eine falsche Sache, die Sie tun, ist

post.fullname = parsedBody.user.fullname; 
post.profilePic = parsedBody.user.profilePic; 

Es

posts.fullname = parsedBody.user.fullname; 
posts.profilePic = parsedBody.user.profilePic;