Re: [Dpp-devel] "should not use PicoContainer directly"?

[Michael J. <ju...@ze...>]

On 21.12.2010 17:04, Andreas Haferburg wrote:
> Hi Michael
>
> On 12/21/2010 3:31 PM, Michael Jurke wrote:
>> I will have a look to ChildContainer then, although I do not get the
>> point considering the above mentioned situation. I will have time in
>> January only after delivering the work.
> There's no need to do that. Once my patch is in the trunk, you can
> simply use the new Saros#reinject (injectDependenciesOnly) method, which
> uses a ChildContainer. All you need to do is give me a +1 and wait. ;)
No, you mean I can use reinjectOld ;), maybe you did not get the point. 
The whiteboard manager is really supposed to BE managed by pico 
container - I was told by Philip that using the old-fashioned ways for 
singletons like I did makes it difficult to test. Thus I would need a 
protected static method, addToPicocontainer that I can call at StartUp 
to really add something to picocontainer where it stays, not a child 
container to be removed. ;) If so, let it be a child container, although 
I do not get Christopher's point - maybe he wants Saros instead to offer 
a method to get a child container, that the whiteboard maintains 
separately - this way I would understand him at the moment and that is 
what I have to check in January because I'm not sure about benefits and 
problems.

Just to understand, I wrote you only because of:

"I like it. Just always use a dummy container (not sure about the runtime
overhead) and don't offer a method for the old way"

I have never had a look to reinjectOld() and just noticed that it really adds the objects to picocontainer. So I could have misused it on StartUp and my views in the latter state could get the one whiteboard manager singleton injected, too, like Saros uses it.

Best regards,
Micha










> Cheers
> Andreas
>
> ------------------------------------------------------------------------------
> Forrester recently released a report on the Return on Investment (ROI) of
> Google Apps. They found a 300% ROI, 38%-56% cost savings, and break-even
> within 7 months.  Over 3 million businesses have gone Google with Google Apps:
> an online email calendar, and document program that's accessible from your
> browser. Read the Forrester report: http://p.sf.net/sfu/googleapps-sfnew
> _______________________________________________
> Dpp-devel mailing list
> Dpp...@li...
> https://lists.sourceforge.net/lists/listinfo/dpp-devel