nodejs async Kontrollfluss mit Schleife

Question

Was ist das Problem mit diesem NodeJS Code?

Ich habe das folgende NodeJS-Snipt.

Profile.findOne(profileId, cb) //is sync function 


function getProfiles(users, cb) { 
    var results = []; 
    var n = users.length; 
    users.forEach(function(user, i) { 
    Profile.findOne(user.profileId, function(err, prf) { 
     if (err) { 
     return cb(err, null); 
     } 
     console.log(prf); 
     console.log(user.profileId); 
     results.push(prf); 
     if (i + 1 == n) { 
     console.log('looping done'); 
     return cb(null, results); 
     } 
    }); 
    }); 
} 

// some where 
var userslist = [{ 
    name: 'ab', 
    profileId: 'daf242' 
}, { 
    name: 'cd', 
    profileId: 'hg535h' 
}, { 
    name: 'ef', 
    profileId: 'cvxv445' 
}]; 
getProfiles(userslist, function(err, data) { 
    if (err) { 
    //do this 
    } else { 
    //do that 
    } 
}); 

Das Problem ist, die Ergebnisse sind Array von nurProfilen für das erste das Profil. wie

[ 
     {username:'ab',avatarUrl:'abcd.png'} 
     {username:'ab',avatarUrl:'abcd.png'}, 
     {username:'ab',avatarUrl:'abcd.png'} 
    ] 

aber ich erwarte Array von differnet Profile.

was fehlt mir?

Answer 1

Sie mischen synchronen und asynchronen Code hier. Ihre forEach-Schleife wird synchron ausgeführt, aber die Methode Profile.findOne ist asynchron. Es ruft dann den Rückruf auf, der an die Anfangsfunktion übergeben wurde. Sie sollten mit async for an asynchronous for loop suchen.

Es gibt jedoch viele Dinge in Ihrer Frage, die darauf hinweisen, dass Sie die asynchrone Natur von Node.js noch nicht vollständig erfasst haben. Versuchen Sie, das Thema zu lesen, z. B. callback hell.

Answer 2

Verwenden async oder verspricht

var async = require('async'); 
... 
async.map(users, Profile.findOne, function(err, results) { 
    if (err) 
     return ...// process errors; 

    userlist = results; 
})