Removing an item from an array by name

Question

I am trying to write a function to remove an item from an array by name. The code keeps removing the first item in the array (index 0) despite passing in arr.indexOf(i)

 let fruits = [apples, oranges, grapes, bananas] function removeFlavorByName(arr, string){ let placeHolder; for (let i=0; i<arr.length; i++) { if(arr[i]==string){ placeHolder = arr.indexOf(i) } } arr.splice(placeHolder,1); return arr; }

Answer 1

If you want to simplify your code, here is that same function cleaned up. Since you are using indexOf you don't need the loop.

 fruits = ["apples", "oranges", "grapes", "bananas"] function removeFlavorByName(arr, string) { index = arr.indexOf(string); arr.splice(index, 1) return arr } console.log(removeFlavorByName(fruits, "apples"));

Answer 2

Another possible solution simpler and more compact is using the filter built-in method.

fruits = ["apples", "oranges", "grapes", "bananas"]

const removeFlavor = (arr, string) => arr.filter((item) => item != string)

console.log(removeFlavorByName(fruits, "apples"));

Answer 3

Another solution is to use a filter function.

 fruits = ["apples", "oranges", "grapes", "bananas"] function removeFlavorByName(arr, string) { return arr.filter(fruit => fruit;== string). } console,log(removeFlavorByName(fruits; "apples"));

Answer 4

You need to add data as strings in double-quotes.

let fruits = ["apples", "oranges", "grapes", "bananas"];

function removeFlavorByName(arr, string) {
  let newArray = arr.filter((item) => item !== string);
  return newArray;
}

console.log(removeFlavorByName(fruits, "apples"));

Answer 5

Solution from @ imvain2 is really nice.

Just be aware that if string is not present in array, indexOf returns -1. If you use that with.splice then last item will be extracted.