Это утечка памяти?

Question

У меня есть класс, показанный ниже. Экземпляр сервера содержит ссылку на экземпляр слушателя. Слушатель содержит ссылку на экземпляр сервера через делегат события. Это помешает GC собирать экземпляр сервера? Если так, как разорвать этот цикл? Должен ли я реализовать IDisposable или переопределить метод Finalize или сделать что-то еще?

public class Server
    {        
        public Listener Listener { get; private set; }        

        public Server(Listener listener)
        {
            Listener = listener;
            Listener.ClientChannelConnected += new EventHandler<ClientChannelConnectedArgs>(listener_ClientChannelConnected);
        }        

        void listener_ClientChannelConnected(object sender, ClientChannelConnectedArgs e)
        {
            ...
        }
}

Answer 1

Странно, я обнаружил, что этот тип кода постоянно просачивается в .net, и должен был быть осторожен с - = обработчиком событий. Это правда, что память освобождается в конце концов , но на практике вы хотите очистить себя раньше, чем это.

Кроме того, никогда никогда не используйте лямбда-выражение для обработки события, поскольку у вас никогда не будет ссылки на этот объект.

Answer 2

Нет. Сборщик мусора .NET достаточно умен, чтобы разрешать циклические ссылки.

Answer 3

Читать эту статью о утечках памяти . Иногда вы должны использовать - =.