Estoy tratando de calcular la suma de cada precio del producto consultado desde la base de datos, usando findById. Donde la identificación es de una matriz de objetos que itero usando forEach loop.
module.exports.checkoutOrder = (reqBody, buyerId, isAdmin)=>{ let sum =0; reqBody.items.forEach(item=>{ Product.findById(item.productId,(error, result)=>{ //console.log(result.price); sum += result.price; }) }) console.log(sum);El problema es que parece que no puedo obtener o establecer ninguna variable dentro del bloque findById. Incluso si pongo return en la variable sum, siempre es el mismo resultado 0. Estoy seguro de que mi consulta está devolviendo datos cuando console.log result.price.
Sé que hay otras formas de sumar datos, solo quiero saber primero por qué no funciona de esta manera.
El segundo argumento de findById (err,result)=>{} es una devolución de llamada, que por naturaleza no se ejecuta inmediatamente. console.log(sum) se ejecuta antes de todas las devoluciones de llamada y, por lo tanto, imprime 0.
Debe esperar hasta que se hayan ejecutado todas las devoluciones de llamada. Una solución rudimentaria sería envolver el archivo console.log console.log(sum) dentro de un setTimeout con un retraso razonable.
Una mejor solución sería usar el método exec para crear objetos Promise. El método Promise.all espera a que se completen todas las promesas.
let promiseList = await reqBody.items.map(async (item,i)=> Product.findById(item.productId).exec() ); Promise.all(promiseList).then((productList)=> { let sum = productList.reduce((acc,product)=> acc + product.price, 0); console.log(sum); });Puede intentar obtener todos los documentos con ID de matriz:
const arrOfIds = reqBody.items.map(item => item.id) Product.find({ '_id': { $in: arrOfIds }}).toArray(function(err, data)...Entonces puedes calcular la suma.