Как удалить слушатель?

Question

[ParaBellum577]

Есть кнопка "fast up", которая появляется только тогда, когда страница пролистана больше, чем на 500px.
Все работает, но есть неприятный варнинг, который говорит об утечки памяти. Из-за постоянного обновления стейта в componentDidMount .
Как правильно это пофиксить?
Вот код:

isFastUpVisible = () => {
    if (window.pageYOffset > 500) {
      this.setState({fastUpHidden: false})
    } else {
      this.setState({fastUpHidden: true})
    }
  }

  goToTop = () => {
    let timer;
       if(window.pageYOffset > 0) {
         window.scrollTo(0, window.pageYOffset - 400);
         timer = setTimeout(this.goToTop, 20)
       } else {
         clearTimeout(timer);
         window.scrollTo(0,0);
     }
   }

  componentDidMount() {
    window.addEventListener("scroll", this.isFastUpVisible);
   }

Вот варнинг:

Answer 1

[Антон Спирин]

componentWillUnmount() {
  window.removeEventListener('scroll', this.isFastUpVisible);
  clearTimeout(this.timeout);
}

Comments

[ParaBellum577]

Пробовал чуть ранее, варнинг не уходит.

[Антон Спирин]

ParaBellum577, ругается именно из-за отсутствия этой строчки. Если warning не уходит, то ищите компоненты, где допустили подобные ошибки.

[ParaBellum577]

Антон Спирин, я понимаю. Но с componentWillUnmount почему-то ничего не меняется. Это первое, что я попробовал.

[Антон Спирин]

ParaBellum577, ищите подобные ошибки. stack trace указывает на проблемный компонент FirstBlock

[ParaBellum577]

Это единственная ошибка в проекте. Именно в этом блоке с кнопкой. Ладно, буду копаться дальше. Спасибо

[Антон Спирин]

ParaBellum577, возможно, у вас проект не пересобирается и вы запускаете старую версию приложения.

[ParaBellum577]

Антон Спирин, нет, с хотрелоадом все впорядке. На всякий случай перепроверил.

[Антон Спирин]

ParaBellum577, код компонента FirstBlock целиком покажите. И таймер тоже надо чистить в componentWillUnmount, для этого его надо в свойство класса сохранять.