Question

I have a class that creates several IDisposable objects, all of these objects are then passed to another ‘manager’ class in a 3rd party library. As I need some of the objects in later calls I kept a local field reference to the created objects so that I could access them at a later time. When I ran FxCop on the class it said that I should implement IDisposable, due to the disposable objects that I kept a reference to. My questions are:

1. Should I implement IDisposable for my class? (or is it the manager’s responsibility?)
2. If so, should I only dispose the objects I kept a reference to? or should I find a way to dispose all objects I created.

My code:

public class MyClass   {       ClassThatIsDisposable aReference;   }  public MyClass(ManagerClass manager)   {       ClassThatIsDisposable transient=new ClassThatIsDisposable();       manager.Add(transient);       aReference=new ClassThatIsDisposable();       manager.Add(aReference);   }  public void LaterCall()   {       areference.Method();   }      

Answer 1

The class that owns the object is the one that should dispose of them… It sounds like your manager is the owner, so he should dispose of the object.

If you’re trying to avoid the FXCop warning then one option is to have MyClass request the disposable object from the manager each time is needs to use it. This way you won’t have a member variable holding a reference to it. You could do this my having the Add() method return akey for the object that has been added and then use this key to retrieve the object from the manager when you need to use it.