关于CodeReview --- Junior不能review senior 开发人员的代码吗

  昨天和老板谈到提高代码质量的问题,也就谈到了CodeReview.按照我们公司的一个惯例,任何工作都要有两个人去共同关注,这样可以互相做备份,不至于一个人休假事情就没法做了。写代码这活儿也是一样,一般两个人会共同关注一个模块,对里面的业务逻辑两个人都要比较清楚。基于这么一个前提,我提出可以让这两个人互相Review代码,老大便提出了一个促使我想写点儿东西的问题,我们一般是一个Senior的带一个Junior的去共同开发一个模块,难道你要Junior去Review Senior的代码吗?
  对于老大的这个问题,我当时并没有回应,但越想越不对劲,为什么Junior就不能Review Senior的代码呢?我想老大的顾虑主要在Junior developer开发经验不足,对公司系统了解不多,难于发现问题吧,我感觉这样的顾虑是大可不必的
1. CodeReview不仅仅是找茬,也是一个互相学习和传播项目知识的过程,Review Senior的代码可以促进Junior开发人员的快速成长
2. CodeReview应该是基于一些团队公认的一个check point的,这些check point应该是大家经验的总结,比如对exception是否处理得当,是否由于偷懒进行了大量的代码复制却没对复制的代码进行重构
3. CodeReview与Junior还是Senior无关的,即使是Senior,有时候也不一定完全遵循最佳实践,大家都有马虎的时候
4. 如果知道自己的代码会被Review, 我想大部分人都会更加注意自己的代码,潜意识里每个人都想证明自己是优秀的

你可能感兴趣的:(工作)