How to find and edit property of object in the array of an objects?

Question

I have an array of objects, and I want to find property 'plane: true' in some objects and set it to false. And it just adds to array on the same area as an objects. I tried to use function of angular forEach() but nothing happened. Help please.

this.typeTransport = [
      {
        currentTransport: 'plane',
        changeTransport: 'plane_disable',
        plane: true
      },
      {
        currentTransport: 'train',
        changeTransport: 'train_disable',
        train: true
      },
      {
        currentTransport: 'bus',
        changeTransport: 'bus_disable',
        bus: true
      },
      {
        currentTransport: 'ship',
        changeTransport: 'ship_disable',
        ship: true
      }
    ];

angular.forEach(this.typeTransport, function(value, key){
          angular.forEach(value, function(value, key){
            if(key === 'plane'){
    this.typeTransport[key] == false ? this.typeTransport[key] = true : this.typeTransport[key] = false;
              console.log(this.typeTransport);
            }
          });
      });

<script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.2.23/angular.min.js"></script>

goeldeepika3 · Accepted Answer · 2015-12-09 15:40:40Z

1

The problem is that typeTransport is an array and 'plane' is not a property of it, the object inside has the property 'plane'

rename inner forEach to value1,key1

 angular.forEach(this.typeTransport, function(value, key) {
 angular.forEach(value, function(value1, key1) {

 if (key1 === 'plane') {
      this.typeTransport[key][key1] = false;
    }
  });
 });

As per your condition, you are setting false if true, for which above condition works. If you needed to make true=>false & false=>true, then below condition

 this.typeTransport[key][key1] = !this.typeTransport[key][key1];

answered Dec 9, 2015 at 15:40

goeldeepika3

914 bronze badges

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

Comments

Nina Scholz · Accepted Answer · 2015-12-09 15:37:15Z

1

A solution in plain Javascript with Array.prototype.forEach()

The forEach() method executes a provided function once per array element.

var typeTransport = [{ currentTransport: 'plane', changeTransport: 'plane_disable', plane: true }, { currentTransport: 'train', changeTransport: 'train_disable', train: true }, { currentTransport: 'bus', changeTransport: 'bus_disable', bus: true }, { currentTransport: 'ship', changeTransport: 'ship_disable', ship: true }];

typeTransport.forEach(function (a) {
    if (a.plane) {
        a.plane = false;
    }
})

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

edited Dec 9, 2015 at 15:37

answered Dec 9, 2015 at 15:31

Nina Scholz

388k26 gold badges367 silver badges417 bronze badges

1 Comment

Arg0n Over a year ago

This would probably be a good solution if OP would like to change train: true to false instead of plane: true :)

parameciostudio · Accepted Answer · 2015-12-09 15:46:04Z

1

You have two forEach loops.

In the second loop you have to refer to each single item instead the array

angular.forEach(this.typeTransport, function(item){
          angular.forEach(item, function(value, key){
            if(key === 'plane'){
    item[key] == false ? item[key] = true : item[key] = false;
             console.log(item); 
             console.log(this.typeTransport); 
            }
          });
      });

answered Dec 9, 2015 at 15:46

parameciostudio

5905 silver badges13 bronze badges

Comments

VahagnNikoghosian · Accepted Answer · 2015-12-09 15:31:23Z

0

I'm not sure, that understand your question but this should work.

this.tranportType.forEach(function(tt){
     if(tt.hasOwnProperty('plane') && tt.plane){
        tt.plane = false; 
     }
})

answered Dec 9, 2015 at 15:31

VahagnNikoghosian

6136 silver badges18 bronze badges

Comments

Arg0n · Accepted Answer · 2015-12-09 15:37:53Z

0

You can still use angular.forEach, just change it a little bit:

angular.forEach(typeTransport, function(value){
    if(value.plane){
        value.plane = false;
    }
});

This way your model will update without having to call $scope.$apply();

Here's a fiddle.

answered Dec 9, 2015 at 15:37

Arg0n

8,4532 gold badges26 silver badges39 bronze badges

Comments

Brett Wolfington · Accepted Answer · 2015-12-09 16:39:53Z

0

Just to add some 'old school' :)

for (var i = 0; i < this.typeTransport.length; i++) {
  if (this.typeTransport[i].plane == true) {
    this.typeTransport[i].plane = false;
  }
}

edited Dec 9, 2015 at 16:39

Brett Wolfington

6,6374 gold badges35 silver badges52 bronze badges

answered Dec 9, 2015 at 15:36

Chris Hermut

1,7564 gold badges17 silver badges32 bronze badges

Collectives™ on Stack Overflow

How to find and edit property of object in the array of an objects?

6 Answers 6

Comments

1 Comment

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

1 Comment

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related