51Testing软件测试论坛

 找回密码
 (注-册)加入51Testing

QQ登录

只需一步,快速开始

微信登录,快人一步

手机号码,快捷登录

查看: 3736|回复: 1
打印 上一主题 下一主题

[讨论] 代码评审流程如何规范

[复制链接]

该用户从未签到

跳转到指定楼层
1#
发表于 2011-8-29 15:16:12 | 只看该作者 回帖奖励 |倒序浏览 |阅读模式
本帖最后由 3513001 于 2011-8-30 14:16 编辑

如题,代码评审流程,代码走查机制,代码抽查机制改如何制定
我是做黑盒的,对JAVA和C语言了解不多,但是这个工作交给我了,改如何制定,求高手指点
以下是我编写的



1.
准备评审

a)
代码评审前主持人需要确定评审范围、评审点和评审人,提供评审资料依据如:《代码规范》…

b)
评审人员构思评审问题,确认讨论的问题

2.
开始评审

a)
参与人员提出问题,讨论问题,确定缺陷

b)
每个问题讨论不超过15分钟,未能讨论清楚的问题,会后解决

c)

d)
参与人需记录会议上提出的所有问题、亮点及最终结论以供其他团队或新来人员借鉴

3.
作出决定

a)
主持人作出评审决定,确定缺陷跟踪人或下次评审时间

b)
评审结束后主持人给出《评审结果》,作者给出《评审报告》


i.
《评审报告》需详细记录缺陷代码的模块和行数


发起时间由项目经理确定
发起人项目经理
参与人所有开发人员
走查内容1.
编码规范问题:命名不规范  
2.
代码结构问题:重复代码、巨大的方法和类、分层不当、紧耦合
3.
工具、框架使用是否恰当,接口代码是否合理
4.
实现问题:错误验证、异常处理、事务划分、线程、性能、安全、实现过于复杂、代码可读性不佳、扩展性不好   
5.
测试问题:测试覆盖度不够、可测试性不好

本帖子中包含更多资源

您需要 登录 才可以下载或查看,没有帐号?(注-册)加入51Testing

x
分享到:  QQ好友和群QQ好友和群 QQ空间QQ空间 腾讯微博腾讯微博 腾讯朋友腾讯朋友
收藏收藏
回复

使用道具 举报

该用户从未签到

2#
 楼主| 发表于 2011-8-29 15:39:48 | 只看该作者
本帖最后由 3513001 于 2011-8-30 14:18 编辑

answer my question。。难道这个不属于测试员该做的范畴么

格式没到编辑器里面没排……
回复 支持 反对

使用道具 举报

本版积分规则

关闭

站长推荐上一条 /2 下一条

小黑屋|手机版|Archiver|51Testing软件测试网 ( 沪ICP备05003035号 关于我们

GMT+8, 2024-6-24 11:04 , Processed in 0.076335 second(s), 28 queries .

Powered by Discuz! X3.2

© 2001-2024 Comsenz Inc.

快速回复 返回顶部 返回列表