《代码大全》:review与inspection

  第21章Collaborative Construction(协同构建)谈到了review和inspection,读起来很有共鸣,让我回想去过去在公司兼职的一段美好时光。

  在公司里,项目的代码是共有的,对我们这些兼职学生也不例外。每天上班第一件事,就是update一下,从depot取回最新的code base(我们管这叫sync,这是我们的版本控制软件的命令)。然后每次check in之前,一定要找同项目组的人review代码,我也经常被人叫去review代码。这是一项非常有效的实践。倒不是因为review能发现多少错误,只是想到待会儿会有人来review自己写的代码,那么在写代码的时候就会比较小心,免得出错。check in的次数倒没有详细规定,大家都避免自己手里的代码out of sync的时间过长,一般一天check in一两次。每次check in都要写明reviewed by whom。

  review是非正式的检查。inspection是正式检查。

  我们每周大约有一次inspection(我们叫code review),参加inspection的人员组成一般是自己项目组的人加上大约同样多的别的项目组的人,一共10多个人的样子(算上我们几个实习生),会提前预约一个不大的会议室,专门找个时间来做这件事。先确定这次轮到review谁的代码,以及review哪些代码,提前把代码打印出来(大约不到10页A4纸,一定要有行号),复印给每个人。大家先看看,到了时间就一起去会议室讨论,讨论大约持续2小时。既讨论代码的设计,也讨论具体语句的优劣。因为代码提前看过,所以一开始就会有人提出意见,然后随着讨论的进行,会不断有新的意见提出来,基本不会冷场。这是全组技术交流的好时候,也能听到别的组的意见。

  《代码大全》也提倡这种协作方式(当然书中提到很多别的协作方式),这是提高产品质量的行之有效的做法。

你可能感兴趣的:(view)