简单可行的code review规则

前言

曾经有一段垃圾代码放在我的面前，我没有拒绝，等我真正开始接手的时候我才后悔莫及，程序员最痛苦的事莫过于此！

每当接手别人的代码，都有一种想重新写一遍的感觉，等到别人再来接手你的代码时，同样的感觉。。。为什么会有这种现象存在？因为没有Code Review

衡量代码质量的唯一标准就是每分钟骂出“WTF” 的频率。（出自Martin(Bob大叔)《代码整洁之道》）当你的代码在做 Code Review 时，审查者要是愤怒地吼道：

“What the fuck, is this shit?” “Dude, What the fuck！” 等言辞激烈的词语时，那说明你写的代码是 Bad Code，如果审查者只是漫不经心的吐出几个：

“What the fuck?” 那说明你写的是 Good Code。

价值

你说我的code需要review？

一些没采取过code review模式的资深工程师可能会有一些抵触情绪，但放眼业界，code reveiw的好处是毋庸置疑的。

Google内部人士的一段话。

The biggest thing that makes Google’s code so good is simple: Code Review.

At Google, no code, for any product, for any project, gets checked in until it gets a positive review.

1、保证项目质量、提高代码可读性

2、加速个人成长、突出团队价值

3、知识传播与共享

4、凝聚团队共识

5、不断提高团队的下限

but，良好的程序设计总是优于code review，做好设计才能减少review工作量。

内容

Code Review的内容：编程素养、业务逻辑、架构设计、单元测试、性能、安全

规则

master项目收回提交权限，由QA把控，develop到qa分支merge需要qa review确认。

feature分支往develop分支提pr必须遵循的游戏规则：

1. 每次提交review代码修改行数不得超过1000行，一小时内review完（先试行，最终目标400行，Cisco开发团队研究显示，进行一次Code Review不要超过400行代码（LOC, Line of Code）。大脑一次只能有效处理这么多信息，超过400 行找到缺陷的能力减弱。）

   

2. 至少每个周期（两周一个开发周期）提一次review，周五早上互相喊一下，最晚提交时间周五下午2点前，周五下午的时间用来修改review代码

3. 需要保证单元测试覆盖到，功能点要拆分足够细

4. checkout feature分支后能编译运行正常

5. 工作日可以上午11点前提pr，需要组内指派资深工程师两名， 至少一人review通过

6. 提交pr两天内需要处理完毕，打回或通过

小步迭代，理论上每周自测完成就可以提交pr merge到develop分支

Checklist

常规项

• 代码能够工作么？它有没有实现预期的功能，逻辑是否正确等。

• 所有的代码是否简单易懂？

• 代码符合你所遵循的编程规范么？这通常包括大括号的位置，变量名和函数名，行的长度，缩进，格式和注释。

• 是否存在多余的或是重复的代码？

• 代码是否尽可能的模块化了？

• 是否有可以被替换的全局变量？

• 是否有被注释掉的代码？

• 循环是否设置了长度和正确的终止条件？

• 是否有可以被库函数替代的代码？

• 是否有可以删除的日志或调试代码？

代码行数统计：查看feature分支提交了多少行代码？ (exclude)掉idl目录 （java thrift项目）

git log --author="shenguanpu" --since='2019-04-22' --pretty=tformat: --numstat ":(exclude)idl" | awk '{ add += $1; subs += $2; loc += $1 - $2 } END { printf "added lines: %s, removed lines: %s, total lines: %s\n", add, subs, loc }'

added lines: 17, removed lines: 15, total lines: 2

参考文献：

https://mp.weixin.qq.com/s/A7XlAliQL__RcpK4AzuacQ 从Code Review 谈如何做技术（陈皓）

https://mp.weixin.qq.com/s/jXT1-bC7dfvzhRmh47H69g 如何写出优雅的代码

https://mp.weixin.qq.com/s/n1B0wLicwQByYslz6hYwnw 你真的会Code Review吗？

https://mp.weixin.qq.com/s/YCw7OP3RKF1EJb064IHEXg code review 有用吗？

https://mp.weixin.qq.com/s/nWntcljELd5g98b8RFTjXg code review 那些事儿

https://mp.weixin.qq.com/s/XvSuLKADV17-uF8mZmYXnw 程序员必备的代码审查（Code Review）清单

https://mp.weixin.qq.com/s/6SoJP9BMRNyLMJjwSkctUw 如何高效的Code Review

https://mp.weixin.qq.com/s/sXwuMb8lQzijWTthnaeinw 论code review代码审查

https://mp.weixin.qq.com/s/YOhKriAtUz2VViJtJRhbBw 享用腾讯免费早餐一样自然的做Code Review，他们如何做的？