`
longssl
  • 浏览: 2493 次
  • 性别: Icon_minigender_1
  • 来自: 北京
最近访客 更多访客>>
文章分类
社区版块
存档分类
最新评论

code review实践-学会享受Code Reivew

 
阅读更多
Code Reivew实践

Code Reivew好处
1、Code reviews 中,可以通过大家的建议增进代码的质量(可读性,可维护性,以及程序的逻辑和对需求和设计的实现 )

2、Code reviews 是一个传递知识的手段,可以让其它并不熟悉代码的人知道作者的意图和想法,从而可以在以后轻松维护代码。

3、Code reviews 也鼓励程序员们相互学习对方的长处和优点。

4、Code reviews 也可以被用来确认自己的设计和实现是一个清楚和简单的。


Code Reivew做什么
从实现的角度,有的从需求的角度,有的从用户使用的角度,有的从算法的角度,有的从性能效率的角度,有的从易读的角度,有的从扩展性的角度


Code Reivew不做什么
1、找到程序的bug(不要做),代码中的bug和错误应该由单元测试,功能测试,性能测试,回归测试来保证的.
2、保证代码风格(不要做)
3、编码标准(不要做)
编码风格和代码规范都属于死的东西,每个程序员在把自己的代码提交团队Review的时候,代码就应该是符合规范的,这是默认值,属于每个人自己的事情,不应该交由团队来完成,否则只会浪费大家本来就不够的时间。

Code Reivew的时间
1、每天,每个team 15-30分。
2、每个迭代做一次,1.5小时。


分享到:
评论

相关推荐

    codeReivew

    codeReivew

    react-video-reivew

    运行命令: 将编译后的输出构建为public / npm run build 使用Mocha,Expect.JS和Sinon运行单元测试: npm run test 运行流程以通过注释进行类型检查: npm run flow

    review-helper-crx插件

    语言:中文 (简体) review-helper 在 git仓库进行 code review 的时候,diff 中会包含很多不需要 reivew 的内容,且这些内容会占据大量篇幅,在展示 diff 的时候将其隐藏会提升review的效率

    《Deep Learning》Yann LeCun, YoshuaBengio, Geoffery Hinton

    Deep learning allows computational models that are composed of multiple processing layers to learn representations of data with multiple levels of abstraction. These methods have dramatically improved...

    质量保证 checkList 例子

    1)本文档配合Reivew规程使用。 2)根据工作产品的不同类型,选择本文档中对应类型的工作产品评审Checklist做为评审指导; 3)本文档提供针对下列几类工作产品的缺陷检查指导: 工程类: SRS、HLD、LLD、Code、...

    srcbos2014:消息来源,波士顿,2014年材料-Source material

    消息来源,波士顿,2014年材料 请发布独立的演示文稿,或为需要多个文件的提交创建子目录。 请reivew的如果你不熟悉如何克隆一个回购协议和/或创建一个拉请求。

    secret_menu_API:在ROR中练习API技能

    Rails API Reivew 在以下Rails应用程序中,我可以按照以下8个步骤将Rails配置为以API的形式工作: Rails步骤 启用CORS(跨源资源共享) 通过rails g resource命令创建模型,控制器,数据库迁移表和路由 指定秘密...

Global site tag (gtag.js) - Google Analytics