The revamp clean old style code and old patterns which are now totally obsoletes for new style code and new pattern that will be obsolete in 2 years.
Don't use iterators anymore
Use the foreach form instead
Don't use singletons anymore
Instead, use picoContainer to create an object that will have only one instance.
For this, you simply need to declare the component in the container
When you need to get the instance, simply put the component as a constructor parameter
Package and module design
If you have a doubt on wether your classes respects this rule, you may use:
- the Dependencies Structure Matrix of Intellij (only available on ultimate version). It is an excellent tool to analyze dependencies and help to delete cycles.
- for Eclipse users, the ispace plugin seems to provide a way to analyze package dependencies.
There were some duplications due to architecture limitations in the past.
Now, you can create a component that may execute the commons work and inject in several classes.