Вызывает ли этот код обработки событий утечку памяти?

Question

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

private void Process()
{
    for (; ; )
    {
        // local variable
        RemoteClient remoteClient = new RemoteClient(..);
        // subscription without unsubscription
        remoteClient.BadClient += new EventHandler(remoteClient_BadClient); 
    }

..
}

public class RemoteClient
{
  ...
  public event EventHandler BadClient;
}

Answer 1

Это зависит от того, что еще находится в классе RemoteClient.Если НЕТ объекта dispose, это не утечка памяти.Если есть какие-либо объекты с содержимым IDisposable, вам нужно унаследовать «Disposable» и уничтожить эти объекты.Также не новость удалить обработчик и цикл выхода, я думаю.

Поскольку клиент звучит как веб-сервис, может быть важно взглянуть на асинхронные потоки. .NET: Нужно ли сохранять ссылку на WebClient при асинхронной загрузке?

Также, если все становится более сложным, важно проверить состояние объектов.

Answer 2

Это не утечка памяти, если вы выйдете из цикла for.Каждый remoteClient будет содержать ссылку на remoteClient_BadClient делегат, но сам remoteClient будет применяться для сборки мусора после каждой итерации (если вы не сохраните ссылку на remoteClient где-то еще!).Сбор remoteClient также приведет к удалению ссылки на делегат remoteClient_BadClient, что также позволит собрать его.

Answer 3

Нет, если только вы не в бесконечном цикле.