Remove object from array with a 0 value

Question

I am generating an array with objects from some outside data. Here is my code to do so:

  modifyFatalitiesByCause(data) {
    let array = [];
    for (let x in data) {
      array.push({
        "name": data[x]['HarmfulEvent'],
        "value": parseInt(data[x]['Deaths']),
      })
    }
    return array;
  }

This works fine and will output this data (here's just a small chunk of a big array):

[
    {name: "Injured in Vehicle (Non-Collision)", value: 1},
    {name: "Fire Hydrant", value: 1},
    {name: "Snow Bank", value: 0},
    {name: "Cargo/Equipment", value: 0}
]

I would like to not have the objects with a value of 0 appear in my array. Will I need to remove these at the end or can I modify my for x in data loop to only push the objects that don't have a value of 0?

Which is the easier solution?

array.filter(x => x.value) ? or simply add an if condition before you push to check that data[x]['Deaths'] !== 0 — Nir Alfasi
– Nir Alfasi, Commented Aug 31, 2018 at 16:59

A l w a y s S u n n y · Accepted Answer · 2018-08-31 17:10:52Z

3

Yes you can do a check before pushing element on array like this if(parseInt(data[x]['Deaths']) > 0 and then push non-zero values but Array.prototype.filter() seems cool to me :)

let array = [{
    name: "Injured in Vehicle (Non-Collision)",
    value: 1
  },
  {
    name: "Fire Hydrant",
    value: 1
  },
  {
    name: "Snow Bank",
    value: 0
  },
  {
    name: "Cargo/Equipment",
    value: 0
  }
];
result = array.filter(elm => elm.value > 0);
console.log(result);

edited Aug 31, 2018 at 17:10

answered Aug 31, 2018 at 17:05

A l w a y s S u n n y

38.8k9 gold badges68 silver badges120 bronze badges

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

2 Comments

cup_of Over a year ago

thanks for the answer, but I went with M. Wyatt's solution. gave you an upvote

A l w a y s S u n n y Over a year ago

@cup_of awesome. It's OK you can go with if(parseInt(data[x]['Deaths'])> 0 way no prob. Best of luck :)

M. Wyatt · Accepted Answer · 2018-08-31 17:01:44Z

2

You could remove them all at the end but it would be more efficient to just not push them to your array while you're building it. Something like this:

if (parseInt(data[x]['Deaths']) { // 0 is falsey
  array.push({
    "name": data[x]['HarmfulEvent'],
    "value": parseInt(data[x]['Deaths']),
  });
}

answered Aug 31, 2018 at 17:01

M. Wyatt

1702 silver badges12 bronze badges

1 Comment

M. Wyatt Over a year ago

@cup_of you should accept this answer or the other one by amrender_singh

amrender singh · Accepted Answer · 2018-08-31 17:03:04Z

1

You can simply add a check in your for loop, Add objects in your array, whose data[x]['Deaths']) are non-zero. Try the following:

for (let x in data) {
     if(parseInt(data[x]['Deaths']) != 0){
       array.push({
         "name": data[x]['HarmfulEvent'],
         "value": parseInt(data[x]['Deaths']),
       })
     } 
}

edited Aug 31, 2018 at 17:03

answered Aug 31, 2018 at 17:01

amrender singh

8,2594 gold badges27 silver badges30 bronze badges

Comments

Aravinda Meewalaarachchi · Accepted Answer · 2018-08-31 17:25:12Z

1

It is always best to create a predicated if statement to check the validation before the business logic execute. Here is my solution to solve this problem.

  modifyFatalitiesByCause(data) {
    let array = [];
    for (let record in data) {
     if(isDeathCountZero(record, data)){
       array.push({
         "name": data[record]['HarmfulEvent'],
         "value": parseInt(data[record]['Deaths']),
       });
     } 
   }
    return array;
  }

 isDeathCountZero(record, data){
    return (parseInt(data[record]['Deaths']) != 0);
}

answered Aug 31, 2018 at 17:25

Aravinda Meewalaarachchi

2,6691 gold badge31 silver badges27 bronze badges

Comments

Gian Tomakin · Accepted Answer · 2018-08-31 17:00:02Z

-1

try to use array filter to filter the results. refrence: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/filter

answered Aug 31, 2018 at 17:00

Gian Tomakin

1931 silver badge5 bronze badges

Collectives™ on Stack Overflow

Remove object from array with a 0 value

5 Answers 5

2 Comments

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

2 Comments

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related