Planteo una solución similar a la de @Fabio, pero con un solo loop.
En vez de recorrer el array para sacar los índices de los elementos que no sean de tipo number
y después volver a recorrerlo para eliminar los elementos de esos índices (además del coste del reverse
del array de índices); podemos iterar una vez, haciendo la comprobación y el borrado del elemento en la misma iteración y controlando el índice.
const filterFn = <R>(arr: (number | R)[]): void => {
const isNumber = (element: number | R): element is number => {
return typeof element === 'number';
}
for (let i = 0; i < arr.length; ++i) {
const element = arr[i];
if (!isNumber(element)) {
// Eliminamos el elemento
arr.splice(i, 1);
// Decrementamos el index para
// hacer referencia al siguiente
// elemento del array
--i;
}
}
}
La sutil diferencia es la expresión --i
.
Cuando eliminamos un elemento, debemos decrementar el índice para apuntar al siguiente elemento del array. Sin decrementarlo, estaríamos "saltándonos" un elemento de la lista.
Con un caso práctico, supongamos que arr
es [1, 'a', 'b', 15]
; cuando i
vale 1
, isNumber('a')
devolverá false
, por lo que eliminaremos ese elemento. Ahora, arr
es [1, 'b', 15]
. En la siguiente iteración, i
valdrá 2
, por tanto, arr[i]
será el tercer elemento, 15
. Nos hemos "saltado" el elemento 'b'
.
Después de invocar a slice
, todos los elementos después de i
, hacen un "shift" a la izquierda. Por eso i
debe ajustarse.
Disclaimer
La firma de la función filterFn<>
puede cambiar, sólo le he querido otorgar una firma más explícita por el bien de la explicación, pero function unaFunc(arr: any[])
también valdría.
Espero que sirva.
arr
y no lo que te devuelve la funciónunaFunc(arr)
arr
lo que te devuelve la función,arr = unaFunc(arr)