JavaScript Deleting all objects from an array which contain a value

Question

I have an obj, with an array of objects - something like below (I've not pasted it here because its HUGE). I'm trying to loop through the array of objects - deleting those objects from the array which contain a value. I have written the following code... (using lodash)

When looping over the array its randomly missing a few 'Foo's - so not all the Foo objects are deleted... even though they contain the key Foo. It does ignore that which doesn't contain Foo though.

obj = {
      array : [
         {
          key1 : 'Foo',
          Key2 : 'fi'
         },
         {
          key1 : 'Foo',
          Key2 : 'fi',
          Key3 : 'blah'
         },
         {
          key1 : 'Fred',
          Key2 : 'fi'
         },
         {
          key1 : 'Foo',
          Key2 : 'fi'
         }
         ... etc....
      ]
}

  var items = obj.array
  _.forEach(items, function(n, index) {
   var isFoo = _.includes(n, 'Foo');
   console.log(isFoo);
   if (isFoo) {
      items.splice(index, 1);
    }
  });

seems like you want Array.prototype.filter

Grundy
– Grundy

2015-07-03 20:04:58 +00:00
Commented Jul 3, 2015 at 20:04 — Grundy
– Grundy, Commented Jul 3, 2015 at 20:04

Rhumborl · Accepted Answer · 2015-07-03 20:03:14Z

1

I suspect things are getting confused because you are changing the array at the same time as you are looping through it.

_.remove is probably a better option for you here:

var obj = {
      array : [
         {
          key1 : 'Foo',
          Key2 : 'fi'
         },
         {
          key1 : 'Foo',
          Key2 : 'fi',
          Key3 : 'blah'
         },
         {
          key1 : 'Fred',
          Key2 : 'fi'
         },
         {
          key1 : 'Foo',
          Key2 : 'fi'
         }
      ]
};

_.remove(obj.array, function(n, index) {
    return _.includes(n, 'Foo');
});

console.dir(obj.array)

<script src="https://cdnjs.cloudflare.com/ajax/libs/lodash.js/3.10.0/lodash.js"></script>

answered Jul 3, 2015 at 20:03

Rhumborl

16.7k4 gold badges41 silver badges47 bronze badges

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

1 Comment

Rob Over a year ago

Accepting this as the answer, as my original question did include lodash

Rob · Accepted Answer · 2015-07-03 20:06:47Z

1

I ditched lodash, and just did a reverse for-loop (my thinking was that as things were being removed, the index would change... and maybe that was causing the errors?)

var items = obj.array;
var i;
for (i = items.length - 1; i >= 0; i -= 1) {
    if (items[i].type === 'Foo') {
        items.splice(i, 1);
    }
}

answered Jul 3, 2015 at 20:06

Rob

1,6363 gold badges26 silver badges54 bronze badges

1 Comment

Rhumborl Over a year ago

Just as valid, this is how I normally do things using raw arrays in various languages

Collectives™ on Stack Overflow

JavaScript Deleting all objects from an array which contain a value

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related