代码审查人员要从实际项目出发,有能力把握全局,给团队成员予以指导,告诉成员们什么样的代码是合格的代码。
代码审查人员要有能力对代码风格和标准做出决定,并且讨论并说服团队成员按照这个风格和标准执行。
具体的案例可能千差万别,可以具体问题具体分析,一般的参考标准如下:
1. 找一个通用的代码格式化工具,大家一起用一套工具,提交之前对代码进行格式化;
2. 代码的复杂度是不是控制在了最低程度?
3. 命名方式要统一;
4. 代码即注释;
5. 在TODO区域中要使用注释;
6. 代码提交工具和方式要统一,有UI界面工具的尽量避免使用命令行工具;
7. 在主要代码分支上,保持代码历史线性化,要避免代码历史交叉。
网友评论