曾经有一段垃圾代码放在我的面前,我没有拒绝,等我真正开始接手的时候我才后悔莫及,程序员最痛苦的事莫过于此!
每当接手别人的代码,都有一种想重新写一遍的感觉,等到别人再来接手你的代码时,同样的感觉。。。为什么会有这种现象存在?因为没有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必须遵循的游戏规则:
小步迭代,理论上每周自测完成就可以提交pr merge到develop分支
常规项
代码行数统计:查看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,他们如何做的?