是什么让Google的程序如此优秀
让Google的程序如此优秀的一个最重要的事情看起来是非常的简单:代码审查
在Google,没有程序,任何产品、任何项目的程序代码,可以在没有经过有效的代码审查前提交到代码库里。
从代码审查里能得到什么?
•在代码提交前,用第二群眼睛检查一遍,防止bug混入。这是最常见的理解,也是它最不重要的一点。
•代码审查的最大的功用是纯社会性的。如果你在编程,而且知道将会有同事检查你的代码,你编程态度就完全不一样了。你写出的代码将更加整洁,有更好的注释,更好的程序结构——因为你知道,那个你很在意的人将会查看你的程序。
•还有一个非常重要的好处。代码审查能传播知识。通过代码审查,至少会有两个人熟悉这些程序——作者,以及审查者。审查者并不能像程序的作者一样对程序十分了解——但他会熟悉程序的设计和架构,这是极其重要的。
代码审查最重要的一个原则:代码审查用意是在代码提交前找到其中的问题——你要发现是它的正确。在代码审查中最常犯的错误——几乎每个新手都会犯的错误——是,审查者根据自己的编程习惯来评判别人的代码。
代码审查的第二个易犯的毛病是,人们觉得有压力,感觉非要说点什么才好。如果你总是力图找出一点什么东西来批评,你这样做的结果只会损害自己的威望。你的评论将不再被人重视。
第三是速度。你不能匆匆忙忙的进行一次代码审查——但你也要能迅速的完成。你的同伴在等你。
Review的概念与意义
Review的概念
›在软件开发过程中邀请同行对工作产品进行审查,以图尽早查找出工作产品缺陷,进行质量控制的的一种质量活动。
Review的意义
›降低返工(rework)的成本。
›越早开展对项目越有价值,推荐开展100% review,在代码完成一个类/函数和文档完成一个Story的分析设计时进行一次review。
代码审查九字真言:
1. 看见了If,就想Else。
2. 看见malloc,就去找Free。
3. 函数调用要小心,需要看看返回值。
4. 看到for循环,就找边界值。
5. 看见return要注意,要去前面找资源。
6. 看见数组把神提,问题往往在下标。
7. 不要小看字符串,长度是个大问题。
8. 得到函数不要急,看看变量初始化,各种路径要小心。
9. 赋值函数最危险,变量没有初始化。
九句句真言不孤立,相互结合显神威。
下面总结了在一个简单列表中最容易保持的11项实践方式:
1、一次评审少于200--400行的代码。
2、目标为每小时低于300-500LOC的检查速率。
3、花足够的时间进行正确缓慢的评审,但是不要超过60-90分钟。
4、确定代码开发者在评审开始之前就已经注释了源代码。
5、为代码评审和获取制度建立可定量化的目标,这样您才能改进流程。
6、使用检查列表,因为它可以极大地改进代码开发者和评审者得作品。
7、确认缺陷确实得到修复了。
8、培养良好的代码评审文化氛围,在这样的氛围中搜索缺陷被看做是积极的活动。
9、警惕“老大哥”效应。(作为一个开发员,您可以自动假设“老大哥正看着您呢”是真的,如果评审制度是由评审支持工具自动评价的,更是这样的。)
10、最少评审一部分代码,就是您不能评审全部的代码,以从Ego Effect(自我效能感)中受益。
11、采用轻量级,能用工具支持的代码评审。