1、diy风格严重，比如随意定义错误码，999代表无权限，898代表信息未找到，一个控制器一个错误映射码 - 不易读
2、对于外来参数格式不做校验，比如int入库MySQL的datetime，对于字符串长度不做校验，比如经常抛出data too long - 不严谨
3、可以用一句MySQL查询语句搞定的事情，偏偏用在循环语句内部增删改查10次 - 效率低
4、变量命名词不达意永远都是$data，foreach循环内部永远是$k，$v， 数组定义直接 $data[‘id’]，从不重新初始化变量，容易造成变量污染 - 不易读容易混淆阅读
5、对于MySQL或者redis等扩展程序操作不加try catch捕获异常，一旦出错就是 500（undefined column、invalid number for datetime）- 错误处理机制不完善
6、内部service接口复用率低，有10个控制器调用同一个service接口，到第11个需要一个小功能，扩展了该接口时需要检查所有调用该接口的地方 - 复用率低
7、状态枚举值随意定义，控制器好几份，前端写死，建议还是common.config或者model的const常量 - 维护困难
8、同样类型的接口返回的数据格式不统一，不规范，个人惯用 { retcode,data,msg } - 不规范不易读