Estoy tratando de trabajar en una aplicación de pila completa MERN donde la interfaz envía una llamada api a "/ createDeckOfCards" a mi backend de nodejs. El objetivo es hacer clic en un botón para crear una nueva baraja de cartas y luego devolver la lista de cartas creadas. El parámetro numOfCards también se envía con esta llamada.
Entonces, en mi backend de nodeJS, tengo el punto final "/ createDeckOfCards" donde uso .map() para crear iterativamente cada tarjeta y luego guardar en mongoDB así:
const allCardsArray = [...Array(req.body.numOfCards).keys()] allCardsArray.map(async (i)=>{ const eachCard = new eachCardModel({ eachCardTitle: String(i) }) eachCard.save((err, doc) => { if (err) return res.status(400).json({ errMsg: "Something went wrong" }); else{ CardDeckModel.findOneAndUpdate( {_id: req.cardDeckCreated._id}, {$push:{allCards: doc}}, function(error, success){ if (error){ console.log(error) return res.status(400).json({ errMsg: "Something went wrong" }); } else { console.log("success") } } ) } }); }) console.log("COMPLETED") //DOES NOT EXECUTE LAST! //THIS RETURNS BEFORE THE .map() is done res.status(200).json({ createdCardDeckID: req.cardDeckCreated._id }) })Después de eso, tengo un segundo punto final "/returnAllCardsInDeck" donde paso la ID del cardDeck de la siguiente manera:
CardDeckModel.findOne({_id: req.body.createdCardDeckID}).populate({path: 'allCards', options: { sort: "eachCardTitle" } }).exec((err, cardDeck) => { if (err) return res.status(400).json({ errMsg: "Something went wrong" }); else { res.status(200).json({ CardDeck: cardDeck }) } })El problema es que CardDeck regresa antes de que se complete allCardsArray.map(). Esto sería un problema porque quiero que el usuario vea TODAS las cartas en el mazo una vez que se crea el mazo. Pero debido a que "/returnAllCardsInDeck" se ejecuta antes que "/createDeckOfCards", devuelve un objeto indefinido.
Además, ¿lo estoy haciendo bien? Esp con respecto a la primera parte ("/createDeckOfCards").
intente esto, no puede hacer una llamada asíncrona como esta con el mapa. Hay patrones para resolver este problema. Promise.all es uno de ellos.
const allCardsArray = [...Array(req.body.numOfCards).keys()] await Promise.all( allCardsArray.map((i)=>{ const eachCard = new eachCardModel({ eachCardTitle: String(i) }) return eachCard.save() .then( () => CardDeckModel.findOneAndUpdate({_id: req.cardDeckCreated._id}, {$push:{allCards: doc}}) .then(() => console.log("success")) .catch((error) => console.log(error) || res.status(400).json({ errMsg: "Something went wrong" });) ).catch(() => res.status(400).json({ errMsg: "Something went wrong" })) }) ) console.log("COMPLETED") //DOES NOT EXECUTE LAST! //THIS RETURNS BEFORE THE .map() is done res.status(200).json({ createdCardDeckID: req.cardDeckCreated._id }) })puede usar for of con async/await en lugar de un map como este
const allCardsArray = [...Array(req.body.numOfCards).keys()]; for (let i of allCardsArray) { const eachCard = new eachCardModel({ eachCardTitle: String(i), }); try { let doc = await eachCard.save(); await CardDeckModel.findOneAndUpdate( { _id: req.cardDeckCreated._id }, { $push: { allCards: doc } } ); } catch (error) { return res.status(400).json({ errMsg: "Something went wrong" }); } } console.log("success"); res.status(200).json({ createdCardDeckID: req.cardDeckCreated._id, });