How to remove elements from a Javascript array? [duplicate]

Question

I want to remove some element in my array using splice method with if condition in my iteration. I do the following:

var list = [{type:"product",name:"Product A"},{type:"product",name:"Product B"},{type:"service", name:"Service A"},{type:"service", name:"Service B"}]

list.forEach(function (item, index) {
    if (item.type == 'service'){
        list.splice(index, 1)
    }
}

//result: list = list = [{type:"product",name:"Product A"},{type:"product",name:"Product B"},{type:"service", name:"Service A"}]

//expected: list = [{type:"product",name:"Product A"},{type:"product",name:"Product B"}]

I expect the both element with type "service" will be removed, but only the first element is removed.

Your approach didn't work because when you reached second service, the index: 3 doesn't exist in your array anymore because after your first splice your array length became 3 so last index is 2. — AvcS
– AvcS, Commented Oct 16, 2019 at 7:25

Yosvel Quintero · Accepted Answer · 2019-10-16 07:18:42Z

2

You can use Array.prototype.filter()

Code:

const list = [{type:"product",name:"Product A"},{type:"product",name:"Product B"},{type:"service", name:"Service A"},{type:"service", name:"Service B"}];

const resultList = list.filter(item => item.type !== 'service');

console.log(resultList);

answered Oct 16, 2019 at 7:18

Yosvel Quintero

19.1k5 gold badges39 silver badges47 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Rajesh Dixit Over a year ago

Should you not vote it as dupe instead?

Jamie Vardy Over a year ago

Thanks for quick response. Problem solved.

Mamun · Accepted Answer · 2019-10-16 10:13:36Z

0

When using forEach(), If the array is modified during iteration, other elements might be skipped

Why don't use Array.prototype.filter():

The filter() method creates a new array with all elements that pass the test implemented by the provided function.

var list = [{type:"product",name:"Product A"},{type:"product",name:"Product B"},{type:"service", name:"Service A"},{type:"service", name:"Service B"}]

list = list.filter( item => item.type != 'service');
console.log(list);

edited Oct 16, 2019 at 10:13

answered Oct 16, 2019 at 7:19

Mamun

69k9 gold badges51 silver badges62 bronze badges

1 Comment

Rajesh Dixit Over a year ago

Should you not vote it as dupe instead?

Collectives™ on Stack Overflow

How to remove elements from a Javascript array? [duplicate]

2 Answers 2

2 Comments

1 Comment

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Linked

Related