如何实践 Code Review?

作者/分享人:庄表伟
华为公司内源社区平台架构师 开源社理事

从开源社区开始实践 Code Review 以来,越来越多的企业内的研发团队,也开始逐步采纳 Code Review 的工作模式。但是在具体的实践中,的确存在着不少的困扰与难题。本文打算沿着宣传、引入、实践、改进的四个阶段,深入的探讨企业引入 Code Review 的方方面面。欢迎订阅!

实录提要:

  • 对Pair Programming 怎么看?适合国内的环境吗?
  • 对于 Review 意识薄弱的项目 leader 和工程师该如何引导他们?
  • 如何评估 Review 的有效性?
  • Code Review 工具的使用有没有更详细一点的说明?
  • 对于前端开发的 Code Review 实践有什么好工具和经验分享吗?
  • 若没有引入 Code Review 工具,想轻量地进行 Code Review 操作思路是什么?
  • 还有 Review 一般有哪些检查点?
  • 用类似 Redmine 这样的管理工具做 Code Review 有什么缺点?
已有202人预订
预订达标
文章出炉
交流日期
     
16.12.08
16.12.15
16.12.20 20:30
查看文章评论/提问
于勇
我们公司也是导入了review系统,但是有些项目组review活动流于形式。觉得review是累赘,反正有测试进行测试,也就忽略了review环节。形式大于了效果。另外比较赞同提前用一些自动化工具去检查编码风格方面的问题,这样review的时候聚焦于流程,业务,架构上的问题。代码编程风格应该在工作中逐步形成,比如命名规范,注释,入参保护,判空等。应该鼓励项目组形成自己的检查项,适当的时候推广到组织中相似的项目。
李金茂
是不是可以这样理解:团队成员(新手、老手)、系统、管理层三方如何来互相协调控制,从而达到自洽的稳定状态?
NIU
如何避免因为没时间导致code review不深入或流于形式?
WangYu
请问作者对 pair programming 怎么看?适合国内的环境吗?
袁帅
code review 工具的使用有没有更详细一点的说明?谢谢
于勇: Gerrit 与git 组合起来用 Swarm 与p4组合起来用 Reviewboard 我们公司不同事业部用过这些产品
Adele
对于前端开发的Code Review实践有什么好工具和经验分享吗?和后端CR的区别在哪里?
一代草莽
code Review感觉有些像项目管理中全员参与的PDCA循环呀……是一个软件开发项目持续改进的手段吧?作者怎么理解二者的关系?
于勇
对于review意识薄弱的项目leader和工程师该如何引导他们? 如何评估review的有效性?
大山一竹
能否用一种快速的方式验证处出代码的一般问题,另外是不是必须review的人水平要比被review的人水平高很多呢?
路加
如果公司没有引入Code Review工具,我们想轻量地进行Code Review 操作思路是什么?
yoke
没有量化数据,就很难管理、改进。REVIEW的困局:我如何证明有效性、如何改进、以及和其他手段(单元测试)的比较。在精益化、智能化的今天,有什么手段没?
yoke
没有量化数据,就很难管理、改进。REVIEW的困局:我如何证明有效性、如何改进、以及和其他手段(单元测试)的比较。在精益化、智能化的今天,有什么手段没?
yoke
没有量化数据,就很难管理、改进。REVIEW的困局:我如何证明有效性、如何改进、以及和其他手段(单元测试)的比较。在精益化、智能化的今天,有什么手段没?
王超Charles
做Code Review是否要结合业务逻辑去做,还是仅仅去做编码规范的检查,如果做业务逻辑的Review,那和白盒测试有什么样的不同,是否检查的耗时很长,是否会影响正常的产品发布速度。
MrWin
移动iOS的端的code review有什么好的实践工具及流程?r
alex
作者在实践中有没有具体的统计数据,比如代码从提交到 review 的平均时间,使用 review 前后项目质量的比较。还有 review 一般有哪些检查点。
NIU
review checklist的内容一般是什么?
侯松林
😀
妮可·罗宾
如何开展呢?没有很明白
你可能还喜欢
Service Mesh 在华为公有云的实践
田晓亮
利用 OpenCV 和 Caffe,根据大合影构造“平均脸”
李烨
从零开始,搭建 AI 音箱 Alexa 语音服务
Mike
Web 安全恩仇录:再谈逻辑漏洞
肖志华
TensorFlow 分布式原理与应用实践
刘光聪
编程和数学基础不佳如何入门人工智能?
赵宁|Neal
微信扫描登录