关于code reiview

先谈谈三个code review的关键因素：

一、创建review要简单

code reivew是一个程序员日常工作中经常做的一件事，理论上来讲，任何一个将要submit到SCM的change，都必须经过peer review。如果创建一个review要傻了吧唧的打包代码，发送邮件，或者shelve一个changelist，再发信告知changelist number，或者进入某个比较先进的code review系统（比如crucible）手工创建一个review，这些步骤都太过繁琐，任何一个懒惰的程序员都不会有耐心来做这种事，更别说日复一日的做这种愚蠢的事了。

我们需要的是一键式创建review - 一个按钮，搞定！比如我以前公司用一个p4插件，直接右键一个pending changelist，就可以创建一个code review(code collaborator)；我现在公司则更加全面，更加彻底，提供了一个命令，可以cover不同的scm，不同的code review系统（支持crucible，gerrit），相当方便、快捷。

二、做review要方便

review的过程，是一个就代码相互交流的过程，为了使这个交流更加高效，我们需要明确知道在讨论的是哪一行代码 - 显然，用邮件是一件相当愚蠢的事情，就像有些人吹嘘用notepad写代码一样。现在市面上这样的系统非常多了：我用过的就有code collaborator, crucible。可以直接就某一行代码开展讨论，并及时邮件通知。

三、被review的change要靠谱

你必须在发出review之前，先review自己的代码 - 编译过了吗，回归测试过了吗? 新feature功能实现了吗？bug真的fix掉了吗？自己啥都不做，写完代码就发review，然后期望别人能够帮你发现问题（把别人当你的编译器，测试工具？）是非常不负责任的做法，尤其是对自己的不负责，久而久之，你在team中名气大臭，终将失去所有人的信任。

再谈谈四个code review的重要作用：

一、威慑

这是我感触最深的一点：知道自己的change会被review，那么在做这个change的过程当中，你就会比较负责任的往代码全局最优的方向去修改，而不是为了临时解决自己当前的问题选择一个比较丑陋的方案 - 因为你知道，即使你选了这个丑陋的方案，然后做了编译测试，发出review之后还是会被人以正当的理由退回来，与其浪费那么些时间做无用功，还被人很没面子的退回来，还不如一开始就做的漂漂亮亮的，久而久之，这种追求最优的习惯就会养成。

二、把关

别人能帮你发现你视觉盲点中，或者视野范围外的一些问题。毕竟，多个人多双眼睛，尤其对于比较senior，或者该领域的专家来说，总能对你有所裨益！

三、学习

别人能从你的代码中学习你优秀的解决问题的思路，写代码的技巧，这是team整体提升的一条非常好的道路。事实上，培养新人的一个方法，除了让他fix bug之外，就是让他review代码，当然，一般是作为副手。

四、通知

让整个team的人知道将有这么个change，我们的做法是创建一个mailgroup：team-cr，把team中每个人都加进来，然后每个review都cc这个组 - 然后对于那些有时间、有需要的同学，就可以监控这个组的邮件。

关于code reiview相关推荐

如何在团队中做好Code Review
一.Code Review的好处想要做好Code Review,必须让参与的工程师充分认识到Code Review的好处 1.互相学习,彼此成就无论是高手云集的架构师团队,还是以CURD为主的业务 ...
Pyinstaller 打包 torch 后执行失败 OSError: could not get source code
1. 问题现象系统环境 Python 3.6.9 torch 1.2.0 torchvision 0.4.0 Pyinstaller 4.5.1 Pyinstaller 打包 torch 后执行失败 ...
VS Code 安装 Go 插件、自定义扩展配置、断点调试
1. 安装插件使用快捷键 Ctrl+Shift+X 打开插件安装页面,安装 Go 插件. 2. 自定义扩展配置使用快捷键 Ctrl+, 打开自定义配置页,编辑 settings.json ,定义与 ...
VS Code 配置调试参数、launch.json 配置文件属性、task.json 变量替换、自动保存并格式化、空格和制表符、函数调用关系、文件搜索和全局搜索、
1. 生成配置参数对于大多数的调试都需要在当前项目目录下创建一个 lanch.json 文件,位置是在当前项目目录下生成一个 .vscode 的隐藏文件夹,在里面放置一些配置内容,比如:settin ...
VS Code 安装插件、自定义模板、自定义配置参数、自定义主题、配置参数说明、常用的扩展插件
1. 下载和官网教程下载地址:https://code.visualstudio.com/ 官方教程:https://code.visualstudio.com/docs 2. 安装插件安装扩展插 ...
VS Code 离线安装插件方法
本文以离线安装 C/C++ 插件为例进说明,其它语言的插件的离线安装方法类似. 离线安装 C/C++ 插件相对比较麻烦一些,主要是因为 C/C++ 插件还依赖其他需要在线下载的组件: C/C++ la ...
离线安装Visual Studio Code插件
在使用Visual Studio Code 开发时候,有时可能会碰到需要离线安装插件的情况.这时候就需要单独下载插件包,本文就以C/C++插件包为例说明如何离线安装Visual Studio Code ...
解决 win10 pycurl安装出错 Command python setup.py egg_info failed with error code 10 编译安装包安装万金油...
版权声明:本文为博主原创文章,遵循 CC 4.0 BY-SA 版权协议,转载请附上原文出处链接和本声明. 本文链接:https://blog.csdn.net/yexiaohhjk/article/d ...
pycharm debug后会出现 step over /step into/step into my code /force step into /step out 分别表示...
1.debug,全部打印 2.打断点debug,出现单步调试等按钮,只运行断点前 3.setup over 调试一行代码 4.setup out 运行断点后面所有代码 5.debug窗口显示调试按钮 ...

关于code reiview

关于code reiview相关推荐

最新文章

热门文章