code review

做好code review并不是一件容易的事情,如果只看一些代码实现会不会出现runtime exception难免觉得有点流于形式。自己日常工作也会做一些review code的工作,经常也会思考如何提高review的质量,让我们的实现更合理,让我们的代码逻辑更清晰,更加易于维护测试。做了几点拙劣的总结,供大家一起参考讨论,欢迎指正

1 框架背景知识很重要,比如写代码的人用了spring,那么bean定义是否合理,依赖注入用的是否得当,需要对spring框架有一定的了解才行

2 场景很重要,每个开发的story都是有场景的,review之前首先要知道他做这次改动的目的,review的过程中思考下如果我来实现,会怎么做,我们的想法为啥为不同

3 代码提交保证一次提交只做一件事,写清楚提交描述,否则做代码review的人会抓狂

4 review出问题来,要评估问题的大小,以及会出现的场景,自己在目前场景下会不会出现,比如代码在多线程跑的时候会有问题,但是目前只有一个线程,那么可以把问题记下来,后面再改,并在代码中加入ToDo标识

5 不管是code  review还是自己写代码,发现一个很小的需求实现起来却比较费力,可能我们的代码需要重构了,分析下来是好的就去做,否则永远没有时间做

你可能感兴趣的:(code review)