码迷,mamicode.com
首页 > 其他好文 > 详细

第二次代码评审

时间:2017-09-20 21:51:28      阅读:145      评论:0      收藏:0      [点我收藏+]

标签:pcl   客户端   不能   封装   枚举   接口   二次   head   spring   

1.判断list是否为空,
最好用list.isempty(),不用list.size()
2.枚举是单例的,单例可以直接用==判断相等
3.for循环里面的代码,可以封装成方法
4.update table(a = a+1),不要在代码里相加。对于Oracle
5.for update
6.圈复杂度不能大于15
7.for(i;i<条件;i++),条件不能是表达式,否则会每次循环都计算一次
8.应用日期和数据库日期,选用数据库日期
9.空行,一个空行就可以,没必要两个


接口规范会议:
1.上下文,header头里放入版本号
2.复杂对象返回类型,耦合依赖。
3.面向对象与面向过程的互相转换
4.调用方式,
httpclient,spring rest template,rpc
5.RPC客户端负责抛出异常
6.throwable
7.DTO成对儿出现

第二次代码评审

标签:pcl   客户端   不能   封装   枚举   接口   二次   head   spring   

原文地址:http://www.cnblogs.com/panxuejun/p/7565020.html

(0)
(0)
   
举报
评论 一句话评论(0
登录后才能评论!
© 2014 mamicode.com 版权所有  联系我们:gaon5@hotmail.com
迷上了代码!