I want to remove a range of elements from an array
:
So I am expecting:
["Banana", "Orange1", "Bananax", "Orangex"]
But the result is:
["Banana", "Orange1", "Orangex"]
Why is this happening?
Are there any faster and better ways of doing this?
The second param of Array.prototype.splice() method is the number of elements to be removed and not an ending index.
You can see from the Array.prototype.splice() MDN Reference that:
Parameters
start
Index at which to start changing the array (with origin 0). If greater than the length of the array, actual starting index will be set to the length of the array. If negative, will begin that many elements from the end of the array (with origin 1) and will be set to 0 if absolute value is greater than the length of the array.
deleteCount
Optional An integer indicating the number of old array elements to remove. If deleteCount is 0, no elements are removed. In this case, you should specify at least one new element. If deleteCount is greater than the number of elements left in the array starting at start, then all of the elements through the end of the array will be deleted.
Solution:
You need to calculate the number of elements between these two indexes, so use b-a+1
to get the correct count.
Demo:
This is how should be your code:
var fruits = ["Banana", "Orange1", "Apple", "Banana", "Orange", "Banana", "Orange", "Mango", "Bananax", "Orangex"];
var a = fruits.indexOf("Apple");
var b = fruits.indexOf("Mango");
var removedFruits = fruits.splice(a, b-a+1);
console.log(fruits);