Реализация и использование IDisposable

Question

Я работаю с сторонним объектом, который реализует IDisposable. Для того, чтобы сделать модульный тест «способным», я создаю обертку. Я понимаю, что объект реализует IDisposable, которому моя оболочка также нуждается в реализации IDisposable.

public interface IWrapper : IDisposable 
{ 
    void Complete(); 
} 

public class Wrapper : IWrapper 
{ 
    private readonly ThirdPartyLib lib; 

    public Wrapper() 
    { 
     lib = new ThirdPartyLib(); 
    } 

    public void Complete() 
    { 
     lib.Comlete(); 
    } 

    public void Dispose() 
    { 
     lib.Dispose(); 
    } 
} 

public class Processor : IProcessor 
{ 
    private readonly IWrapper wrapper; 
    public Processor(IWrapper wrapper) 
    { 
     this.wrapper = wrapper; 
    } 

    public void Process() 
    { 
     // do some work 
     using (wrapper) { 
      // do more work 
     } 
    } 
} 

Пусть процессор впрыскивается в некотором классе, который использует его и процесс() выполняется

• Что happends в обертку, если мы называем процесс() еще раз? - не будет ли ThirdPartyLib() выдавать исключение, поскольку он был создан только один раз (в конструкторе оберток), и теперь он был удален
• Не может ли он быть удален до тех пор, пока есть ссылка на него?
• Если обертка возможно будет построить таким образом, чтобы новый() «ИНГ» из ThirdPartyLib быть выполнена не в конструкторе, но в отдельном методе, скажем, Begin() - например, так:

public class Wrapper : IWrapper 
    { 
     private ThirdPartyLib lib; 

     public void Begin() 
     { 
      lib = new ThirdPartyLib(); 
     } 

     public void Complete() 
     { 
      lib.Comlete(); 
     } 

     public void Dispose() 
     { 
      lib.Dispose(); 
     } 
    } 

Затем, используя его:

using (wrapper.Begin()) { 

Answer 1

Вы правы для беспокойства. Вызов Process() дважды, скорее всего, вызовет исключение. Не вызывать Process() вообще не оставил бы объект нераскрытым.

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

Если Processor ссылается на внешнюю принадлежность Wrapper, тогда он не должен располагать эту обертку вообще.

Если Processor принимает на себя ответственность за Wrapper во время строительства, то он должен распоряжаться этой оберткой в ​​распоряжение временем. Это означает, что Processor должен реализовать также IDisposable.

Если Processor может быть изменен для создания нового Wrapper по запросу, то эта обертка может быть удалена, если с ней выполнено Process().

Или, как вы предположили, если Wrapper может быть изменен для создания ThirdPartyLib по запросу, это тоже может сработать. Но будьте осторожны: позвоните wrapper.Begin(); wrapper.Begin();, оставив один ThirdPartyLib нераскрытым и не связанным. Вам нужно будет немного перестроить ваш API, чтобы это не стало проблемой, и, фактически, это означало бы превращение вашего Wrapper в ThirdPartyLibFactory.

Answer 2

Я думаю, что Processor не должен распоряжаться IWrapper, потому что это не тот экземпляр, который его создал, и он не знает, может ли он быть удален или если он вводится в другие объекты. В этом случае Processor не должен использовать using с оберткой.

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