怎么样让代码走查做起来？

step365he

1. 代码走查需要摘录所检查的代码文件名。如果有问题，需要标注哪一个文件名和代码行出现问题。最后检查人需要签字，表示对检查结果负责人。
2. 代码走查不用检查所有的代码，比如架构搭建，基础类编码，重要存储过程，核心算法，处理高并发的代码等需要检查。

step365he

感谢msn群中的vindy贡献了下面的建议：
我们公司是code review 之后测试，测试不通过打回给开发人员然后再code riview，codereview要有记录，但是没有非常严格的检查表；不过执行的好像还可以。 一般coderiview人员是同级别更有经验的开发人员构成。有些规模的项目是测试同期做codereview，同时report bug。而这些参与codereview的开发人员 一边从事开发，一边codereview其它项目组代码 。公司会根据优先级给不忙碌的人。每个人工作的时候任务也是有优先级的，有时候codereview更高。具体操作根据项目特点而定，比如用在40人的大项目组（有几个小项目组）会比较严格。
vindy还提到我们没有对代码评审，但是test之后出问题还得来一轮更花时间，而且通过review又发现问题。怎么也是一种无形的批评。。。所以是靠习惯人品来做的，效果一直不错。:)

ps 我稍许对vindy建议做了些文字调整。

[ 本帖最后由 step365he 于 2008-5-20 17:14 编辑 ]

step365he

1.
代码审查需要的精力是不小的，如果检查人手上有任务，而且在进度压力下，不可能坚持高质量进行代码审查。
如果单独设立检查人，虽然能保证检查质量，但不符合人力资源要求。
正如vindy所言，需要权衡。

2.
无论是代码审查，还是代码检查；归根结底都要阅读代码，查看是否存在问题。
如果考虑到资源共享，可以考虑将白盒测试结合。