# 审查代码有风险吗？

几年前，我是一家大公司的一个小团队的成员。该公司的业务遍及全球。这意味着我们必须与其他国家的其他团队合作。有一天，我在论坛上发现了一段对代码的评论。该页面是由另一位团队成员的开发人员编写的，他代码审查时被某些人冒犯了。

他在代码审查中添加了一个comment。评论评论只有3个字母。WTF。编写代码的人对此评论并不感到很开心，并感到冒犯。我们团队中的开发人员没有看到问题。只是提及，这是每个人都应该知道的琐碎问题。

但这是一个琐碎的问题吗？也许是这样，但是也有可能不是每个有其他背景的人都知道问题出在哪里。在那种情况下，以这种方式编写代码是完全正常的。

# 审查目的

在继续发表自己的看法之前，让我们看一下代码审查的目的是什么，因为没有一个人一劳永逸地解决所有问题。有两个原因：

发现缺陷或改进机会。可以检查代码和数据以发现弱点。 对其他（新）开发人员的教育。确保每个人都看到与缺陷修复或增强相关的修改，以便他们可以理解软件的其余部分。

# 团队道德

引入代码审查后，有些人将其视为破坏团队文化的事情。他们说，有些人会在代码审查方面搞些事情，并借此机会恐吓其他人。

代码审查的目的是使软件尽可能地没有错误，并在过程中进行教与学。当树立正确的态度时，大多数团队会团结成一个更好的团队，向他人学习以成为更好的开发人员以及生产更好的产品的概念。它为开发人员提供了一个交流框架，这是非常好的事情。

当然，团队成员有可能利用代码审查作为尝试建立优于他人的机会。

这种态度不是很有效。如果有一种文化开放的交流，您还需要一种尊重的文化。交流的语气有助于形成更积极的态度。每个人都会犯错。错误应被视为学习的机会。

如果您设定了正确的阶段，那么代码审查不会破坏您的团队文化，而是可以改善文化！

# 注意你的语气

这就引出了我在Cem Kaner的书《从软件测试中学到的教训》中提到的。它是关于错误报告的，但是它也可以应用于代码审查。

第86课指出：
在错误报告中采用指责或无礼的态度不会带来任何好处。 称呼程序员不专业，头脑笨拙或傻瓜是没有用的。 此刻可能感觉不错，但是您失去了信誉，需要对工作进行微观管理，并且不太可能修复许多错误。 也请注意格式。 例如，以“全部大写”形式编写的报告就好像您在尖叫一样。 如果您不确定如何阅读该报告，请让其他人阅读并仔细听他们的评论

代码检查也是如此，请礼貌一点。既要说缺点，也要说说代码的优点。即使你在别人的代码中发现了很多问题，也总有一些好的和非常好的东西，值得你去学习。对好的部分也做出评论。正面的评论提醒编码员还有很多东西要学，也增加了系统的价值。

# WTF

这使我明白了。WTF这个词肯定是令人反感的，它无助于团队精神。要有礼貌和耐心，请记住并非每个团队成员都拥有与您相同的知识或经验。在其他领域，同一个团队成员比您拥有更多的经验。您会被视为您在该域中的一个愚蠢的人吗？

请记住，每个人都会犯错误，即使您也是如此。 提出问题，而不要输入三个或四个字母的单词。请作者解释背后的原因。这将承认您尊重他或她。 但是要注意，不要问任何指责性的问题。最好问一下……什么时候您在想什么，而不是为什么…………第一个问题为对话和学习打开了一扇门。 这些是我可以在代码审查和错误报告中提供的一些技巧。

请记住，所有团队成员都在同一列火车上。如果车队的士气低落，火车将永远无法到达下一个车站。那是很大的风险。

我的个人主页：qaseven.cn

** qileilove@gmail.com ** 我的微信: 15619208923 期待认识更多的朋友