第二次代码评审

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

posted @ 2017-09-20 20:49  跨境电商杂货铺  阅读(160)  评论(0编辑  收藏  举报