标签：开心 Code 审查 review 评审 code 代码
什么是Code review
Code review is the process of making pieces of source code available for other developers to review, with the intention of catching bugs and design errors before the code becomes part of the product.
为什么需要做代码评审
提升产品质量，打造技术的好牌子
产品开心，业务开心，测试开心，领导开心，自己也开心，大家一起爽的事情值得做。
及早发现缺陷
风险前置，提测前，上线前就发现代码和设计层面的缺陷，免得你故障复盘的时候，要和大家一遍遍的review代码。
把代码晒一晒
把代码暴露在阳光下，写给自己的代码和给其他人看的代码，那是相当不一样的，敢于亮剑的代码是不会发霉的。
知识传播与共享
微观层面：观察学习别人优先的设计思路和实现细节；宏观层面：
寻找更好的解决方案
每个人的背景长处各有不同，大家review的时候，思想的火花四溅激烈碰撞，提出更好的创意。
凝聚团队共识
因为陌生所以疏远，因为熟悉所以亲切。
不断提高团队的下限
因为优秀所以优秀，不断向明星代码靠拢，成为明星中的一员。
何时需要review
频率越高越好，交流放在每一天，具体根据各团队实际情况安排，下面列出几个关键节点。
需求评审
技术是为业务服务的，代码的所有价值最终都要通过业务结果来呈现，希望大家具备产品思维、业务思维，具有预见未来的能力。
技术方案
实现思路，库表，系统架构，逻辑流程梳理。
系统骨架
哪些类，哪些接口，系统内部流程，系统间流程调用关系。
核心代码
关键节点的代码，核心业务逻辑的代码。
评审完成的代码
最终开发完成待上线的产品。
Code review的形式
线上走查
通过ReviewBoard的线上评审功能，review可以通过工具及时评审并记录问题。owner也可以查看和答复问题的解决进展，是一种轻量级、灵活的代码评审形式。
1V1走查
2-3人坐在电脑前，互相讲解和审视，直接沟通。
会议走查
大需求或者新同学推荐采用这种方式，在很多人的眼睛反复扫射之下，潜在的bug、编码不规范、涉及不合理、注释不严谨、性能不友好等很多问题都会暴露，有不同的看法也可以直接当面沟通交流，取长补短，扩展大家的认知。
Code review最佳实践
积极的态度
Review，你是建设者，不是破坏者，提出合理的建议而不是单纯的吐糟。”你这个代码写的太挫了“ ----> “这个变量名字u难以理解，咱们换成这个user是不是更好”。
Owner，虚心接受大家的建议，都是一个锅里吃饭，谁也不会闲的没事儿砸自己的锅。
Owner负责跟进拿到review结果
Owner发起review前，确保单元测试和静态分析扫描是通过的。
不要抱怨他人不给你review，找到他然后请求他的支持。
欢迎关注我的公众号沐于清明一起交流
标签：开心,Code,审查,review,评审,code,代码 来源： https://blog.csdn.net/u012914981/article/details/88936321
本站声明： 1. iCode9 技术分享网（下文简称本站）提供的所有内容，仅供技术学习、探讨和分享； 2. 关于本站的所有留言、评论、转载及引用，纯属内容发起人的个人观点，与本站观点和立场无关； 3. 关于本站的所有言论和文字，纯属内容发起人的个人观点，与本站观点和立场无关； 4. 本站文章均是网友提供，不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属；如您发现该文章侵犯了您的权益，可联系我们第一时间进行删除； 5. 本站为非盈利性的个人网站，所有内容不会用来进行牟利，也不会利用任何形式的广告来间接获益，纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。