Нужно ли с EventEmitter заботиться об освобождении памяти?

Question

[Виталий]

Пример. Есть 2 эмиттера, один содержит другой. Нужно подписать верхний на нижний, чтобы события нижнего передавались в верхний.

class Up extends EventEmitter {
  down = new EventEmitter();
  constructor() {
    down.on('event', () => this.emit);
  }
}

В Angular 2 при использовании subscribe нужно в деструкторе отписываться вызовом unsubscribe.
Есть ли подобная необходимость с EventEmitter? Что будет, если свойству down присваивается новое значение? Что если объект класса Up перезаписывается?

Сразу хочу уточнить, как быть с ограничителями на количество подписываемых событий. Практикуется ли снятие ограничений или такая необходимость означает ошибочную архитектуру?

В общем, нужно устаканить этот момент в голове, чтобы в последствии не оказаться перед непонятными утечками памяти. Спасибо за понимание)

Answer 1

[Владимир Сергеевич]

Может быть я не до конца компетентен в этом вопросе и сам ошибаюсь, но утечки не видно
Утечка памяти может происходить если "долгоживущий" объект ссылается на "короткоживущий"
когда ты пишешь:

down.on('event', () => this.emit);

у down в массиве events добавляется ссылка на метод emit в экземпляре класса Up, если не будет экземпляра класса Up то и не будет ссылок на него из down. Проблема может быть если экземпляр класса Up тоже подпишется на события в down
Если что, то сорри за сумбурность.
Если кто-то прочитает и увидит в этом ответе ошибку, призываю поправить, мне это тоже будет полезно)

Comments

[Виталий]

Не сумбур, всё понятно)
Согласен, при двусторонней подписке объекты замыкают друг друга - это как раз тот случай, о котором идёт речь. И если перезаписать down, а из него генерируются события таймаутом, наш экземпляр Up будет продолжать получать новые события, но потеряет доступ к источнику.
Окей, подождём ещё ответы.

[Aves]

Виталий: там не происходит замыкания. Пример с ошибками, поэтому сложно понять что он подразумевает. Можно предположить что должно быть this.down.on(...) Но если потом сделать instance.down = null то сборщик мусора просто удалит объект EventEmitter бывший его значением, так как к нему ничто не будет иметь доступа.