Include if..else... statement in JS forEach method?

Question

While trying to update a dictionary inside the iterator I got the error "SyntaxError: expected expression, got keyword 'if'"

The ut_frequencies data looks like:

{ "_id" : "app1", "cnt" : 3422 }
{ "_id" : "app2", "cnt" : 2752 }
{ "_id" : "app3", "cnt" : 2736 }
{ "_id" : "app4", "cnt" : 2711 }

Which I suppose is a list of dictionaries. I want to check if the app is already in the data dictionary and if so, update the "utf" value, if not, then add a new item only with the "utf" attribute set.

ut_frequencies.forEach(item=> 
  if(item._id in data){
    data[item._id]["utf"] = item.cnt;

  }else{
    data[item._id] = {utf: item.cnt, atf: 0, ptf: 0};
  }
)

Which is the best way to solve this.

You actually dont need the if/else yet.

Jonas Wilms
– Jonas Wilms

2018-02-06 13:48:23 +00:00
Commented Feb 6, 2018 at 13:48 — Jonas Wilms
– Jonas Wilms, Commented Feb 6, 2018 at 13:48

Federico klez Culloca · Accepted Answer · 2018-02-06 13:42:54Z

2

The problem is that on the right of the arrow, an expression is expected.

An if condition is not an expression, therefore you should put braces around the body of the function.

ut_frequencies.forEach(item => {
    if(item._id in data){
        data[item._id]["utf"] = item.cnt;
    }else{
        data[item._id] = {utf: item.cnt, atf: 0, ptf: 0};
    }
});

edited Feb 6, 2018 at 13:42

answered Feb 6, 2018 at 13:41

Federico klez Culloca

27.3k17 gold badges61 silver badges102 bronze badges

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

Collectives™ on Stack Overflow

Include if..else... statement in JS forEach method?

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related