Code Reviewer 太烦人？如何与之斗智斗勇

九章算法

于 2018-02-13 12:38:24 发布

阅读量851

点赞数

本文链接：https://blog.csdn.net/JiuZhang_ninechapter/article/details/103666852

版权

编译 | April & Emma
专栏 | 九章算法

在 Code Review 时经常会发生以下情况让程序员心烦：

自己代码写的很好，但是 Code Reviewer 鸡蛋里挑骨头，无尽刁难
无意义的争论。比如花一两个小时和并不如自己精通技术的 Code Reviewer 争论是否要使用隐式类型
有些 Code Reviewer 根本不懂技术还居高临下，比如他们会说：“我闭着眼睛写的算法也能比你的高效。”

在这样的环境中，Code Review 渐渐演变得像是政治游戏里的对抗性活动，严重影响工作心情。那么这时我们该怎么做呢？

1.Code Review的初衷是什么？

在给出建议之前，我们先来了解一下 Code Review 的初衷是什么：

发现程序的缺陷并找到更好的解决方案
帮助团队成员统一编程风格，提高编程技能，促进知识共享
激励开发人员提交更优质的代码

一个良好的 Code Review 过程有助于代码质量，有助于开发人员学习。但也不乏会遇到一些恼人的Code Reviewer，在你什么都做完了之后才告诉你，你提交的程序中有些地方让他不爽，而且理由还特别 personal。遇到这样的 Code Reviewer，我们该如何从容应对呢？

2.如何应对Code Reviewer？

（1）审视自己，提高自己

在遇到 Code Reviewer 提出意见时，思考以下两个问题：

他的反馈真的有帮助吗？
自己有没有改进空间？

如果回答是 yes，那么，认真改进自己。

如果 Code Reviewer 在 Review 过程中做了一些让你不舒服的事情，比如：

居高临下或是嘲笑你。建议忍受一下，因为在任何地方工作都会接触到一些令人讨厌的人，你要做的就是提高自己。
吹毛求疵。比如，code reviewer 说你注释第一个单词应该大写开头，但你觉得这种事情无所谓，是 Code Reviewer 在吹毛求疵，没事找事。但是以后你会发现，从整个项目、整个 Repo 乃至整个公司的角度看代码，这个小细节就会成为一个问题。

（2）直截了当地提出你的反馈

如果你写的程序已经很完美没有太大的改进空间了，Code Reviewer 却因为你的代码风格而不断的唠叨；又或者你已经根据之前他们的反馈作出改进了，但他们不断的提出新的问题来“刁难”你，这时该怎么办？

直接点，告诉他们：我可以接受你对我的建议，但是你的表达方式很让人不舒服，希望你一次把问题说清，不要再这样了。

他们对此可能会有两种反应：

他们可能会接受你的反馈，然后改变自己的行为。如果是这样，那么问题就解决了，即使有时候可能过程不是那么的完美。
如果他们不接受你的反馈，甚至说出：“做一个 nice 的人不在我的工作范围内。”那么这些代码审查者大概潜意识下觉得自己是 Gregory House 或 Linus Torvalds，即使他们没有半点成就。

如果发生了第二种情况，那么你就该采取点措施了。

（3）远离烦人的 code reviewer

关于采取什么措施，提供以下几点建议：

找双方都尊重的开发人员来决策
重新从事一个不同的项目，不跟他在一个团队
如果当前项目的不同部分由不同的人审查，可以选择从事当前项目的不同部分
要求换人进行 Code Review

注意，采取措施时，如果没到万不得已的地步，理由最好不要是“和 Code Reviewer 处不来”。你可以说，是因为对新项目感兴趣等。

如果你的管理人员够精明，他们会理解你真正的想法。一般管理人员都会支持你的做法，因为用这种方式解决，是员工为自己的问题提出解决方案，而不再需要让他们想办法为你解决问题。

而当发生你与Code Reviewer争论这种情况时，很多人都会找管理请求解决，这不是一个好主意。因为通常，管理人员也不懂技术，他们会因为要和你讨论一些技术问题而感到头疼。而且，管理人员和 Code Reviewer 处于相互依存的关系，所以当你向他们提出这个问题之后，他们一般只会对你表示同情。所以当年你与 Code Reviewer 争论时最好不要把管理人员加入进来。