下载下来看了几分钟,数据层DAO模式用的不错,Service层和Web层联系太紧,应该在参数传递上进行分装,用Map隔离更好一些。
建议有BaseAction和BaseService类,更加易于扩展,并且可以把通用的代码、权限验证等放到Base类中处理
XXXDAOImpl中每个方法都处理HibernateException,这个应该统一解决,不应分散在各个独立的方法中。
Jsp也存在同样问题,缺少统一的include.jsp, header.jsp, footer.jsp等等,设计时多考虑重用性和扩展性会更有利于提高,再接再厉!