javascript で map関数を使うと、undefineの配列が返ってきます。

Question

map関数を使うと、undefineの配列が返ってきます。原因はなんでしょうか。
変換前の配列
["57b047f22bbe3edd921de394","57b047f22bbe3edd921de394"]

変換後の配列
[ undefined, undefined ]

    function getMemberName (memberId) {
      User.findOne({_id: mongodb.ObjectId(memberId)}, function(err,user){
        return String(user.username);
      });
    }
    var membersId = ["57b047f22bbe3edd921de394","57b047f22bbe3edd921de394"];
    var membersName = membersId.map(getMemberName);
    console.log(membersName);

なぜでしょうか？？

宜しくお願いいたします。

Answer 1

callback関数( function(err,user){)でreturnしても、外側の関数のreturnとはならないからです。
getMemberName はUser.findOneの指示だけ出して終了します。
ここで戻り値は指定していないので、undefinedが返ります。

たとえば以下の様にしたとき、この membersId だと(2件なので)、mapの結果は [1,1] となるはずです。

    function getMemberName (memberId) {
      User.findOne({_id: mongodb.ObjectId(memberId)}, function(err,user){
        return String(user.username);
      });
      return 1; // 追加
    }

Comments

[退会済みユーザー]

ありがとうございます。
では、callback関数のreturn値を取り出したい場合は、どうすればいいでしょう？？

[flied_onion]

callbackを直列にという話だと、callbackで連鎖していったり、Promiseを作ってreduceでという手もありますが１から説明するのは難しいですね。

callback指定しなければそのまま結果とれるんじゃなかったでしたっけ？

return User.findOne({_id: mongodb.ObjectId(memberId)});

あとは monogoなら $in で指定するとか。

var idList = membersId.map(function(x){ return mongodb.ObjectId(x); });
User.find({ _id: { $in : idList }}, function(err, results){
/* returnではなく ここで処理する */
});
または
var idList = membersId.map(function(x){ return mongodb.ObjectId(x); });
var results = User.find({ _id: { $in : idList }});

実行ためしてないのでなにか間違ってるかもしれませんが。

[退会済みユーザー]

callback指定しなかったところ、
[ Query {
_mongooseOptions: {},
mongooseCollection:
NativeCollection {
collection: [Object],
opts: [Object],
name: 'users',
collectionName: 'users',
conn: [Object],以下続く...
のような、大量のデータを取ってきました。

これだと欲しいデータが抽出できないので、下の方法でもやってみます！

[flied_onion]

とれるのは結果じゃなくてcursor(query)というmongodbのオブジェクトでしたね。

ここが参考になると思います。
https://docs.mongodb.com/manual/tutorial/iterate-a-cursor/#read-operations-cursors

1件ずつ取得も toArray() をうまく使えば、getMemberName の中で1件分returnすることができるかもしれません。

カーソルのデフォルトの有効期限は10分で、必要がなくなったら .close() してあげる必要があるようです。