alt text http://img8.imageshack.us/img8/8558/classdiagram.png
Short description: I have a doubt whether it's normal that AbstractCrudDaoImpl
implements both interface and abstract class which are inherited from the same parent (ReadOnlyDao
).
Unless you have some special method defined in AbsractReadOnlyDaoImpl
which is not in the ReadOnlyDao
interface, that specific inheritance is pretty much useless.
Otherwise, looks fine.