软件开发中的同行评审

简介: 对于任何一个软件公司来说,几乎在所有的软件项目中,都需要同行评审。一个人不管能力多强,看问题的角度总会受到限制,写出来的程序和文档等也不会是十全十美的。如果能够让懂行的同事给参阅一下,并提出他们认为正确的意见,那么势必会弥补单个开发人员思维的缺陷,并在一定程度上提高程序和产品的质量。

在《浪潮之巅》这本书中,吴军老师描述了在Google早期的工作方式,其中有一段是这么写的:我一般会在吃完晚饭后把代码修改的清单发给克雷格做代码审核,他一般晚上10点左右会回复我,给我修改意见,详细到某一行多了一个空格。吴军老师所描述的内容,其实就是软件开发过程中的同行评审流程。

对于同行评审,我有相当的体会。之前在某大公司工作的时候,我参与了多个软件版本的维护工作,发现不同版本程序质量差别很大。究竟是什么原因造成的?细究之后才发现,程序质量高的项目组在最终提交版本之前,无一例外都做了一件事情,那就是同行评审。看来,同行评审真的是十分重要。

对于任何一个软件公司来说,几乎在所有的软件项目中,都需要同行评审。一个人不管能力多强,看问题的角度总会受到限制,写出来的程序和文档等也不会是十全十美的。如果能够让懂行的同事给参阅一下,并提出他们认为正确的意见,那么势必会弥补单个开发人员思维的缺陷,并在一定程度上提高程序和产品的质量。

什么是同行评审?
同行评审(Peer Review)是一种结构化的正式评审,由生产者(作者)的同行,为识别异常和需要修改的部分而对工作产品进行的有组织、有计划的检查。

同行评审并不是对个人的工作不信任,其目的是尽早有效地消除软件产品中的异常。

为什么要做同行评审?
在学校的时候,老师教导我们要独立完成作业,因此,如果你在那个时候对别人的作业或在考试过程中进行“同行评审”,就是违规的。但软件开发是一项团队性的工作,需要大家协作来完成,所以,那种不愿意公开自己工作成果接受他人审阅的做法是行不通的。

为什么要做同行评审呢?原因有以下几个:

第一,凡是人皆有出错的时候,要保证你做出的东西错误尽量少,就需要他人对其进行评审。这就是所谓的“三个臭皮匠,顶个诸葛亮。”

第二,对于软件产品来说,缺陷发现得越早,纠正缺陷所需的费用就越少。因此,在软件的开发阶段,如果严格进行同行评审,那么后续流程中出现的错误就会很少,这也可以为公司节约纠错的成本。

第三,同行评审过程中发现的错误可作为案例传承下去,避免开发人员再次掉进同一个陷阱。“前事不忘,后事之师”,别人所犯的错误对自己有警示的作用。特别是对于新员工来说,经常参与同行评审,可减少试错的次数,也能够达到对新工作及早上手的目的。

同行评审的对象是什么?
在产品开发的各个阶段中,需要参与同行评审的对象包括:

概念阶段:产品需求说明书等。
计划阶段:系统方案、项目计划等。

开发阶段:详细设计、单元测试规程(方案)、集成测试规程(方案)、代码、数据库脚本等。一般而言,在开始编码之前,先要进行详细设计评审,以确保程序流程的正确性,减少后续修改带来的不良影响。
验证阶段:系统测试计划、系统测试规程(方案)等。

同行评审有哪几种类型?
同行评审的类型包括:审查、走查、单人复审和多人复审等。

什么是审查?审查的角色都有哪些?审查的步骤有哪些?有效评审的原则有哪些?
审查是同行评审的一种,在软件项目中占有重要地位。

1.审查的定义:审查(Inspection)是一种正规的同行评审,适用于重要性和重复性较高的软件产品。

2.审查的角色:包括主持人、讲解员、记录员、作者和审查员。一般说来,参与审查的有3到7人。各角色的分工如下:

主持人:需要具备领导技巧,负责管理审查过程,是审查流程中的关键角色。

作者:提供审查材料,并提供概要介绍;此外,作者也可以充当评审员。
记录员:按照主持人的示意记录产品和程序等的异常,在记录的同时要对异常进行分类。
讲解员:充分理解审查材料,并在讲解的同时加以解释。该角色不能由作者担当。
审查员:所有参与审查的人员都可以是审查员;主要负责理解被审材料、进行个人检查、做好准备并参加审查会议。

3.审查活动的步骤有7个,如下图所示:
image

4.有效评审的原则包括:

第一,在预审期间要使用检查单,检查单用以记录发现的缺陷。

第二,避免过度依赖检查单。
第三,审查会议要限制在2小时之内,以避免长时间讨论而偏离了审查会议的主题。
第四,审查的对象是产品而非生产者(作者),因此要避免对作者本人进行人身攻击。
第五,“磨刀不误砍柴工”,要给评审员提供足够的预审时间,一般以提前两天为佳。
第六,如果有与会人员未准备好,则将会议延期;如果有人确实抽不出时间,则取消评审。

当然,以上有效评审的原则很难同时满足,各项目组要根据实际的情况来组织评审。

什么是代码评审?其目的是什么?
对于软件开发工程师来说,代码评审是不可缺少的一个环节,也是提高代码质量的一种有效的手段。

1.代码评审的定义:代码评审是一种正式的针对程序发起的一项同行评审活动,它由一组职责明确的主持人、作者、记录员、审查员等人员通过一系列明确定义的过程对代码进行静态检查来发现代码中的异常。

2.代码评审的目的:提早发现代码缺陷、去除缺陷,以降低开发成本并提高产品质量。

如何彻底地执行同行评审?
同行评审流程固然重要,但要彻底地执行下去,需要做到以下几点:

第一,公司要明确规定所有的软件开发项目必须走同行评审的流程,并定期抽查同行评审的执行情况。

第二,作者本人要持有开放心态,愿意分享自己的工作成果,并勇于承认自身的不足而加以改进。很多人都不大乐意被别人指出自身的问题,而同行评审的目的就是要发现问题,因此,这在一定程度上是对作者心态的考验。作者本人要明白是产品有缺陷,要对产品中的问题进行完善。

第三,评审人员要对事不对人,要懂得同行评审是为了发现产品的缺陷,而非人自身的问题。不要将同行评审活动变性为针对作者本人的人身攻击,这就有伤整个团队的凝聚力。同行评审的一个宗旨就是大家在一起学习,取长补短,共同提高。

第四,公司要对同行评审的结果进行总结,并以文档的形式保存起来,方便后期查阅。很多软件产品出现的问题都是相似的。在着手开发产品之前,如果能够了解前期类似产品中出现的问题,那么大家就可以少走很多弯路,工作效率和产品质量也在无形中得到了提高。

总结
不管是成熟团队也好,创业团队也罢,为了确保软件产品的质量,所有的工作成果(包括程序、文档、图形等)都应该接受同行评审。“三人行,则必有我师”,不管是老员工,还是新员工,都能够从同行评审中受益。

目录
相关文章
|
敏捷开发 项目管理
当敏捷遇上PMP:项目管理的完美结合
项目管理领域一直在不断发展,不断涌现出新的方法和工具,以满足不断变化的商业需求。在这个多变的环境中,PMP(项目管理专业人员)认证一直以其强大的项目管理框架而著称,而敏捷方法论则在敏捷开发和快速响应市场需求方面表现出色。本文将深入探讨PMP和敏捷如何相互结合,为项目管理带来新的维度和可能性。
|
4月前
|
监控 程序员 测试技术
多年的项目管理工作总结,分享软件项目经理把控好项目质量的 9 点经验
多年的项目管理工作总结,分享软件项目经理把控好项目质量的 9 点经验
|
4月前
|
测试技术 项目管理
如何做好一个信息系统项目经理,一个项目经理的个人体会和经验总结(四)
如何做好一个信息系统项目经理,一个项目经理的个人体会和经验总结(四)
|
4月前
|
程序员 测试技术
如何做好一个信息系统项目经理,一个项目经理的个人体会和经验总结(三)
如何做好一个信息系统项目经理,一个项目经理的个人体会和经验总结(三)
|
4月前
如何做好一个信息系统项目经理,一个项目经理的个人体会和经验总结(一)
如何做好一个信息系统项目经理,一个项目经理的个人体会和经验总结(一)
|
4月前
|
敏捷开发 前端开发 架构师
如何做好一个信息系统项目经理,一个项目经理的个人体会和经验总结(二)
如何做好一个信息系统项目经理,一个项目经理的个人体会和经验总结(二)
|
6月前
|
敏捷开发 前端开发 架构师
如何做好一个信息系统项目经理,一个项目经理的个人体会和经验总结
该文讲述了信息系统项目经理在项目开发阶段应关注的要点。首先,需组建项目小组,确保团队中包含熟悉客户业务的成员,以便有效沟通。其次,选择稳定的技术栈,避免使用未经充分测试的新版本以降低风险。此外,项目经理应合理分解任务,设定可检查的交付标准,并利用项目管理工具控制进度和时间。通过敏捷开发方法提高效率,同时避免过度加班。建议项目经理充当客户与开发团队间的桥梁,减少现场开发带来的冲突。
134 0
|
项目管理
产研项目管理-实用经验
项目管理方法是一门通用技能。当你正在管理一个项目时,如果没有系统的方法,那么只能事倍功半。项目管理用结构化的方法告诉我们:如何就目标达成共识,如何与相关方协作,如何拆分工作,如何控制项目工期,如何达成项目目标,从而为公司收益做出贡献。
671 0
|
程序员 测试技术 项目管理
项目管理-人月神话
人月神话
192 0