代码Review的内容
编码规范
- 硬编码
- 吃异常
- 无用的代码(import,变量,private方法)
- 冗余的代码(重复调用,重复判断)
- 冗余的服务请求(重复调用,循环调用)
规范
- 接口定义(接口与实现是否一致)
- 领域对象(字段,常量,拼写,XML生成)
- 格式转换(JSON与XML的一致性和完整性)
- 配置文件(完整性,与WIKI文档的一致性)
wiki文档
- API名称
- 入参与出参(名称是否规范,两个单词间统一用下划线)
- 字段名称(是否符合TOP的一惯命名规范,是否有拼写错误)
- Java SDK(SDK与文档是否一致)
业务逻辑
- 需求(与PM/PDM提供的需求否一致)