如何做好Code Review:思考、方法和实践

发表于:2013-3-15 11:00

字体: | 上一篇 | 下一篇 | 我要投稿

 作者:kingwufan    来源:51Testing软件测试网采编

  3、其他问题

    a、缺少验证和异常处理。例如不对数据进行验证,或者不处理异常又或者捕获无法处理的异常

    b、对工具和框架的错误使用。例如有的ORM框架提供非常方便的运行时延迟加载功能,方便倒是方便,滥用却会有性能隐忧

    c、缺少可读性。注释和命名规范是一个问题,不过过深的调用层次都会影响代码可读性。

    d、缺少扩展性。例如在没有DLR的情况下在业务层使用匿名对象。

    e、缺少安全控制

    f、性能隐患。例如在C#中使用了非托管资源而没有进行释放,或者说有过多、过于频繁的I/O访问

  4、测试问题

    a、没有测试或者覆盖度不够

    b、测试代码滞后,在更改逻辑后没有对测试进行同步更新

  以上是一些通常的Code Review的内容了。当然这里再强调一点的就是规范的完备性,和开发过程的成熟度。Code Review是一种相对高级软件开发活动,没有必要一定要执行。不过一旦实施成功将会有巨大的回报。

  在实施过程中还有一些心得:

  1、做好准备。如果对项目很熟的话,应该知道哪些地方会有问题,Code Review是重新审视和从内因上解决这些问题的良机。

  2、不要考虑业务逻辑。完全专注于代码的实现,例如算法的效率,抽象的粒度。整个代码体系结构的平衡性

  3、虚心学习。不要有针对性

  4、在原则和现实之间平衡。不可能所有事情都完美,所以有的时候我们坚持原则,有的时候修改规范。

  5、不要让自己Review自己的代码

  6、总结跟进Review的结果,并且坚持开展这个活动。这才是最重要的

22/2<12
《2023软件测试行业现状调查报告》独家发布~

关注51Testing

联系我们

快捷面板 站点地图 联系我们 广告服务 关于我们 站长统计 发展历程

法律顾问:上海兰迪律师事务所 项棋律师
版权所有 上海博为峰软件技术股份有限公司 Copyright©51testing.com 2003-2024
投诉及意见反馈:webmaster@51testing.com; 业务联系:service@51testing.com 021-64471599-8017

沪ICP备05003035号

沪公网安备 31010102002173号