Ловля исключений в Dispose во время исключения

Question

Представьте, что на C# (в .NET 4.5.x) у вас есть блок using вокруг объекта, который реализует IDisposible. Представьте себе, что

1. исключение выбрасывается в использовании блока
2. когда метод Dispose называется, дополнительное исключение

Каков наилучший способ передать два исключения глобальной обработчик ошибок?

Пример кода:

class SampleClass : IDisposable 
{ 
    void doSomething() 
    { 
     // Imagine that this code is actually doing something 
     // Except that it unexpectedly hits an exception 
     throw new Exception("Exception A"); 
    } 

    void Dispose() 
    { 
     // Imagine that this code is doing some cleanup 
     // Except that it is buggy 
     // And throws more exceptions 
     throw new Exception("Exception B"); 
    } 
} 

static void Main() 
{ 
    try 
    { 
     using(var c = new SampleClass()) 
     { 
      c.doSomething(); 
     } 
    } 
    catch (Exception e) 
    { 
     // Code that records and reports exceptions 
     RecordException(e); 
    } 
} 

В приведенном выше примере, я хочу, чтобы все детали как Exception A и Exception B записаны. Каков наилучший способ передать два исключения в RecordException?

This article предлагает расширение SafeUsingBlock. Это установленная передовая практика?

Edit: В этом случае я контролировать и может повторно написать SampleClass и SampleClass.Dispose()

Answer 1

Если у вас есть контроль метода Dispose(), я бы предложил сделать это ошибка-доказательство любой ценой. В общем, вы должны надежно полагаться на IDisposable s на Dispose() безопасно.

Если это класс, который вы не имеете никакого контроля над, ваше предложение о try оберточной using будет работать, хотя, чтобы сделать его более изящным, я бы, вероятно, сделать что-то ближе к:

try 
{ 
    var c = new SampleClass(); 
    c.doSomething(); 
    c.Dispose(); 
} 
catch(Exception e) 
{ 
    RecordException(e); 
} 

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

Кроме того, если SampleClass это один вы не имеете никакого контроля над, рассмотрим окружив его фасада, который также реализует IDisposable и обрабатывает ошибки в методе Dispose(), например,

public class SampleClassWrapper : IDisposable 
{ 
    private readonly SampleClass _target; 
    public SampleClassWrapper(SampleClass target) 
    { 
     _target = target; 
    } 

    public void doSomething() 
    { 
     _target.doSomething(); 
    } 

    public void Dispose() 
    { 
     try 
     { 
      _target.Dispose(); 
     } 
     catch(Exception e) 
     { 
      RecordException(e); 
     } 
    } 
} 

Тогда вы можете смело использовать свой SampleClassWrapper в using блоке, не заботясь о сбоях в Dispose()