tacitknowledge / autopatch

An automated database patching framework for Java.
41 stars 19 forks source link

Add a lightweight IoC container to Autopatch #9

Open MexicanHacker opened 13 years ago

MexicanHacker commented 13 years ago

Currently the lack of a IoC being used in the project prevents us from favoring composition over inheritance, to change this and at the same time favor the payment of the technical debt accumulated in terms of missing tests it would be great to get some help from a IoC container. Right now we are looking at Guice to make this happen but we are not closed to other options (code.google.com/p/google-guice/)

scottfromsf commented 13 years ago

Great point, Oscar. In terms of container selection you already know my preference for Guice since it's 1) extremely quick, 2) very lightweight, and 3) favors constructor over getter/setter injection. That said other containers should such as PicoContainer be considered.

artur-ciocanu commented 13 years ago

I guess PicoContainer could be a better choice, 'cause it doesn't rely on annotation and some other "magic" used in Google Guice, just my 2 cents.

MexicanHacker commented 13 years ago

Artur, PicoContainer also could be considered, this would be something we need to decide in the near future.