Google Code Review 浏览评论中的CL
浏览评论中的CL
摘要
现在您知道要查找的内容了,管理分布在多个文件中的审阅的最有效方法是什么?
- 更改有意义吗?它有一个很好的 描述吗?
- 首先看一下变化中最重要的部分。整体设计得好吗?
- 以适当的顺序查看其余的CL。
第一步:全面了解变化
查看CL的说明以及CL的一般功能。这种变化甚至有意义吗?如果最初不应该进行此更改,请立即做出答复,说明为什么不应该进行更改。当您拒绝这样的更改时,最好还是向开发人员建议他们应该做些什么。
例如,您可能会说:“看起来您为此做了一些出色的工作,谢谢!但是,实际上,我们正朝着删除您在此处修改的FooWidget系统的方向发展,因此我们现在不希望对其进行任何新的修改。相反,您可以重构我们的新BarWidget类吗?”
请注意,审阅者不仅拒绝当前的CL并提出其他建议,而且还礼貌地做到了。这种礼貌很重要,因为即使我们不同意,我们也希望表明我们彼此尊重,作为开发人员。
如果获得的多个CL代表您不想进行的更改,则应考虑重新设计团队的开发流程或外部贡献者的已发布流程,以便在编写CL之前进行更多的沟通。最好先告诉人们“不”,然后再做大量的工作,现在必须将它们扔掉或彻底改写。
第二步:检查CL的主要部分
查找属于此CL的“主要”部分的文件。通常,一个文件的逻辑更改数量最多,这是CL的主要部分。首先看这些主要部分。这有助于为CL的所有较小部分提供上下文,并通常加快执行代码审阅的速度。如果CL太大,您无法确定哪些部分是主要零件,请询问开发人员您应该首先看什么,或要求他们 将CL分成多个CL。
如果您发现CL的这一部分存在一些主要的设计问题,则即使您现在没有时间审查CL的其余部分,也应立即发送这些评论。实际上,复查其余的CL可能会浪费时间,因为如果设计问题足够严重,那么正在复查的许多其他代码都将消失,并且无论如何都不会变得很重要。
立即发出这些主要设计评论非常重要有两个主要原因:
- 开发人员通常会邮寄一个CL,然后在等待审阅时立即根据该CL开始新工作。如果您正在审查的CL中存在重大设计问题,那么他们也将不得不重新设计其以后的CL。您希望在他们在有问题的设计之上进行过多额外工作之前就抓住他们。
- 重大的设计变更比小的变更需要更长的时间。开发人员几乎都有截止日期;为了在截止日期之前完成任务,并在代码库中保留高质量的代码,开发人员需要尽快开始CL的所有重大重做。
第三步:按适当的顺序浏览其余的CL
确认CL整体上没有大的设计问题后,请尝试找出逻辑顺序来浏览文件,同时还要确保不要错过对任何文件的审查。通常,在浏览了主要文件之后,按照代码审查工具向您展示它们的顺序浏览每个文件是最简单的。有时在阅读主要代码之前先阅读测试也是有帮助的,因为这样您就可以知道更改应该做什么。
参考
https://google.github.io/eng-practices/review/reviewer/navigate.html
Google Code Review 浏览评论中的CL相关推荐
- google code review系列6 - 处理code review中的pushback(完结篇)
接上篇:google code review系列5 - 如何编写code review评论.本篇是code review的完结篇,pushback可以解释成对code review出来的问题的拖延.推 ...
- Google Code Review最新指南
本文译自Google最新开放的code review指南:How to do a code review 原文地址:https://google.github.io/eng-practices/rev ...
- 如何在团队中做好Code Review
一.Code Review的好处 想要做好Code Review,必须让参与的工程师充分认识到Code Review的好处 1.互相学习,彼此成就 无论是高手云集的架构师团队,还是以CURD为主的业务 ...
- 从零开始 Code Review,两年实战经验分享!
点击上方"方志朋",选择"置顶公众号" 技术文章第一时间送达! 来源:http://t.cn/RtHE14S 前几天看了<Code Review 程序员的 ...
- 我们是怎么做Code Review的
前几天看了<Code Review 程序员的寄望与哀伤>,想到我们团队开展Code Review也有2年了,结果还算比较满意,有些经验应该可以和大家一起分享.探讨. 我们为什么要推行Cod ...
- 四剑客与Code Review的恩怨情仇:“始乱终弃”到“浪子回头”
背景 Code Reivew 的主要目的是通过逐步改善团队所输出的代码以确保整体代码结构的健康整洁与可演进.知微产品前端团队在Code Reivew 的近两年探索中,总结出一套适合小型研发团队的Cod ...
- 刚进美团,就被各种Code Review,真的有必要吗?
点击关注公众号,Java干货及时送达 众所周知,Code Review是开发过程中一个非常重要的环节,但是很多公司或者团队是没有这一环节的,今天笔者结合自己所在团队,浅谈Code Review的价值及 ...
- 刚入职,就被各种 Code Review,真的有必要吗?
点击上方蓝色"方志朋",选择"设为星标" 回复"666"获取独家整理的学习资料! 来源:juejin.cn/post/ 6882333635 ...
- 有必要做 Code Review 吗???
点击上方蓝色"方志朋",选择"设为星标" 回复"666"获取独家整理的学习资料!作者:梨香 链接:https://juejin.im/pos ...
- 从Code Review 谈如何做技术
本文转载自 www.coolshell.cn (这篇文章缘由我的微博,我想多说一些,有些杂乱,想到哪写到哪) 这两天,在微博上表达了一下Code Review的重要性.因为翻看了阿里内部的Review ...
最新文章
- 《结对-贪吃蛇游戏-最终程序》
- .NET: 使用.NET Core CLI开发应用程序
- Python修饰器的函数式编程
- java 铁路的运费与路程_Java语言程序设计2(16)
- Vue常用属性和方法
- 使用for循环遍历文件、使用while循环遍历文件
- php curl post text,php – POST适用于Postman,但不适用于CURL
- OpenWrt/Wifidog本地化解决方案
- python-基站位置查询
- ROCKCHIP UART开发指南
- 经纬度转GeoHash
- Jetson nano使用anaconda 2021-5-15
- ES7 IK中文分词器
- 世界杯开赛在即,告诉你5个用区块链玩转球赛的秘密 | 内附独家资源
- jQuery源码解析(架构与依赖模块)
- springboot热部署该怎么实现?springboot热部署实现方式
- 选择语句switch和循环语句while
- 局域网内QQ代理设置
- 电路与模拟电子技术笔记 绪论
- “刷脸”时代带火3D传感器
热门文章
- keepalived java,keepalived+nginx+tomcat高可用负载均衡
- linux 中文 bterm fbterm 内核,Fbterm (简体中文)
- 大物实验计算弹性模量_什么是材料的杨氏模量?它的定义与计算公式是什么?...
- 解决AJAX中使用UpdatePanel后再用Response.Write();等无法弹出对话框问题 3法
- Windows下搭建GCC + Eclipse + OpenOCD的ARM开发环境
- 科技部再公布网络安全等10项国家重点研发计划
- sqool导出oracle数据
- 牛客--二维数组中的查找
- Coder Essential之编程语言学习知识点纲要
- oracle参数文件与启动过程