let names = ['Rachel', '', 'Meghana', '', '', 'Tim'] function deleteBlankItems(items) { for (let i = 0; i < items.length; i++) { if (items[i].length === 0) { items.splice(i, 1); } } return items; }Creo que este código debería eliminar todos los espacios vacíos en la matriz. Pero, por alguna razón, no elimina este segundo espacio de matriz vacío, por lo que la matriz final es ['Rachel', 'Meghana', '', 'Tim'] ¿Por qué?
En cada ciclo del bucle for, se modifica la longitud original de la matriz. Con la función splice() se modifica la longitud del array y se recalcula el índice de cada elemento. Es decir, después de que se elimine la primera cadena vacía, todas las claves de los elementos siguientes se reorganizarán, y así sucesivamente cada vez que elimine un elemento.
Por ejemplo: In let names = ['Rachel', '', 'Meghana']; la segunda cadena vacía tiene el índice 1 y 'Megana' tiene el índice 2. Cuando se elimina la cadena vacía, la nueva matriz se convierte en ['Rachel', 'Meghana']; , donde la cadena 'Meghana' toma el índice 1.
Entonces, cuando en el bucle for, se elimina un elemento de la matriz, los índices de los siguientes elementos se reducen en 1 mientras que la variable del iterador i se aumenta en uno ( i++ ).
En su ejemplo, en la primera iteración, i = 0, 'Rachel' está en el índice 0 y no se elimina.
En la segunda iteración i = 1, '' (cadena vacía) está en el índice 1 y se elimina; ahora 'Meghana' está en el índice 1, '' está en el índice 2, etc.
En la tercera iteración i = 2, '' está en el índice 2 (por eso se omite 'Meghana') y se elimina; ahora el siguiente '' está en el índice 2.
Y así sucesivamente con las otras iteraciones.
Espero que lo que sucede en el ciclo for sea más claro.
Usar el filter (como se sugiere en otras respuestas) es seguramente la mejor práctica ahora, ya que su implementación no altera la matriz original sino que devuelve una nueva.
De todos modos, solo por el bien de la información, también podría disminuir la variable i en 1 cuando se elimine el elemento.
Posible ejemplo:
let names = ['Rachel', '', 'Meghana', '', '', 'Tim']; function deleteBlankItems(items) { for (let i = 0; i < items.length; i++) { if (items[i].length === 0) { items.splice(i, 1); i -= 1; } } return items; }Ya que está modificando la matriz. Para este caso, solo puede usar la función de filtro.
items.splice(i, 1); modificará el tamaño de la matriz real.
let names = ['Rachel', '', 'Meghana', '', '', 'Tim'] function compact(items) { return items.filter((item) => Boolean(item)); } console.log(compact(names)); // short version const compact2 = (items) => items.filter(Boolean); console.log(compact2(names));Como se indicó en los otros comentarios, esto ocurre porque está modificando la matriz actual en su lugar y continúa iterando sobre la matriz modificada. Para solucionar esto, puede utilizar el método de filtro:
var names = ['Rachel', '', 'Meghana', '', '', 'Tim']; console.log("Original Array: ", names); names = names.filter(function (name) { return name.length; }); console.log("New Array: ", names);