(( Nodejs + Mongo DB )) mapとPromiseを用いて配列数だけくりかえしfindOneAndUpdateをし、その戻り値を配列が欲しい。(( 非同期が難しい !! ))

Question

やりたいこと

findOneAndUpdate後の情報を配列[]にjsonで格納された状態で貰いたい。

現状

例えば、['旅行','おでかけ','デート']という配列を
increaseQuantity()の引数に渡しても、
console.logでは[null,null,null]になってしまう。

const updateTaggedQuantity = async() => {
  let promiseArr = [];
  ['旅行','おでかけ','デート'].forEach(tagName => promiseArr.push(increaseQuantity(tagName)));
  Promise.all(promiseArr)
    .then(res  => console.log(res))
    .catch(err => console.log(err))
}


const increaseQuantity = async (tagNames) => {
  console.log('開始');
  const updates = tagNames.map((tagname) => {
    return Tag.findOneAndUpdate({ name: tagname }, { $inc: { taggedQuantity: 1 } }, { new: true });
  });
  await Promise.all(updates).then(function (values) {
    console.log(values); // [ null, null, null ]
  });
  console.log('終了');
  
　// こうしたいのですが... 現状難しい...
　// 1つ1つのJSONはfindOneAndUpdateの返り値であってほしいです
  return JSONの入った配列 

}; 

おそらく

Promiseの挙動がキーな気がしているのですが、自身にはまだ難しく、ご助力いただけると幸いです。よろしくお願いします。

Comments

[shou6]

これコード全部じゃないですよね？
定義されてない変数ありますし、increaseQuantityに引数として渡されたはずの"tagNames"使いませんし。

Answer 1

MongoDBは分からない為、そこの取得処理などは不明です。
それ以外の部分は下記の様な感じで如何でしょうか。

# 呼び出す方

const tagList = ["旅行", "おでかけ", "デート"]

updateTaggedQuantity(tagList)
  .then(res => {
    console.log(res)
  })
  .catch(err => {
    console.log(err)
  })

# 処理する関数

const updateTaggedQuantity = tagList => {
  console.log("開始")

  return new Promise((resolve, reject) => {
    // tagListのタグによって、findOneAndUpdate？処理
    const updates = tagList.map(tag => Tag.findOneAndUpdate({ name: tag }, { $inc: { taggedQuantity: 1 } }, { new: true }))

    // エラーを逃がすならreject()

    console.log("終了")
    resolve(updates)
  })
}