review代码,需要做些什么???
有一种习惯,叫看代码找问题;有另一种习惯,叫不看代码很不习惯。
这,矛盾,处处不在!
review代码(code diff升级)到底可以做些什么?该做些什么?
1、整体代码风格是否贴切已有框架的设计风格
一个系统本有一套体系,你就不按这个走?前人踏过无数的坑,你就要去踩?
2、注释
别人问,这定义的什么?回答:忘了
别人问,这个是干嘛的?回答:忘了
!!!!!!
3、入参的定义,出参定义(特别是枚举)
考虑某个入参是否以前已有定义?是否其他系统已有定义?是否数据库已有定义?
本部门内各系统,同一含义的参数最好(应该强制)都统一,不然系统与系统之间交互要转来转去,与数据库交互要转来转去。
4、日志打印
a、前端入参、或接受其他系统调用的入参必须打印;
b、调用依赖服务入参、出参必须打印;
c、捕获的未处理异常堆栈信息必须打印;
d、捕获的处理异常打印的信息必须明确问题所在;
e、日志级别得明确
5、异常处理
a、异常类型定义必须明确,不能一股脑抛系统异常;
b、调用第三方服务,最好单独包一层try catch(不单单是整个外部方法的异常捕获);
c、。。。。。。
6、埋点统计
我要用户访问量!
我要异常访问量!
我要今天多少用户干嘛干嘛的量!
7、报警机制
调用第三方出问题了,自己不知道;
别人要服务大概响应时间,自己不知道;
自己服务有问题了,自己不知道;
多么尴尬的事!
8、业务实现
a、了解清楚需求了吗?
b、这设计方案讲得通吗?
c、依赖服务文档看没?
d、联调过没?交互流数据确定过没?
e、在什么环境联调?本地也叫联调?
f、表设计合理?索引创建合理?
g、增删改sql没问题?
h、简单的参数check完善?
i、完全信赖别人的传入?
j、穿插的不是很重要的消息推送做了无伤大雅处理?
k、能异步处理的开了新线程?开的新线程有效?
l、 。。。。。。
review代码,需要做些什么???的更多相关文章
- 1 初级.net web工程师,在工作中都做些什么
初级.Net Web工程师,在工作中都做些神马? 职责 初级.Net Web工程师的主要职责,就是按比较详细的要求去完成代码. 比较详细的要求是指:一般会把页面式样.功能的描述.数据库结构.性能要 ...
- spring扩展点之三:Spring 的监听事件 ApplicationListener 和 ApplicationEvent 用法,在spring启动后做些事情
<spring扩展点之三:Spring 的监听事件 ApplicationListener 和 ApplicationEvent 用法,在spring启动后做些事情> <服务网关zu ...
- 学java可以做些什么
学java可以做些什么 对于很多新手来说,刚开始接触Java会很迷惘,不知道Java可以做什么.其实Java 可以做的东西太多了,手机游戏.中间件.软件.网站,电脑游戏,以及现在流行的安卓手机app等 ...
- WebVR大潮来袭时,前端开发能做些什么
WebVR大潮来袭时,前端开发能做些什么? WebVR即web + VR的体验方式,我们可以戴着头显享受沉浸式的网页,新的API标准让我们可以使用js语言来开发.本文将介绍如何快速开发一个We ...
- 曹工说Redis源码(7)-- redis server 的周期执行任务,到底要做些啥
文章导航 Redis源码系列的初衷,是帮助我们更好地理解Redis,更懂Redis,而怎么才能懂,光看是不够的,建议跟着下面的这一篇,把环境搭建起来,后续可以自己阅读源码,或者跟着我这边一起阅读.由于 ...
- 这样Review代码牛逼啦!
这样Review代码牛逼啦! 一个对项目负责的团队代码质量检查是必不可少的,有条件的团队经常有代码review习惯,这样可以使技术团队共同进步,但是一个庞大的工程做代码review其实是很麻烦的,所以 ...
- Vue 3.0 来了,我们该做些什么?
靓仔路过,不要错过 想必 Vue3.0 发布这件事,大家都知道了. 我也是从朋友圈的转发得知此事,博客平台.公众号.朋友圈基本都有这么一条新闻,可见 Vue3.0 的被期待程度,因为 React 16 ...
- /proc/sysrq-trigger该文件能做些什么事情-转载
/proc/sysrq-trigger该文件能做些什么事情呢? # 立即重新启动计算机 (Reboots the kernel without first unmounting file system ...
- Review 代码
最近需要 Review 代码,学习了<代码整洁之道>.<代码质量>等书籍. 把对这些代码之道的学习心得整理成文
随机推荐
- 牛客练习赛55 E-树 树形DP
题意 你有一颗大小为\(n\)的树,点从\(1\)到\(n\)标号. 设\(dis(x,y)\)表示\(x\)到\(y\)的距离. 求\(\sum_{i=1}^{n}\sum_{j=1}^{n}di ...
- List对象遍历时null判断逻辑梳理
凡是对集合list,set,map,数组等进行循环一定要判断是否为null,增强代码的健壮性.下面以list为例, 使用for循环遍历list对象,处理其中的元素时,需要对null值判断: ...
- 动手动脑(ppt中6个问题)
问题一:仔细阅读示例: EnumTest.java,运行它,分析运行结果? public class EnumTest { public static void main(String[] args) ...
- jquer绑定和获取属性
最近每天都在熬夜,今天感觉眼睛特别涩,我决定,今天早睡,哈哈哈,上次总结了jquery控制节点,今天总结jquery控制属性,学习完基础知识,看看下面的案例练习一下,掌握的会更好 属性绑定和获取 ...
- vue实战_从头开始搭建vue工程
写在前面:vue工程入口文件分析 /index.html,/src/main.js,/src/APP.vue /index.html文件示例: <!DOCTYPE html> <ht ...
- redis数据结构有哪些
1.String 可以是字符串,整数或者浮点数,对整个字符串或者字符串中的一部分执行操作,对整个整数或者浮点执行自增(increment)或者自减(decrement)操作. 2.list 一个链表, ...
- Flutter安装
下载右边的安装包以获取最新版本 stable 的 Flutter SDK 将压缩包解压,然后把其中的 flutter 目录整个放在你预想的 Flutter SDK 安装目录中(比如 C:\src\fl ...
- nginx配置不当引起的错误
1.CRLF注入 1.1环境配置 apt install nginx vi /etc/nginx/sites-available/default location / { return 302 htt ...
- 设计一个支持 push,pop,top 操作,并能在常数时间内检索到最小元素的栈 绝对值?相对值
小结: 1. 常数时间内检索到最小元素 2.存储 存储绝对值?相对值 存储差异 3. java-ide-debug 最小栈 - 力扣(LeetCode)https://leetcode-cn.com/ ...
- SQL-W3School-高级:SQL LEFT JOIN 关键字
ylbtech-SQL-W3School-高级:SQL LEFT JOIN 关键字 1.返回顶部 1. SQL LEFT JOIN 关键字 LEFT JOIN 关键字会从左表 (table_name1 ...