まだ動かしてはないですが、ちょっとソースを覗いてみて気になった点です。 bitterpetshop.daoパッケージ内のクラスはS2Containerを直接使っていましたが、ここでもDIを使ったほうが良いと思いました。 あと、DAOが２層になっていて分かりにくいと感じました…