Приостановка событий в C#, игнорирующих исключения, вызванные обработчиками

Question

Одно из моих любимых мобов с поднятием событий на C# заключается в том, что исключение в обработчике событий приведет к поломке моего кода и, возможно, к запрету вызова других обработчиков, если сломанный один из них вызвал первый вызов; В большинстве случаев мой код не волновался, если кто-то другой код, который слушает его события, нарушен.

Я создал метод расширения, который перехватывает исключения:

public static void Raise(this EventHandler eh, object sender, EventArgs e) 
{ 
    if (eh == null) 
    return; 
    try 
    { 
    eh(sender, e); 
    } 
    catch { } 
} 

Хотя это не означает, мой код осуществляет независимо, этот метод не останавливает первый обработчик событий бросает исключение и предотвращение второго и последующих обработчиков, являющихся уведомление об этом событии. Я просматриваю способ итерации через GetInvocationList, чтобы обернуть каждый обработчик отдельных событий в свой собственный try/catch, но это кажется неэффективным, и я не уверен, что это лучший способ сделать это, или даже если я буду.

Кроме того, мне действительно неудобно просто игнорировать исключение здесь (и ни FxCop/Resharper ни на что); реалистично, что должно произойти с исключением в этом случае?

Answer 1

Что делать, если вы сделали что-то подобное?

public static void Raise(this EventHandler eh, object sender, EventArgs e) 
{ 
    if (eh == null) 
     return; 

    foreach(var handler in eh.GetInvocationList().Cast<EventHandler>()) 
    { 
     try 
     { 
      handler(sender, e); 
     } 
     catch { } 
    } 
} 

Answer 2

Вы должны использовать только те исключения, которые вы можете использовать. Например, вы можете получить доступ к файлу и не иметь достаточных привилегий. Если вы знаете, что это может произойти изредка, и все, что должно произойти, - это то, что дело регистрируется, а затем ловушка , только это исключение.

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

Это исключительное обстоятельство, поэтому есть вероятность, что другие обработчики событий не смогут «делать свои вещи», поэтому пусть он распространяется до уровня, где его можно безопасно обрабатывать.

Answer 3

Если обработчик события не обнаружил исключения, я не вижу никакого хорошего способа обработать исключение в классе, бросающем событие. У вас нет контекста, чтобы знать, что делает обработчик. В этом случае безопаснее всего вредить приложению, так как вы не представляете, какие побочные эффекты могут вызвать другие обработчики событий, которые могут дестабилизировать ваше приложение. Урок здесь заключается в том, что обработчики событий должны быть надежными (обрабатывать собственные исключения), поскольку классы, которые запускают события, никогда не должны ловить исключения, брошенные обработчиками.

Answer 4

Простое, но важное правило:

Каждый дефект в коде должны быть летальным, насколько это возможно уже в возможно.

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