Tengo este código:
let mix = [1, 2, 3, "E", 4, "l", "z", "e", "r", 5, "o"]; let newMix = mix.map(function (ele, index) { if (typeof ele === "number") { mix.splice(index, 1) } }) console.log(mix);Y la salida es la siguiente:
[2, 'l', 'z', 'e', 'r', 'o'] Mi pregunta es: ¿Por qué no se splice() eliminó el elemento 2?
A medida que empalma, los índices cambian. Creo que la operación de recopilación que desea aquí es filter en lugar de map :
let mix = [1, 2, 3, "E", 4, "l", "z", "e", "r", 5, "o"]; mix = mix.filter(x => typeof x != "number"); console.log(mix);No es una buena idea alterar la matriz durante un map/forEach , porque cuando elimina un elemento en el índice 0, el elemento en el índice 1 se convierte en el índice 0 y el elemento en el índice 2 pasa al índice 1 . Entonces, en su caso, se elimina 1 en el índice 0 , 2 se mueve al índice 0 y la siguiente iteración comienza en el índice 1 , que tiene 3 actualmente. Entonces se quita 3 .
Consulte la especificación estándar de ECMAScript para
Array.prototype.map para tener una mejor idea.
Una mejor manera de hacer esto sería usar filter , aquí hay un enfoque
let mix = [1, 2, 3, "E", 4, "l", "z", "e", "r", 5, "o"]; let newMix = mix.filter(isNaN); // OR you can use typeof: let newMix = mix.filter((element) => typeof element !== "number"); console.log(newMix);Porque, cuando está empalmando la matriz mix , los índices se modifican como resultado.
Puedes usar el filtro en su lugar
let newMix = mix.filter(function(elem){ return typeof elem !== "number" }) console.log(newMix)