首页 > 其他 > 详细

第二次代码评审

时间:2017-09-20 21:26:57      阅读:274      评论:0      收藏:0      [点我收藏+]

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成对儿出现

第二次代码评审

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

(0)
(0)
   
举报
评论 一句话评论(0
关于我们 - 联系我们 - 留言反馈 - 联系我们:wmxa8@hotmail.com
© 2014 bubuko.com 版权所有
打开技术之扣,分享程序人生!