Skip to content

Commit 56afbca

Browse files
author
wangningning
committed
review
1 parent fac930f commit 56afbca

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

codingstyle/codingstyle.rst

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -681,6 +681,7 @@ Readme Driven Development:
681681
Code Review(代码复查)
682682
--------------------------------------
683683
笔者认为code review是一件非常重要的事情,可以有效防止代码腐化,同时方便同事了解业务(可以说编码规范、静态分析、代码审查和单元测试是保证代码质量的几个重要工具,没有使用这几个工具之一将来代码都可能难以维护)。可以在公司搭建Phabricator(facebook在用)gitlab 类似工具进行代码review。可惜小公司流程不严格,codereview总是坚持不下去,要不就是被同事吐槽总是给他挑刺。实际上如果是新手能够从code review当中快速学到很多东西,比如编程惯用法,摆脱不良编码习惯,不良设计和难以维护的代码等。review的时候对事不对人,代码如果有明显缺陷快速记录个TODO等待review后修正,以一种开放和学习的心态看待review,慢慢整个团队的实力和代码质量就会提高。review就是个互相学习进步的过程,正规的团队都应该严格遵守,而不只是走走流程。
684+
(没有 review 过的代码可能很快就会成为一坨shit)
684685

685686
- 建立 review 检查表,防止不合理、过于复杂、明显缺陷、可读性差的代码。眼睛足够多,bug 无处藏。越早修复缺陷,成本越低。
686687
- 建立提交模板,每个提交是需求、bugfix还是啥一目了然,同时贴上需求、jira 等地址,方便追溯。

0 commit comments

Comments
 (0)