codereview重点排查

原则：

代码都要经过review才能上线
小组内交叉review
输出review记录

步骤：

先过静态代码扫描。
讲解项目设计和代码结构。
线下交叉review，打好todo tag。
组织小会针对todo tag讨论，明确是否需要修改，做好会议纪要。

重点review内容：

配置：资源配置正确性，重点关注数据库、redis、dubbo等中间件连接池配置，连接池大小、超时时间。
数据库：涉及的sql改动，重点关注查询条件索引、是否分页、分页大小、更新和删除条件（避免出现无条件删除、更新）。查询操作走从库。
线程、线程池、锁的使用要重点关注。
循环：循环跳出条件，避免死循环。
涉及数据处理的job，注意控制处理速度，和dba、依赖业务方协调好。
依赖管理：只保留必要的依赖，依赖的版本显示声明，减少依赖传递造成的包冲突。
安全：配置好登录验证，敏感接口数据加密、防重放、签名；写入数据接口，注意数据过滤。
代码规范（参考阿里技术手册）
消息处理、任务处理幂等性检查
异常基本处理：error日志输出、入参必输出
根据qps预判接口合理性
项目命名：假设业务名b，则api命名=b-api; dubbo服务命名=b-service; web层命名=b-mobile; 数据层命名：b-dal/b-core

CodeReview总结相关推荐

什么叫取反_转载：CodeReview正确的姿势是什么？
作者:微博是阿里孤尽链接:https://www.zhihu.com/question/383079175/answer/1109655276 来源:知乎著作权归作者所有.商业转载请联系作者获得授 ...
codereview介绍
1. 定义: Code review is systematic examination (often known as peer review) of computer source code. I ...
常用工具说明--搭建基于rietveld的CodeReview平台(未测试)
为什么要codereview . 整个团队的编码风格是统一的. . 有高手能对自己的代码指点一二,从而提高编码水平. . 减少低级错误的出现 . 约束自己写高质量的代码,因为是要给人看的. 我们对co ...
技术分享 | CodeReview主要Review什么?
源宝导读:Code Review, 意即代码审查,是指一种有意识和系统的召集其他程序员来检查彼此的代码是否有错误的地方. 在敏捷团队中推行CodeReview, 可以帮助团队快速成长.本文将分享在&q ...
java中的codereview
关于codereview,在平时的开发中,经常忽略的环节,参照目前介绍写好代码的几本书和之前掉进的坑,做了一个总结,分享出来. 为什么要做通过review规避一些代码层面的问题提升可读性,方 ...
Testing - Codereview Checklist
Gieno Testing : Codereview Checklist Overall/General Does the code live up to requirements? Is the ...
关于CodeReview
CodeReview的目的是提升代码质量,尽早发现潜在缺陷与BUG,降低修复成本,同时促进团队内部知识共享,帮助更多人更好地理解系统. 下面我们开始了解关于CodeReview的一些原则. 架构设计 ...
自动化CodeReview - ASP.NET Core请求参数验证
自动化CodeReview系列目录自动化CodeReview - ASP.NET Core依赖注入自动化CodeReview - ASP.NET Core请求参数验证参数验证实现在做服务端开发 ...
review代码从哪些角度_转载：CodeReview正确的姿势是什么？
CodeReview正确的姿势是什么?www.zhihu.com 全文转载于 "微博是阿里孤尽"在上面的回答,已征求本人同意. 以下是原文: 最近 CodeReview(以下简称 ...
docker部署codereview/gerrit
gerrit官网上有最新的release,但是下载服务器应该是在国外的缘故,无法访问(不是慢...),所以只能放弃该方法.另外从网上冲浪得知安装release版还需要准备各种环境,比如Java,jdk ...

CodeReview总结

codereview重点排查

CodeReview总结相关推荐

最新文章

热门文章