51Testing软件测试论坛

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

QQ登录

只需一步,快速开始

微信登录,快人一步

手机号码,快捷登录

查看: 3429|回复: 6
打印 上一主题 下一主题

[原创] 关于代码审查的若干问题!

[复制链接]

该用户从未签到

跳转到指定楼层
1#
发表于 2006-5-25 15:21:11 | 只看该作者 回帖奖励 |倒序浏览 |阅读模式
由于现在都是工程文件,所以这就给软件的代码审查带来很多麻烦,例如:
仅审查一个.c文件,但是包含文件很多,而且还有很多全局变量不在这个文件中初始化,等等。因此这造成了审查者花了很多时间但是却找不到问题在哪。所以大部分代码审查都是走形式,不知道大家对代码审查是一种什么态度!
分享到:  QQ好友和群QQ好友和群 QQ空间QQ空间 腾讯微博腾讯微博 腾讯朋友腾讯朋友
收藏收藏
回复

使用道具 举报

该用户从未签到

2#
发表于 2006-5-25 16:17:51 | 只看该作者
刚开始会很难 我们公司也是刚开始继续代码走查的工作
可能是用的方法不对吧。可以找相关的文章看看,从中学点别人的经验。
回复 支持 反对

使用道具 举报

  • TA的每日心情
    奋斗
    2018-2-28 18:04
  • 签到天数: 40 天

    连续签到: 1 天

    [LV.5]测试团长

    3#
    发表于 2006-5-29 15:28:58 | 只看该作者

    拙见。

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

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

    使用道具 举报

    该用户从未签到

    4#
    发表于 2006-5-30 00:05:04 | 只看该作者
    套句老话,错误被发现的越早,改正所需要的成本越低。所以如果有足够的人力和时间,代码审查还是值得一做的。可以借助一些工具来提高审查的效率,如Source Insight。
    回复 支持 反对

    使用道具 举报

    该用户从未签到

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

    使用道具 举报

    该用户从未签到

    6#
    发表于 2006-5-30 14:48:30 | 只看该作者
    首先你得搞清楚审查的重点和审查的目的
    回复 支持 反对

    使用道具 举报

    该用户从未签到

    7#
    发表于 2011-9-6 23:16:55 | 只看该作者
    good
    回复 支持 反对

    使用道具 举报

    本版积分规则

    关闭

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

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

    GMT+8, 2024-11-25 05:34 , Processed in 0.080822 second(s), 28 queries .

    Powered by Discuz! X3.2

    © 2001-2024 Comsenz Inc.

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