Как правильно сбросить пользовательские события в C#?

Question

У меня есть этот код:

private void loadGENIOFileToolStripMenuItem_Click(object sender, EventArgs e) 
{ 
    OpenFileDialog dlgFile = new OpenFileDialog(); 

    dlgFile.InitialDirectory = Properties.Settings.Default.PreviousPath; 
    dlgFile.Title = "Select GENIO file"; 
    dlgFile.Filter = "GENIO files (*.txt)|*.txt"; 
    dlgFile.FilterIndex = 0; 
    dlgFile.Multiselect = false; 

    if (dlgFile.ShowDialog() == DialogResult.OK) 
    { 
     Properties.Settings.Default.PreviousPath = Path.GetDirectoryName(dlgFile.FileName); 

     DeleteView(); 

     m_oThreadServices.OnLoadingCompleted += (_sender, _e) => 
     { 
      mruMenu.AddFile(dlgFile.FileName); 
      m_sUITInfo.dbDatabase = m_oThreadServices.GetDatabase(); 
      CreateView(); 
     }; 

     m_oThreadServices.SetGenioFilePath(dlgFile.FileName); 
     m_oThreadServices.start(); 
    } 
} 

Но я также пытаюсь реализовать обработчик MRU:

private void OnMruFile(int number, String filename) 
    { 
     if (File.Exists(filename)) 
     { 
      Properties.Settings.Default.PreviousPath = Path.GetDirectoryName(filename); 

      DeleteView(); 

      m_oThreadServices.OnLoadingCompleted += (_sender, _e) => 
      { 
       mruMenu.SetFirstFile(number); 
       m_sUITInfo.dbDatabase = m_oThreadServices.GetDatabase(); 
       CreateView(); 
      }; 

      m_oThreadServices.SetGenioFilePath(filename); 
      m_oThreadServices.start(); 
     } 
     else 
      mruMenu.RemoveFile(number); 
    } 
} 

Моя m_oThreadServices.OnLoadingCompleted строка кода, кажется, требует, чтобы я использую + = и, как следствие, если я первый загружает файл, он добавляет первый обработчик событий. Если я затем перейду к использованию списка MRU, чтобы загрузить другой файл, он заканчивает работу OnLoadingCompleted обработчики.

Я пробовал m_oThreadServices.OnLoadingCompleted =, но это не позволит. Итак, каков правильный способ для меня перехватить обработчик событий и не в конечном итоге вызвать оба набора кода? Неужели я ошибаюсь?

спасибо.

Answer 1

Вы должны убедиться, что обработчики событий не подписаны с источником события после поднятия события.

Для этого вам необходимо немного изменить анонимные обработчики. Например, этот фрагмент кода:

m_oThreadServices.OnLoadingCompleted += (_sender, _e) => 
{ 
    mruMenu.AddFile(dlgFile.FileName); 
    m_sUITInfo.dbDatabase = m_oThreadServices.GetDatabase(); 
    CreateView(); 
}; 

должно быть так:

EventHandler onLoadingCompleted = null; 
onLoadingCompleted = (_sender, _e) => 
{ 
    m_oThreadServices.OnLoadingCompleted -= onLoadingCompleted; 
    mruMenu.AddFile(dlgFile.FileName); 
    m_sUITInfo.dbDatabase = m_oThreadServices.GetDatabase(); 
    CreateView(); 
}; 
m_oThreadServices.OnLoadingCompleted += onLoadingCompleted; 

То же самое для другого.

Линия

EventHandler onLoadingCompleted = null; 

необходим, чтобы избежать с помощью неинициализированной переменной ошибки компиляторя

m_oThreadServices.OnLoadingCompleted -= onLoadingCompleted; 

Answer 2

Существует нет простого способа удаления анонимных или неизвестных событий из обработчика. Тем не менее, вы можете посмотреть этот форум на MSDN: https://social.msdn.microsoft.com/Forums/vstudio/en-US/45071852-3a61-4181-9a25-068a8698b8b6/how-do-i-determine-if-an-event-has-a-handler-already?forum=netfxbcl

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

Возможно, было бы лучше понять, что именно вы хотите достичь. Возможно, есть лучший способ получить конечный результат, который вы ищете, а не переписывать события.

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

С другой стороны, если это код, добавленный вами или в вашей базе кода, вы можете удалить его, если вы сделали правильное исследование, чтобы подтвердить его удаление, а не заставлять приложение прерывать работу в другом месте , Лучший способ сделать это будет иметь код события в именованной функции:

public void MyEventCode(object sender, EventArgs args) 
{ 
    // Do event stuff.. 
} 

Затем вы можете удалить событие по имени:

control.DoMyEvent -= MyEventCode; 

Answer 3

Поэтому в основном + = является синтаксический сахар для вызова Комбинируйте на своем мероприятии. Делегаты хранятся в списке вызовов, а поведение по умолчанию при запуске события - для каждого делегата в списке вызовов, чтобы получить вызов в том порядке, в котором они были добавлены. Вот почему вы не можете просто установить OnLoadingCompleted для одного делегата с помощью знака = - событие хранит список делегатов, а не один.

Вы можете удалить делегат с - = (синтаксический сахар для вызова Remove). Возможно, вы хотите официально объявить предыдущий делегат где-то, а не передавать его как лямбда. Это позволит вам удалить его, когда вы закончите с ним.

Answer 4

Вы можете удалить обработчик, если это именованная функция:

private void OnLoadingComplete_AddFile(_sender, _e) 
{ 
    mruMenu.AddFile(dlgFile.FileName); 
    m_sUITInfo.dbDatabase = m_oThreadServices.GetDatabase(); 
    CreateView(); 
} 

... 

m_oThreadServices.OnLoadingCompleted += OnLoadingComplete_AddFile; 

... 

m_oThreadServices.OnLoadingCompleted -= OnLoadingComplete_AddFile; 

Удаление обработчика, который не был добавлен (или уже удален) является не-оп, так что вы можете просто удалить «другой» обработчик, прежде чем добавить его: это обеспечит наличие не более одного обработчика.