Removing matched object from array of objects using javascript

Question

I have an array of 4 objects and each object contains property array of 8 object.

I am trying to remove an object from properties Array[8]

        var responseArray = new Array();
        responseArray = response.data;
         responseArray.forEach(function (resProp) {
            if (resProp.alias == "General Details") {
                resProp.properties.forEach(function (checkProp) {

                    if (checkProp.alias == "name") {
                        responseArray.pop(checkProp);
                    }
                });
            }

        });

I am able to pop it however the responseArray having only 3 object Array instead of 4.i think, this code is removing the whole 4th object.

responseArray.pop(checkProp);

any suggestions on removing only matched object?

please add the object in text form.

Nina Scholz
– Nina Scholz

2016-04-06 09:15:39 +00:00
Commented Apr 6, 2016 at 9:15 — Nina Scholz
– Nina Scholz, Commented Apr 6, 2016 at 9:15
see docs Array.prototype.pop

Hacketo
– Hacketo

2016-04-06 09:15:56 +00:00
Commented Apr 6, 2016 at 9:15 — Hacketo
– Hacketo, Commented Apr 6, 2016 at 9:15

tchelidze · Accepted Answer · 2016-04-06 09:51:35Z

2

Pop method is not suitable to remove specific object from an array

The pop() method removes the last element from an array and returns that element.

You should remove object from responseArray.properties array instead of responseArray

Replace responseArray.pop(checkProp); with resProp.properties.splice( resProp.properties.indexOf(checkProp) , 1 );

edited Apr 6, 2016 at 9:51

answered Apr 6, 2016 at 9:16

tchelidze

8,3781 gold badge33 silver badges51 bronze badges

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

4 Comments

Harshit Over a year ago

i tried its not working.i think it need to splice from responseArray as its the only which is defined however getting "indexOf is not defined"

tchelidze Over a year ago

@Harshit Oh, my bad, sure you would get that error, indexOf does not have a context. I updated my answer try it now.

Harshit Over a year ago

great.thanks a lot :) its working.just concern on the compatibility of indexOf operator.is there a other way to get the index?

tchelidze Over a year ago

good point, indexOf is supported from IE9 +, if that is still matter , you could iterate thru resProp.properties using for loop instead of forEach and for current element (for which you have index too) pass that index to splice method as a first parameter.

Collectives™ on Stack Overflow

Removing matched object from array of objects using javascript

1 Answer 1

4 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Linked

Related