Cannot call this.displayMaps() in for each loop

Question

I am not sure why I can't call my class method in forEach loop. Is there a way to fix this?

This works:

this.displayMaps()

But, this doesn't work:

this.state.floors.forEach(function(floor) {
  this.displayMaps(floor)
})

Could be. But very hard to tell without looking at the actual code. — mbojko, May 28 '19 at 15:28
`forEach(function(floor) {` the callback function has its own context (`this`). You could use an arrow function or use other context binding tricks. — Emile Bergeron, May 28 '19 at 15:29
your `this` may be screwed up because you are within forEach. How is your `displayMaps` declared? — Rikin, May 28 '19 at 15:30
`forEach` does not return anything. Use an arrow function and `map`. — Ross Allen, May 28 '19 at 15:30
Possible duplicate of [How to access the correct \`this\` inside a callback?](https://stackoverflow.com/questions/20279484/how-to-access-the-correct-this-inside-a-callback) — Emile Bergeron, May 28 '19 at 15:31
Also, _"this doesn't work"_ is not a clear enough description to get help most of the time. Describe the exact problem that you're having and the expected behaviour you'd like to achieve. — Emile Bergeron, May 28 '19 at 15:36

brk · Accepted Answer · 2019-05-28T15:31:31.007

2

The scope of this changes inside the forEach callback function.Instead use arrow function

{
  this.state.floors.forEach((floor) => {
    this.displayMaps()
  })
}

edited May 28 '19 at 15:31

answered May 28 '19 at 15:29

brk

1

you may not need this statement: `this.displayMaps = this.displayMaps.bind(this);` – Rikin May 28 '19 at 15:30

1 Answers1