处理代码审查中的推回

有时，开发人员会推迟进行代码审查。他们要么会不同意您的建议，要么会抱怨您总体上过于严格。

谁是对的？

当开发人员不同意您的建议时，请先花点时间考虑一下它们是否正确。通常，它们比您更接近代码，因此他们实际上可能对代码的某些方面有更好的了解。他们的论点有意义吗？从代码健康角度来看，这有意义吗？如果是这样，请让他们知道他们是对的，然后让问题解决。

但是，开发人员并不总是正确的。在这种情况下，审稿人应进一步解释为什么他们认为自己的建议正确。良好的解释不仅说明了对开发人员回复的理解，而且还说明了为什么要求进行更改的其他信息。

尤其是，当审阅者认为他们的建议将改善代码的健康状况时，如果他们认为所导致的代码质量的改善证明所要求的额外工作是合理的，则他们应继续倡导该更改。改善代码运行状况的步骤往往很短。

有时，在真正采纳建议之前，需要花几轮的时间来解释建议。请确保始终保持礼貌，并让开发人员知道您听到他们在说什么，您只是不同意。

烦人的开发人员

审稿人有时认为，如果审稿人坚持要改进，开发人员会感到沮丧。有时，开发人员确实会感到不高兴，但这通常是简短的，后来他们非常感谢您帮助他们提高了代码质量。通常，如果您对您的评论有礼貌，那么开发人员实际上根本不会感到烦恼，而担心只是在审阅者的脑海中。烦恼通常与注释的编写方式有关，而不是与审阅者对代码质量的坚持有关。

稍后清理

回退的一个常见原因是开发人员（可以理解）想要完成任务。他们不想仅仅为了获得此CL而进行另一轮审核。因此，他们说他们将在以后的CL中清理某些内容，因此您现在应该LGTM 此 CL。一些开发人员对此非常擅长，并会立即编写后续的CL来解决此问题。但是，经验表明，开发人员在编写原始CL之后花费的时间越多，清理工作的可能性就越小。实际上，通常除非开发人员立即进行清理在当前的CL之后，它永远不会发生。这不是因为开发人员不负责任，而是因为他们有很多工作要做，并且清理工作在其他工作中被遗忘或遗忘。因此，通常最好是坚持要求开发人员现在在代码进入代码库并“完成”之前清理其CL 。让人们“稍后清理”是代码库退化的一种常见方法。

如果CL引入了新的复杂性，除非是紧急情况，否则必须在提交之前将其清除。如果CL暴露了周围的问题，并且现在无法解决，则开发人员应提交清理错误，并将其分配给自己，以免丢失。他们还可以选择在引用已提交错误的代码中编写TODO注释。

有关严格性的一般投诉

如果您以前对代码的审查松懈，而转而对代码进行严格的审查，那么一些开发人员将大声抱怨。提高代码审查的速度通常会使这些抱怨消失。

有时，这些抱怨可能要花费数月的时间才能消失，但是最终，开发人员往往会看到严格的代码审查的价值，因为他们会看到自己帮助生成的出色代码。有时，一旦发生某种事情，使严格的抗议者甚至成为您最坚强的支持者，这会使他们通过严格遵守而真正看到您所增加的价值。

解决冲突

如果您遵循上述所有内容，但是仍然遇到无法解决的开发人员与您之间的冲突，请参阅《标准代码审查》以获取有助于解决冲突的准则和原则。

参考

https://google.github.io/eng-practices/review/reviewer/pushback.html

Google Code Review 处理代码审查中的推回相关推荐

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 ...
Eclipse Code Review（代码审查）工具介绍
http://soartju.iteye.com/blog/564690 最近组内一直在做代码改进,既然是代码级别的改进,我想比较直接有效的办法就是Code Review. 虽然以前开发部也做过代码评 ...
Code Review 是软件开发中必不可少的一步
前言我一直认为Code Review(代码审查)是软件开发中的最佳实践之一,可以有效提高整体代码质量,及时发现代码中可能存在的问题.包括像Google.微软这些公司,Code Review都是基本要 ...
如何在团队中做好Code Review
一.Code Review的好处想要做好Code Review,必须让参与的工程师充分认识到Code Review的好处 1.互相学习,彼此成就无论是高手云集的架构师团队,还是以CURD为主的业务 ...
谈谈对代码审查Code Review的简单认识
谈谈对代码审查Code Review的简单认识 Code Review(代码审查)是软件开发中的最佳实践之一,可以有效提高整体代码质量,及时发现代码中可能存在的问题.包括像Google.微软这些公司, ...
你太菜了，竟然不知道Code Review...
点击上方"方志朋",选择"设为星标" 回复"666"获取新整理的面试资料作者:宝玉来源:http://1t.click/aA4h 我 ...
从零开始 Code Review，两年实战经验分享！
点击上方"方志朋",选择"置顶公众号" 技术文章第一时间送达! 来源:http://t.cn/RtHE14S 前几天看了<Code Review 程序员的 ...
作为CTO，我为什么必须要求代码进行Code Review！
来源:宝玉链接:https://cnblogs.com/dotey/p/11216430.html 我一直认为Code Review(代码审查)是软件开发中的最佳实践之一,可以有效提高整体代码质量, ...
Code Review最佳实践
我一直认为Code Review(代码审查)是软件开发中的最佳实践之一,可以有效提高整体代码质量,及时发现代码中可能存在的问题.包括像Google.微软这些公司,Code Review都是基本要求,代 ...

Google Code Review 处理代码审查中的推回