Esto es solo un ejercicio en un curso, esta aplicación selecciona una fruta al azar de la matriz de frutas y luego se supone que la función removeItem() la elimina de la matriz y devuelve la matriz modificada, pero obtengo un resultado extraño, y la función no está funcionando.
El problema se puede ver aquí
function randomFruit(importedArray) { let random = Math.floor(Math.random() * importedArray.length); return importedArray[random]; } function removeItem(importedArray, item) { for (let i = 0; i < importedArray.length; i++) { if (importedArray[i] === item) { importedArray.splice(i, 1); return [...importedArray.slice(0, i), ...importedArray.slice(i + 1)]; } else { return "not found"; } } } function makeFruitArray() { var foods = ["🍒", "🍉", "🍑", "🍐", "🍏"]; return foods; } let fruitArray = makeFruitArray(); let fruitItem = randomFruit(fruitArray); let remaining = removeItem(fruitArray, fruitItem); console.log({fruitArray, fruitItem, remaining});Hay dos problemas en la función removeItem :
Si el aleatorio no es el primer elemento de la matriz, la función devuelve not found . No se ejecutaría para el segundo ciclo en absoluto, ya que su función not found después de la primera iteración.
El método de splice actualiza la matriz original. Mientras pasa fruitArray al método removeItem , se pasa como referencia y al actualizarlo dentro de la función usando splice también se actualizará la matriz real.
La forma más sencilla y segura de eliminar un elemento de una matriz sería:
function removeItem(importedArray, item) { const filteredArray = importedArray.filter((each) => each !== item); if (filteredArray.length === 0) return 'Not Found'; return filteredArray; }Como dijo himayan, el problema era que el empalme ya cambia la matriz.
Aquí está mi solución:
function removeItem(importedArray, item) { for (let i = 0; i < importedArray.length; i++) { if (importedArray[i] === item) { importedArray.splice(i, 1); break; } } return importedArray; }Su función de eliminación de elementos no funciona correctamente. En lugar de escribir bucles y empalmar la matriz para crear una nueva, solo debe usar el método de filter
function removeItem(importedArray, item) { let newArray = importedArray.filter(function (element) { return element !== item; }); return newArray; }