今天对同事写的代码做codereview。
在粗略看了后,没有发现什么逻辑和常规错误。但是,当提交测试部测试后,个别计算机蓝苹了。(注:蓝屏正是由于codereview的那个驱动程序工程导致的)对此,研发经理大为恼火。所以让我不得不重新审视一下codereview。
codereview本意识用来提高代码质量的手段,那么有什么方式来保证codereview的质量呢?
codereview实施的前提是什么?
codereview的内容是什么?
codereview通常是编码当事人将代码提交到代码服务器上的最后一个环节,这就要求review的代码需要满足以下几个条件:
1、代码已经通过内部测试,意味着主要逻辑应该没有问题!
2、符合团队制定的编码规范
除了代码这个必要条件外,编码人需要 条理清晰 地将代码讲述一遍!并且应该明确:复审人对代码的建议并不见得是最佳的,所以有必要对关键点同程序经理讨论决定!