I seem to be mentally stuck in a Flyweight pattern dilemma.
First, let's say I have a disposable type DisposableFiddle
and a factory FiddleFactory
:
public interface DisposableFiddle : IDisposable
{
// Implements IDisposable
}
public class FiddleFactory
{
public DisposableFiddle CreateFiddle(SomethingThatDifferentiatesFiddles s)
{
// returns a newly created fiddle.
}
}
Then, in my opinion, it's quite clear to the client of FiddleFactory
that the factory claims no ownership of the created fiddle and that it's the client's responsibility to dispose the fiddle when done with it.
However, let's instead say that I want to share fiddles between clients by using the Flyweight pattern:
public class FiddleFactory
{
private Dictionary<SomethingThatDifferentiatesFiddles, DisposableFiddle> fiddles = new ...;
public DisposableFiddle CreateFiddle(SomethingThatDifferentiatesFiddles s)
{
// returns an existing fiddle if a corresponding s is found,
// or a newly created fiddle, after adding it to the dictionary,
// if no corresponding s is found.
}
}
Then I feel morally obliged to make the factory itself disposable, since it creates the fiddles and keeps references to them during all of their lifetime. But that would cause problems to the clients that assumed they owned the fiddles and should therefore dispose them.
Is the problem actually that I call the factory FiddleFactory
instead of, say, FiddlePool
, and the "creation" method CreateFiddle
instead of GetFiddle
? Like this:
public class FiddlePool : IDisposable
{
private Dictionary<SomethingThatDifferentiatesFiddles, DisposableFiddle> fiddles = new ...;
public DisposableFiddle GetFiddle(SomethingThatDifferentiatesFiddles s)
{
// returns an existing fiddle if a corresponding s is found,
// or a newly created fiddle, after adding it to the dictionary,
// if no corresponding s is found.
}
// Implements IDisposable
}
Then it's clearer to the client that it'll not own the returned fiddle and it's the pool's responsibility to dispose the fiddles.
Or can this only be readily solved documentation-wise?
Is there a way out of the dilemma? Is there even a dilemma? :-)