Estaba haciendo un programa de resolución de problemas en el que tenía que encontrar todos los pares de una matriz que se suman a un objetivo (programa clásico de dos sumas solo con una pequeña adición). Este fue mi primer acercamiento:
let findPair = (arr, target) => { let res = []; let mappingArr = new Map() let obj = { first: 0, second: 0 } for(let i=0; i<arr.length; i++){ let comp = target - arr[i]; if(mappingArr.has(comp)){ obj.first = arr[mappingArr.get(comp)]; obj.second = arr[i]; res.push(obj) } else { mappingArr.set(arr[i], i) } } return res; } let arr = [8, 7, 2, 5, 3, 1] let target = 10; let sumPair = findPair(arr, target); console.log(sumPair);** Esto debería generar: [{primero: 8, segundo: 2},{primero: 7, segundo: 3}] pero en su lugar imprime: [{primero: 7, segundo: 3},{primero: 7, segundo: 3}] **
Ahora, si hago un pequeño cambio en el bucle for como este:
if(mappingArr.has(comp)){ res.push({first: arr[mappingArr.get(comp)], second: arr[i]}); } else { mappingArr.set(arr[i], i) }al empujar directamente el objeto en lugar de asignarlo primero a un objeto y luego empujar el objeto a una matriz, funciona bien.
También enfrenté un problema similar con mi código de back-end una vez. Esa vez tampoco pude entender por qué estaba surgiendo el problema.
¿Alguna idea de por qué sucede esto?
Gracias.!
En su primera versión, solo tiene un obj :
let obj = { first: 0, second: 0 }Luego, el bucle muta ese mismo objeto y lo empuja a la matriz. Esto significa que la matriz obtiene múltiples referencias al mismo objeto. Incluso cuando el objeto ya está insertado en la matriz, la siguiente iteración muta ese objeto.
Dado que solo existe un objeto de este tipo, se espera que vea el mismo resultado para todas las entradas de la matriz.
La solución más sencilla es definir un nuevo objeto en cada iteración. Así que mueve el let anterior dentro del ciclo, y funcionará. Pero lo que hizo en la segunda versión usa el mismo principio: crea un nuevo objeto cada vez que lo empuja a la matriz.
El problema ocurre porque ha declarado la variable "obj" fuera del bucle for, por lo que su referencia siempre es la misma cuando la modifica en iteraciones.
Declararlo dentro del ciclo y eso resolverá su problema.
for(let i=0; i<arr.length; i++){ let obj = { first: 0, second: 0 } let comp = target - arr[i]; if(mappingArr.has(comp)){ obj.first = arr[mappingArr.get(comp)]; obj.second = arr[i]; res.push(obj) } else { mappingArr.set(arr[i], i) } }