代码审查不是用来…… ( Code Review )

　　在我们公司里要执行代码审查。这是我们每天的例行工作。事实上，今天的我们正是从这种一直坚持探索的漫长道路上走出来的。我们尝试各种技术、方法和工具，直到我们走到今天的成就(但这并不是说我们就此停步)。

　　在这旅途中，我们发现了很多的陷阱和危险，它们等待新手们上钩。这篇文章就是关于它们的：代码审查中的陷阱和误解。

　　代码控制：

　　很多公司都把代码审查当成控制代码的方法。很多这样的公司都使用预提交策略。这种策略大多时候都是开源项目中使用，因为会有成百上千的提交者。可在一般的公 司里，很少会有这种情况。如果你雇用一个人，这意味这你要完全信任他，允许他将代码提交到代码库里。我知道有些公司会忍不住制定一些规程，要求程序员在提 交代码前必须进行“审查”和“批准”，但这并不能保证代码的质量。而且，程序员很快就会把这种代码审查当作一种“愚蠢”的公司形式过程，会开始抵制它(例 如，每月改一次密码。例如，使用像mypass1,mypass2等的密码)。

　　审判厅：

　　不要把代码审查当成寻找替罪羊和追究责任的工具。比如说，这有一个错误，你找到“审查”这段代码的人，并责备他没有发现这个问题。这种事情会给公司里的开发 工作带来严重的影响。人们会挑出每个分号放置不正确的地方，因为他们担心会被当成替罪羊。团队成员开始缺乏信心，并最终失去互信。