从逻辑实现来看，这个作品基本上完成了数据的查询和组装。
这是从公司的商业代码需求来谈，在架构和格式上都有一定的问题

1、把所有的程序都集中在了外部层，包括数据查询。
那么就会出现一个问题，在其他的所有方法里面都会要重新写一遍，怎么查这个数据？耦合性就非常的大
也就是我们框架提供的三层分层是没有用到。这个是最大的一个问题

2、对传进来的参数一定要先进行判断，比如有人调用的时候传的是-1，那就没有必要往下去进行代码逻辑。
对性能有一定的提高

3、一般在对对象进行判空的时候，在web层，我们喜欢反写，就是他等于null返回，就不用写else了