代码审查是在我们公司进行的。这是我们的日常。事实上,今天的我们已经走出了这条漫长的执着探索之路。我们尝试了各种技术、方法和工具,直到我们取得了今天的成就(但这并不意味着我们就此止步)。在这段旅程中,我们发现了很多陷阱和危险,等待新手上钩。这篇文章是关于它们的:代码审查中的陷阱和误解。代码控制:许多公司使用代码审查作为控制代码的一种方式。其中许多公司都使用预先承诺策略。这种策略主要用于开源项目,那里有成百上千的提交者。但在一般公司,这种情况很少见。如果您雇用某人,则意味着您完全信任他并允许他向代码库提交代码。我知道有些公司不免会有一些程序要求程序员在提交代码之前先“审核”和“批准”,但这并不能保证代码的质量。此外,程序员很快就会将这种代码审查视为“愚蠢的”公司式流程,并开始抵制它(例如,每月更改一次密码。例如,使用类似mypass1、mypass2等的密码)。法庭:不要将代码审查用作找替罪羊和追究责任的工具。假设有一个错误,你去找“审查”这段代码的人并责怪他没有看到问题。这种事情可以对公司的开发工作造成严重的影响。人们会挑出每个分号放错地方的地方,以免成为替罪羊。团队成员开始缺乏信心,最终失去互信。责任任务:不要过分要求程序员做代码审查。如果你强迫他们每天做一个小时的代码审查,他们很快就会讨厌它并将其视为一项无聊的任务。Codereview是学习,是表扬,是得到反馈,是一种社交性很强的活动。代码审查应该是有趣的,而不是无聊的。我和我的代码如果你的代码被留下一些评论的人审查(有时不那么友好),请不要生气。他并不是说你是一个糟糕的程序员。这不是他的意思,这也不是代码审查的目的。他所做的是批评代码(而不是作者)。代码审查是针对代码的,而不是针对你的。不要将代码审查作为相互讽刺的论坛和相互批评的工具。当你写评论笔记时,尽量不要粗鲁也不要太苛刻。尝试将代码放在作者的位置上。总而言之,有很多种错误会破坏代码审查。以上四种是我经历过或预料到的,请大家提高警惕。我保证会写另一篇文章描述“代码审查来了......”如果你想试用代码审查工具,请访问codebrag.com,它是我们经验和实践的成果。原文链接:http://blog.zuchos.com/2013/10/what-c??ode-review-is-not-about.html翻译链接:http://www.oschina.net/news/45315/code-review-不是关于
