zoukankan      html  css  js  c++  java
  • 代码Review实践

    • 如何保证代码都被Review?人一是有惰性,二是习惯问题,可能会导致有些代码没有Review而进入了代码库。如果你用Gitlab系统,可以把主分支设置为protected,不准任何人push。开发只能在分支上进行,开发完后,在网页上发一个merge request。请其他人Review后,merge到主分支上。
    • Review的规模多大?代码如果太多,涉及太多特性,Review的时间可能会很长,Review的内容也会很多,这会不利于Review的效率和效果。Google给Review代码的长度定了一些名称,比较适合的是在100行以内。每次Review的特性要尽量少,如果你Review的内容混合了好几个特性,就很难分清楚。
    • Review的职责是可读性、可扩展性、设计模式、程序逻辑等方面。首先你要能完全看懂这部分代码,了解他们的业务实现。并可以思考有没有改进、重构的空间,是否有更好的设计,是否有更好的性能。
    • 不应该是找Bug这是单元测试、协议测试、集成测试的内容,当然,可能也会发现,但发现Bug不是目标。
    • 不应该负责代码风格。代码风格在提交的时候就应该是好的。
    • Review的方式是,叫人过来,坐你旁边,开始讲解你的设计和编码。先讲你的设计思路,然后讲代码实现。
    • 尽量让不同人的Review,而不是让“领导”Review
    • Review要仔细,要详细了解设计、编码,不能蜻蜓点水。一般来说,100行代码的变动Review要一个小时左右,可以作为一个参考值。
    • 可以用版本的diff查看哪些被改动了,不过重点Review最好还是在IDE上,可以联系上下文进行阅读。diff工具往往就显示了改动,反而不好阅读。
  • 相关阅读:
    REGIONAL SCRUM GATHERING(RSG)2019 CHINA.
    《敏捷革命》读书笔记
    敏捷之旅2017年北京站活动圆满结束
    团队合作的Ground Rules
    开发团队(Team)的主要职责和特征
    敏捷之旅2017年北京站的活动主题和讲师话题征集中
    产品负责人(Product Owner)的主要职责和技能
    战地记者也在使用Scrum
    Scrum由来
    他们是今年最可爱的人——敏捷之旅2017年北京活动志愿者
  • 原文地址:https://www.cnblogs.com/bobdeng/p/8516415.html
Copyright © 2011-2022 走看看