We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
1 parent fac930f commit 56afbcaCopy full SHA for 56afbca
codingstyle/codingstyle.rst
@@ -681,6 +681,7 @@ Readme Driven Development:
681
Code Review(代码复查)
682
--------------------------------------
683
笔者认为code review是一件非常重要的事情,可以有效防止代码腐化,同时方便同事了解业务(可以说编码规范、静态分析、代码审查和单元测试是保证代码质量的几个重要工具,没有使用这几个工具之一将来代码都可能难以维护)。可以在公司搭建Phabricator(facebook在用)gitlab 类似工具进行代码review。可惜小公司流程不严格,codereview总是坚持不下去,要不就是被同事吐槽总是给他挑刺。实际上如果是新手能够从code review当中快速学到很多东西,比如编程惯用法,摆脱不良编码习惯,不良设计和难以维护的代码等。review的时候对事不对人,代码如果有明显缺陷快速记录个TODO等待review后修正,以一种开放和学习的心态看待review,慢慢整个团队的实力和代码质量就会提高。review就是个互相学习进步的过程,正规的团队都应该严格遵守,而不只是走走流程。
684
+(没有 review 过的代码可能很快就会成为一坨shit)
685
686
- 建立 review 检查表,防止不合理、过于复杂、明显缺陷、可读性差的代码。眼睛足够多,bug 无处藏。越早修复缺陷,成本越低。
687
- 建立提交模板,每个提交是需求、bugfix还是啥一目了然,同时贴上需求、jira 等地址,方便追溯。
0 commit comments