Tengo una matriz como esta:
const inputArray = [ { userId: 1, sum: 30, }, { userId: 1, sum: 20, }, { userId: 2, sum: 50, }, { userId: 2, sum: 80, }, ];Luego escribí una función que suma los valores por clave y obtuve el siguiente resultado:
const output = [ { userId: 1, sum: 50, }, 50, { userId: 2, sum: 130, }, 130, ];¿Cómo se puede corregir este error? Código de función a continuación:
const output = inputArray.reduce((accumulator, currentItem) => { const index = accumulator.findIndex((item) => item.userId === currentItem.userId); if (index < 0) { return [...accumulator, currentItem]; } else { return [...accumulator, (accumulator[index].sum += currentItem.sum)]; } }, []);Sugeriría, en lugar de finding index , ya que puede ser costoso iterar la matriz una y otra vez, simplemente puede crear un objeto con el userId de usuario como clave y la sum como valor.
Después de eso, puede hacer Object.entries(inputObj) para crear una matriz de objetos a partir de esto.
var inputArray = [{ userId: 1,sum: 30},{userId: 1,sum: 20},{userId: 2,sum: 50},{userId: 2,sum: 80}]; var outputObj = inputArray.reduce((finalObj, curObj) => { if (!finalObj[curObj.userId]) finalObj[curObj.userId] = curObj.sum //<--- adding new user if do not exist in the object else finalObj[curObj.userId] += curObj.sum; // <---- if already present, the add the sum return finalObj; }, {}) var output = Object.entries(outputObj).map((obj) => ({ userId: obj[0], //<--- key of object as userId sum: obj[1] //<--- value of object as sum })); console.log(output)Probar
const inputArray = [ { userId: 1, sum: 30, }, { userId: 1, sum: 20, }, { userId: 2, sum: 50, }, { userId: 2, sum: 80, }, ]; const output = inputArray.reduce((accumulator, currentItem) => { const matchedItem = accumulator.find((item) => item.userId === currentItem.userId); if (!matchedItem) { return [...accumulator, currentItem]; } else { matchedItem.sum += currentItem.sum return accumulator } }, []); console.log(output) estaba agregando un nuevo elemento al acumulador incluso cuando el elemento ya existía, solo necesitaba mutarlo. Además, dado que se trata de una matriz de objetos, puede usar find ya que devolverá una referencia al objeto coincidente en sí, lo que le permitirá actualizar el objeto directamente, en lugar de tener que usar findIndex y luego usar el índice para encontrarlo en la matriz. otra vez
Puedes probar esto:
const inputArray = [ { userId: 1, sum: 30, }, { userId: 1, sum: 20, }, { userId: 2, sum: 50, }, { userId: 2, sum: 80, }, ]; const res = Array.from(inputArray.reduce((m, {userId, sum}) => m.set(userId, (m.get(userId) || 0) + sum), new Map), ([userId, sum]) => ({userId, sum}));