当前位置:   article > 正文

代码Review经常碰到的几个问题与应对方法_代码review 为什么发现不了问题

代码review 为什么发现不了问题

1、项目组人少,比如某个模块就一个人开发,如何做走查?
----建立研发中心级别的走查机制,这个模块人少,但是可以邀请他所属的开发组参与走查,比如他用的是C++,可以邀请其他C++成员参与走查,不一定要本项目的人才能走查。

2、走查问题记录不方便,是否有好用的走查工具?
----我们的代码是用Git管理的,与Gerrit很好的集成,Gerrit就是个走查工具,提交代码到Git时就可以设置必须走查,走查记录也可以保存下来
----网上也有很多免费的走查插件,如果图省事,可以在需要修改的代码上记录特殊标记,比如【BUG】

3、走查发现的很多问题都是规范性的,对代码质量没有太大帮助
----走查一定要先用工具做静态扫描,扫面的问题改完后才进行代码走查,否则两者工作重复。Java推荐checkstyle、findbugs,C++推荐CPPlint、cppcheck

4、开发的时间本来就不多,再加上代码走查,时间太紧张
----代码走查的作用没必要赘述,建议在固定时间举行,当团队养成习惯后,就会很自然成为团队日常工作的一部分,建议安排在每天下午17:00--17:30,流出半小时解决走查发现的问题
----如果有同事反馈正在定位紧急故障或有其它紧急事情,可以申请延缓或者不参加本次走查活动

5、评审的同事对代码不熟悉,发现不了问题。
----走查活动不仅仅要看功能是否实现了,还要看代码和设计是否一致?测试用例是否完备和有效?
----有利于知识共享,打破技能壁垒,同时开发同学通过讲解自己

声明:本文内容由网友自发贡献,不代表【wpsshop博客】立场,版权归原作者所有,本站不承担相应法律责任。如您发现有侵权的内容,请联系我们。转载请注明出处:https://www.wpsshop.cn/article/detail/56239
推荐阅读
相关标签
  

闽ICP备14008679号