nodejs async control flow with loop

Question

В чем проблема с этим кодом NodeJS?

У меня есть следующий 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 
    } 
}); 

Проблема представляет результаты массив толькопрофилей для первого ProfileID. как

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

, но я ожидал массив Differnet профилей.

Что мне не хватает?

Answer 1

Здесь вы смешиваете синхронный и асинхронный код. Ваш цикл forEach работает синхронно, но метод Profile.findOne является асинхронным. Затем он вызывает обратный вызов, переданный в начальную функцию. Вы должны посмотреть на использование async for an asynchronous for loop.

Однако в вашем вопросе есть много вещей, которые предполагают, что вы еще не полностью поняли асинхронный характер Node.js. Попробуйте прочитать эту тему, например, callback hell.

Answer 2

Использование async или обещает

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

    userlist = results; 
})