思步网

标题: 怎么样让代码走查做起来？ [打印本页]

作者: sungubbi 时间: 2008-5-20 14:04
标题: 怎么样让代码走查做起来？
项目资源有限，安排的代码走查的任务，却很可能没有被执行，或马虎应付，形同虚设。请问，大家是如何让代码走查工作做起来的呢？

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

作者: step365he 时间: 2008-5-20 17:13
感谢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 编辑 ]

作者: sungubbi 时间: 2008-5-20 18:10
关于代码走查的范围、走查问题的处理，也就是与走查的流程相关的内容，我们也是有定义的，但是定义归定义，实施情况不太好。
无法得到落实的原因在于安排了走查任务的人员并没有充分的时间、开发人员缺乏意识、走查只能发现轻量级的问题。
在QA审计的时候我们发现有认真进行走查并发现问题的人只有几个人，大多数的走查人在走形式。

我想流程是一个方面，最主要还是在怎么落实下去。
我们目前也打算开一个课程来教大家怎么进行走查，以便于提高走查的效率。
我们还在考虑是否先将交叉走查改为代码评审，先把意识和能力带起来？
又或者，先激励那几个走查的好的人？

大家都认为资源不得到解决，这个问题就是无解，怎么办？:(

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

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

作者: iamredeye 时间: 2008-5-22 17:29
刚弄明白走查的意思。:loveliness:

code review采用walkthrough的方式有效吗？我觉得还是让每个人offline review一小块代码更合适，当然还要其它注意事项

作者: cecilia 时间: 2008-5-23 09:00
当大家都在忙着赶进度的时候，代码走查就成为了一种形式，敷衍，应付。

作者: imqjzf 时间: 2008-6-13 13:00
代码走查基本上在单元测试做完以后才开始.

作者: 奔放洋气吴世勋 时间: 2014-9-10 18:34
支持，赞一个

作者: 讓愛下地獄 时间: 2014-9-17 22:05
非常好，顶一下占位编辑

作者: 有志者事竟成 时间: 2016-1-4 10:18
很有借鉴意义，先收藏了，谢谢楼主。

作者: 心鬼 时间: 2016-1-30 18:14
路过的帮顶

作者: 心隨你動。 时间: 2017-11-27 22:04
支持，赞一个

作者: 妖艳 时间: 2018-1-4 07:38
还不错哦，如果再能多分享一些就perfect了！

欢迎光临思步网 (http://www.step365.com/)