关于代码审查的若干问题！

peiping790809

由于现在都是工程文件，所以这就给软件的代码审查带来很多麻烦，例如：
仅审查一个.c文件，但是包含文件很多，而且还有很多全局变量不在这个文件中初始化，等等。因此这造成了审查者花了很多时间但是却找不到问题在哪。所以大部分代码审查都是走形式，不知道大家对代码审查是一种什么态度！

swallow0918

刚开始会很难 我们公司也是刚开始继续代码走查的工作
可能是用的方法不对吧。可以找相关的文章看看，从中学点别人的经验。

archonwang

对较为规范的组织应采取该方式提高代码及文档质量。需要注意到的问题是：代码和文档的同步化程度将影响到代码审查的过程和结果。这里需要开发规范的支撑和组织对文档撰写的重视程度。

一般进行代码审查是依据设计文档及开发规范对现有的编码文件进行审理，确保文档、编码一致，在编码中不发生低级错误，如：变量初始化。在严格的组织中，会对开发人员的编码质量作评估。

morn

套句老话，错误被发现的越早，改正所需要的成本越低。所以如果有足够的人力和时间，代码审查还是值得一做的。可以借助一些工具来提高审查的效率，如Source Insight。

zkxz

个人觉得，做代码审查应该抱着这样的态度：要明白工程对于代码的编写、管理以及维护都起到了非常重要的作用，这种形式的好处是显而易见而且被广泛使用的。而代码审查工作的性质是service，而不是manage。不能想着如何改变编码形式来让审查工作变的容易，而应该努力适应现有的形式，总结工作中的一些小技巧并不断地改进和创新工作方法，并与大家交流分享。当然，使用合适的工具是一个明智的选择，楼上所讲的Source Insight就是这样一个工具。
        对于走走形式的代码审查，做，还不如不做。这样还可以省下不少人力去做其它更需要的工作。（因为没做好代码审查，自然会有更多的问题需要人力去解决:P）

ilovejolly

首先你得搞清楚审查的重点和审查的目的

magicwin

good