nodejs 비동기 제어 흐름 (루프 포함)

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'} ] 

좋아하지만 난 전혀 다른 프로파일의 배열을 기대하고있다.

무엇이 누락 되었습니까?

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; 
})