How to filter array of objects?

Question

I have this array:

$scope.damageEvants =
    [
       {"id":2, "StartDate":21/05/2012, EndDate:null},
       {"id":3, "StartDate":null, EndDate:02/09/2014},
       {"id":4, "StartDate":null, EndDate:null},
       {"id":5, "StartDate":21/05/2012, EndDate:null}
    ];

I want to filter(remove) all objects from array where property startDate and EndDate are null.

The result after filtering:

$scope.damageEvants =
    [
       {"id":2, "StartDate":21/05/2012, EndDate:null},
       {"id":3, "StartDate":null, EndDate:02/09/2014},
       {"id":5, "StartDate":21/05/2012, EndDate:null}
    ];

I tried this:

$scope.damageEvants.filter(return item.StartDate!== null && item.EndDate !== null  )

But it seem to be wrong way.

How can filter $scope.damageEvants using filter function?

Sushanth -- · Accepted Answer · 2016-04-29 19:49:15Z

3

$scope.damageEvants.filter(return item.StartDate!== null && item.EndDate !== null )

supposed to be

$scope.damageEvants.filter(function(item) {
     return item.StartDate!== null && item.EndDate !== null;
});

The filter

edited Apr 29, 2016 at 19:49

answered Apr 29, 2016 at 19:01

Sushanth --

55.8k9 gold badges70 silver badges109 bronze badges

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

Comments

Ravi Tiwari · Accepted Answer · 2016-04-29 19:04:37Z

1

$scope.damageEvants = $scope.damageEvants.filter(function(item) {
     return item.StartDate!== null && item.EndDate !== null;
});

answered Apr 29, 2016 at 19:04

Ravi Tiwari

9729 silver badges18 bronze badges

Comments

Nina Scholz · Accepted Answer · 2016-04-29 19:05:23Z

Just check if either date is set.

var array = [{ "id": 2, "StartDate": '21/05/2012', EndDate: null }, { "id": 3, "StartDate": null, EndDate: '02/09/2014' }, { "id": 4, "StartDate": null, EndDate: null }, { "id": 5, "StartDate": '21/05/2012', EndDate: null }],
    result = array.filter(function (a) {
        return a.StartDate || a.EndDate;
    });

document.write('<pre>' + JSON.stringify(result, 0, 4) + '</pre>');

ES6 version

var array = [{ "id": 2, "StartDate": '21/05/2012', EndDate: null }, { "id": 3, "StartDate": null, EndDate: '02/09/2014' }, { "id": 4, "StartDate": null, EndDate: null }, { "id": 5, "StartDate": '21/05/2012', EndDate: null }],
    result = array.filter(a => a.StartDate || a.EndDate);

document.write('<pre>' + JSON.stringify(result, 0, 4) + '</pre>');

aeb0 · Accepted Answer · 2016-04-29 19:06:42Z

1

As Sushanth's answer indicates, .filter() expects to receive a function value, with that function returning a boolean value that determines if the current array member will be in the filtered array or not. Your code is passing filter a boolean value rather than a function value.

answered Apr 29, 2016 at 19:06

aeb0

8269 silver badges12 bronze badges

Comments

Rob · Accepted Answer · 2016-04-29 19:30:08Z

Another option would be to splice the object in an angular.forEach loop. This will actually remove the object from the original array rather than creating a new array.

Example: https://jsfiddle.net/gh03zur4/

Controller

        function Controller($scope) {
      $scope.damageEvants = [{
        "id": 2,
        "StartDate": 21 / 05 / 2012,
        EndDate: null
      }, {
        "id": 3,
        "StartDate": null,
        EndDate: 02 / 09 / 2014
      }, {
        "id": 4,
        "StartDate": null,
        EndDate: null
      }, {
        "id": 5,
        "StartDate": 21 / 05 / 2012,
        EndDate: null
      }];

      angular.forEach($scope.damageEvants, function(event) {
        if (!event.EndDate && !event.StartDate) {
          var idx = $scope.damageEvants.indexOf(event);
          $scope.damageEvants.splice(idx, 1);
        }
      });
    }

Another option based on Sushanth's answer would be to apply a filter in your ng-repeat list like this:

https://jsfiddle.net/qj7dz2eq/

Controller

$scope.filterEvents = function(item) {
  return item.StartDate !== null || item.EndDate !== null;
}

HTML

<ul>
    <li ng-repeat="item in damageEvants | filter:filterEvents">{{item.id}}</li>
</ul>

Doing it either one of these ways prevents the same array being bound to the $scope twice.

Shaishab Roy · Accepted Answer · 2016-04-29 19:36:52Z

1

Can try like:

$scope.damageEvants = $scope.damageEvants.filter(function(obj) {
    return !(obj.StartDate === obj.EndDate && obj.StartDate === null);
});

answered Apr 29, 2016 at 19:36

Shaishab Roy

16.9k7 gold badges56 silver badges69 bronze badges

Collectives™ on Stack Overflow

How to filter array of objects?

6 Answers 6

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related