代码审查

想要留住的季节和温度

周一早上一出门,气温刚合适,穿上薄薄的衬衫,配上马甲,七分裤,单鞋……不觉得冷也不觉得热,空气很清新,感觉整个世界都很明亮,各种花儿慢慢开谢了,叶子却全都长了出来,到了欣赏叶子的季节,锦江大道两边的刺桐仿佛也是一夜之间全长满了新叶,车行驶在这样美的道路上,心情也很愉快。跟同事聊天说,想要永远留住这个季节,这样的天气,这样的温度,实在太讨喜。

代码审查

和团队QA一起走查小伙伴的专用补丁代码,有疑问的代码多从业务角度问几个为什么,大胆推测,小心取证,发现几个大问题。技术实践类的活动跟具体的操作方法以及执行力度实在有太大的关联。我们实在太擅长制定各种规则、条款。最终的出发点是很好的,可落地时,如果深度不够,也许就是徒劳。团队质量改进,一直在做探索和尝试,而目前看来,当前最有效便是更加深度的代码审查,至少对于新修改合入的代码来讲,如果把关做得好,应该效果是很可期待的。这里提到的是“代码审查”而不是“代码走查”。走查有一种走马观花的感觉,好像有点追求快和速度。对于结构好,清晰的代码可能够用。但对于复杂的代码可能用审查更合适,因为“走查”略浅,且这样的代码往往需要结合上下文和业务仔细评审才有可能发现问题,这样的实践活动也更适用于烂代码的修改。


图片发自简书App
最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
平台声明:文章内容(如有图片或视频亦包括在内)由作者上传并发布,文章内容仅代表作者本人观点,简书系信息发布平台,仅提供信息存储服务。

推荐阅读更多精彩内容

  • 【编者按】本文作者为 Hugo Giraudel,主要从各个角度论证了代码审查的重要性以及实现方法。文章系国内 I...
    OneAPM_Official阅读 1,295评论 0 2
  • 代码走查和代码审查是白盒测试方法中的2种。 代码走查: 从参加人员来说,应该是项目的整体参与者,如果项目太大,整体...
    许小小晴阅读 10,348评论 1 3
  • 最近一直在从事一款iOS的app的开发工作。为了提高团队整体的代码质量,从项目开始我就一直担负着代码审查重构的工...
    Ifdef_Max阅读 1,833评论 1 13
  • 本文为转载 第一个层次:业务架构的审查重构 这是最高层次的代码审查重构。其实,这个阶段的审查并未真正涉及到具体的代...
    _源计划阅读 338评论 0 0
  • 要点 代码审查者在审查代码时有非常多的东西需要关注。一个团队需要明确对于自己的项目哪些点是重要的,并不断在审查中就...
    kkzzzzzz阅读 675评论 0 1