Mi código debería eliminar los valores falsos, pero solo elimina el primer valor falso en una matriz determinada
const arr = [7, "ate", "", false, 9] function bouncer(arr) { for (let i = 0; i < arr.length; i++) { if (!arr[i]) { arr.splice(i, 1) } } return arr; } console.log(bouncer(arr))¿Por qué no usar filter como:
const arr = [false,7, "ate", "", false, 9]; console.log(arr.filter(el => el !== false));Referencia:
No puede iterar hacia arriba Y acortar su matriz de esta manera; splice tiene el efecto secundario de modificar la matriz original.
Considere lo que sucede en el caso [false, false, true]
i = 0 , length = 3 , arr = [(false), false, true] -> arr se convierte en [false, true]i = 1 , length = 2 , arr = [false, (true)] -> arr permanece [false, true](los paréntesis indican el objetivo actual)
Soluciones:
Iterar hacia abajo, no hacia arriba
for (let i = arr.length - 1; i >= 0; --i) if (!arr[i]) arr.splice(i, 1); Use Array.prototype.filter en su lugar (esto también mejorará la legibilidad)
arr = arr.filter((item) => !!item);El enfoque .filter() sugerido por Rossaini o PaulS es definitivamente un mejor camino a seguir. Sin embargo, su enfoque funciona si trabaja en la matriz al revés:
const test=[7, "ate", "", false, 9]; function bouncer(arr) { if (!arr.length) return []; for (let i=arr.length-1; i--;){ if(!arr[i]){ arr.splice(i, 1) } } return arr; } console.log(bouncer(test)); console.log(bouncer([])); // solution with filter: console.log(test.filter(e=>e)); Debe tener en cuenta que .splice() en realidad cambia la matriz de origen en la que está trabajando. Usando un índice creciente i omitirá una entrada cada vez que se elimine un elemento de la matriz. Al recorrer la matriz hacia atrás (desde el final hasta el principio) como se hizo con for (let i=arr.length; i--;) , evitará este problema.