review & inspection

原创 2006年06月12日 23:00:00

这里的文档,要经过一遍又一遍的review 。听取了多方面的意见,而且经常对一个文档的review,要开一个全球的电话会议。对于代码,也要经过一个Inspection的过程。Inspection的时候,至少要4个人参加,一个是代码的auther,还有代码的reader, 还有一个是recorder,还有一个moderator

其中,moderator负责主持,监督代码检查会议的质量和效率,确认与会者是否有足够的时间查看了相关的代码和文档。Reader的角色最为艰巨,他负责讲述代码的功能,但他不是代码的作者。我们都知道,看别人的代码是意见痛苦的事情。Recorder主要负责记录整个会议过程。

这里还特别规定了每个检查代码的会议不超过2小时,每小时检查代码的行数不超过180行。每千行代码所发现的问题不少于30个。规定的制定者认为,如果超过2小时,人就困了,没有什么效率,如果单位时间内检查的代码过多,发现的问题过少,就没有办法保证检查的质量。

 

这里体现出来两个技巧。一个是对脑力劳动的检查,以为脑力劳动不好用特定的量化指标来衡量,那么就用reviewInspection的方法来使得这个成果是高质量的。对于reviewinspection有通过上面的方法来保证质量。前些天听童兄说,他们那里的review必须提出一定数量的书面意见。看来这样的review的质量更好一些。

 

另外,这样大规模的reviewinspection的另外一个好处就是防止特定的技术掌握在个别人手中从而带来的风险。

 

同时 ,也增进了对其他同事工作的了解。有助于提高整个团队的效率。

《代码大全》:review与inspection

  第21章Collaborative Construction(协同构建)谈到了review和inspection,读起来很有共鸣,让我回想去过去在公司兼职的一段美好时光。  在公司里,项目的代码是...
  • Solstice
  • Solstice
  • 2006年01月18日 14:18
  • 5961

《代码大全》:review与inspection

  第21章Collaborative Construction(协同构建)谈到了review和inspection,读起来很有共鸣,让我回想去过去在公司兼职的一段美好时光。  在公司里,项目的代码是...
  • hejishan
  • hejishan
  • 2007年12月18日 03:58
  • 250

This inspection checks references to files and directories

使用WebStorm编写CSS文件出现如下问题: li.vip a{   padding-left:30px;   background:url(Images/vip.jpg) no-repea...
  • The_Rolling_Kid
  • The_Rolling_Kid
  • 2017年04月27日 21:54
  • 815

Pycharm idea 报警告:This inspection detects shadowing built-in names, such as 'len' or 'list'

因为这几条刚刚开始自学Python,然后也就简单的跟着书上的例子来,做个简单的不能再简单的实验。然后发现编辑器报如下的警告信息。 报警告:This inspection detects shadow...
  • qq_27093465
  • qq_27093465
  • 2017年02月09日 15:58
  • 3968

android studio的一些技术

android的优化检查中,ADT使用lint进行检查,而检查中有一些我们自己知道需要忽略的,可以通过lint.xml进行配置,也就是说,lint.xml是用来配置忽略检查的文件。我们可以自己手动创建...
  • hangeqq685042
  • hangeqq685042
  • 2015年08月31日 18:01
  • 2515

Code Inspection

从这个release开始,我所在的团队会对每一名developer的代码,也就是产品代码,在正式check-in之前进行code inspection。要求开发团队和peer test参加,人员控制在...
  • Maybesky
  • Maybesky
  • 2010年02月23日 19:47
  • 1529

This inspection controls whether the Persistence QL Queries are error-checked

转载:http://www.progtown.com/topic1153146-idea-7-swears-on-hibernate-hql.html 就是在你的那个  红色的那个灯泡 那里  禁用...
  • u013125680
  • u013125680
  • 2015年08月17日 11:39
  • 3296

android studio慢慢熟悉-好用的 设置 查找框

开始使用android studio了。反正是各种有问题,慢慢来 以前看到ec的黄线就难受,现在as的又是波浪线了。看下面这个 波浪线 Spellchecker inspection helps ...
  • wanghao200906
  • wanghao200906
  • 2015年05月10日 19:45
  • 3290

Android studio-----inspection

Android的优化检查中,ADT使用lint进行检查,而检查中有一些我们自己知道需要忽略的,可以通过lint.xml进行配置,也就是说,lint.xml是用来配置忽略检查的文件。我们可以自己手动创建...
  • qq_32239767
  • qq_32239767
  • 2017年03月28日 12:57
  • 279

Inspection

Once the individual CN domain name is the management of the tightening expected in the near future t...
  • hanlanzi
  • hanlanzi
  • 2010年01月18日 12:31
  • 51
内容举报
返回顶部
收藏助手
不良信息举报
您举报文章:review & inspection
举报原因:
原因补充:

(最多只允许输入30个字)