CA2000 при возврате одноразового объекта из метода

Question

У меня есть фабричный метод, который строит объекты, которые реализуют IDisposable. В конечном счете это вызывающие, которые управляют временем жизни созданных объектов. Эта конструкция вызывает пучок CA2000 errors. Есть ли что-то принципиально неправильное в моем дизайне, нужно ли ему рефакторинг, или это просто слишком волнует предупреждения о статическом анализе кода?

Завод метод

public static DisposableType BuildTheDisposableType(string param1, int param2) 
{ 
    var theDisposable = new DisposableType(); 

    // Do some work to setup theDisposable 

    return theDisposable 
} 

Вызывающий

using(var dt = FactoryClass.BuildTheDisposableType("data", 4)) 
{ 
    // use dt 
}  

Answer 1

Я бы рекомендовал вам подавить предупреждение CA2000 по каждому индивидуальному заводскому методу или, возможно, по всему классу, который их содержит (но только если это единственная функция этого класса).

Я также рекомендую вам включать в себя обоснование:

[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Reliability", 
    "CA2000:Dispose objects before losing scope", 
    Justification = "This is a factory method. Caller must dispose")] 

Answer 2

Вы получаете ошибку, потому что создатель одноразового объекта не управлять им. Однако в дизайне нет ничего принципиально неправильного. Вы просто полагаетесь на потребителей, чтобы использовать using. Не так сильно отличается от текущих объектов ADO, например.

Answer 3

Другой альтернативой является изменить фабричный метод в метод «конфигурации» и поставить ответственность за создание одноразового использования объекта на клиенте. Пример:

public void SetupDisosableThing(IDisposable foo) 
{ 
foo.Bar = "baz"; 
} 

void Main() 
{ 
    using (var x = new Thing()) 
    { 
    SetupDisposableThing(x); 
    } 
} 

Answer 4

Вы должны хранить его в локальной переменной, и завернуть инициализации в блоке примерки всеобъемлющая Rethrow, отчуждать в случае каких-либо исключений:

public MyDisposable CreateDisposable() 
{ 
    var myDisposable = new MyDisposable(); 
    try 
    { 
     // Additional initialization here which may throw exceptions. 
     ThrowException(); 
    } 
    catch 
    { 
     // If an exception occurred, then this is the last chance to 
     // dispose before the object goes out of scope. 
     myDisposable.Dispose(); 
     throw; 
    } 
    return myDisposable; 
} 

Try, чтобы никогда не оставить одноразовый объект уязвимы для исключения, когда Dispose не будет называться

PS: кто-то уже упоминалось ранее распоряжаться внутри, наконец, - это, очевидно, не так - в пути без исключений вы не хотите позвонить Dispose