文档章节

一次服务重构上线后的反思

ljhlgj
 ljhlgj
发布于 2017/07/01 15:13
字数 1599
阅读 14
收藏 0
点赞 0
评论 0

背景

我们的一个服务模块使用websocket技术,通过长链接来协同广播用户的操作行为,但是服务职责不单一,边界划分不清晰,承载了很多职责之外的业务,所以决定对服务进行彻底重构,差不多是重写。

经过一个多月的努力,服务重构完成,包括架构、集群方式调整,数据存储、操作模型设计等,经过严格测试后上线,虽然核心业务可用,但还是出现很多本不该出现的问题,现总结一下,权当做最近一段时间工作的总结,和对问题的反思,以后的工作中要引以为戒。

一、不要忽视简单的业务逻辑、代码

无论业务多简单,都不要小觑它,必须要有单元测试,且要尽量覆盖到每一个分支、每一行代码,据说行业中行覆盖率达到60%以上,就已经很好了,最好的是68%。

删除确认无用的代码,问题往往出现在这些不经意的地方,可有可无的代码不仅给业务的理解带来困难,给维护者增加维护成本,甚至影响整个代码的质量。因为你感觉这些代码不重要,从而意识上就不会过多的关注它,结果往往就是因为这些代码造成系统难用,或者不可用,影响整个服务。

二、重视偶发的、不起眼的exception

对于偶发的错误,也要引起足够重视,刨根问底,找到根本原因并解决它。否则,一旦上线,就会放大这些问题,造成整个服务不可用。有时只要在平时开发、测试的过程中稍稍关注一下发现的问题,就能很快的解决它。

这次重构后,在自测的过程中,偶发出现一个序列化错误,简单看了一下错误抛出的地方,是调用第三方服务的,所以就没有放在心上,当时就感觉是第三方服务可能偶尔出现的返回数据格式问题,另外第三方服务也没有修改,一直在线上稳定运行,所以就没有过多的关注。上线后,该错误频繁抛出,就进行了回滚,经查证原因是在解析第三方服务返回的数据时候,一个符号写错了,造成序列化失败,如果在开发、自测的过程中对这个问题关注一下,根本就不会把它带到线上,也不会造成服务回滚,浪费了很多时间与精力。

三、碰到的疑问要立即验证确认

平时在开发、自测的过程中,会碰到一些自己不确定的地方,无论是业务,还是技术,都要及时确认验证,不要想着随后有时间了再说,一旦有了这个想法,即便以后有时间了,你也不会去处理,这也许就是拖延症吧,那么问题随后就会进一步放大,导致服务不可用。

四、多关注测试日志

服务提测后,多数RD会认为此时重点在QA,而自己相对来说就会轻松一些(^__^) ,实际上此时RD也确实是比开发时的时候时间多一点。但是,如果你的想法是现在终于有时间来放松一下了,聊聊天、刷刷微信、微博,看看新闻啥的,等着QA来告诉你问题,那就大错特错了,千万不要浪费此时的空闲时间。

此时,如果QA问题不多、或者没有汇报问题的时候,RD的时间还是比较空余的,此时对RD来说是很好的时机,因为现在问题不多,没人打扰,才能够真正的静下心来好好想一想,回顾一下还有没有什么问题自己没有注意到;一些棘手的问题当时考虑的是否全面,是否有遗漏;一些逻辑是否能够更加严谨等。站在一个更高的层面看看是否还有能够优化、完善的地方,多看看测试日志,利用好这段时间,能够发现并解决很多问题,不要等到上线后出问题让自己后悔不已。回过头来看,上线的问题,其实测试日志中都出现过,上线前有人关注一下,就不会导致上线后问题的发生。毕竟QA是对功能进行测试,很多不明显的异常他们是不知道的,QA不反馈,不代表没有问题,QA也会有遗漏,也不能察觉到所有的错误,所以在QA测试过程中,RD也要多测试、多关心日志,多思考,将问题杀死在摇篮之中。

本次上线前,有2天时间,QA在进行整体的回归测试,而RD都在放松、浪费自己的时间,没有一人去看测试日志,结果上线后,一堆错误日志,导致回滚,下来再看,这些错误在测试日志中早已经存在,如果2天内多多关注测试日志,就不会将错误带到线上,造成上线失败。

五、不要抱任何侥幸心理

对于任何问题,都不要抱有侥幸心理,说可能是环境的差异才导致的,如果确实是环境问题导致的,也要拿出确实可信的理由,不要认为现在的问题上到线上就没事了,一定要刨根问底,找到根源,如果确实是环境的问题,要看看线上环境有没有该问题,该问题是如何发生的等。通常情况下,你感觉有问题的地方,也确实是有问题,无论什么环境。

总得来说,这次重构学会了很多东西,也让自己对事物、对人的看法与以往有了很大不同,无论开始你认为多么复杂的事情,只要一步步认真的去思考、去动手,都会慢慢的明朗起来,最终解决它。

© 著作权归作者所有

共有 人打赏支持
ljhlgj
粉丝 3
博文 8
码字总数 15153
作品 0
昌平
程序员
内建控制结构之重构指令风格的代码为函数式风格

本节我们将重构上一节中的例子(乘法表),将其重构为纯函数式风格代码。如例: 结果: 注:这里只要注意两个地方,第一个,val padding = “ “ * (4 -prod.length)这句使用了隐式转换;第二...

柳哥 ⋅ 2014/06/07 ⋅ 0

JBoss Web 3.0.0 Beta 2 发布

JBoss的Web服务器是一个基于 Tomcat 的企业中型和大型应用软件准备的Web服务器。 该版本在 Beta1 的基础上主要集中在处理与 JBoss AS 的集成重构上,另外该版本不管是程序的体积上还是启动的...

红薯 ⋅ 2010/04/10 ⋅ 2

zxing代码中core和javase模块介绍

zxing的源码中包含很多的模块,模块列表如下: core: The core image decoding library, and test code javase: J2SE-specific client code zxingorg: The source behind zxing.org/w zxing......

彭苏云 ⋅ 2013/12/17 ⋅ 0

AjaxUpload v1.3.0 发布,Javascript 异步上传插件

AjaxUpload v1.3.0 已发布。 更新内容 修复BUpload组件,管理 API 的分页第一条数据获取不到的bug 分别给 JUpload, TUpload, BUpload 三个组件实现css的自动加载功能,不需要再手动引入css 修...

阳建 ⋅ 2017/09/15 ⋅ 5

重构随笔——重构的原则

重构的基本原则之一:不改变软件的可观察行为。 这一基本原则阐述的是我们在做重构的时候,维持外部的功能外观不变,让用户无法感知重构的变化。其实这很好理解,重构与添加新功能两者的角色...

人独立 ⋅ 2016/07/21 ⋅ 0

用SpringSource Tool Suite,出现奇怪错误,不妨删掉服务(如tomcat),再建

一直没动的项目,以前是没问题的,现在再跑,咦,跑不动了、 为啥? 反思: (1)代码没问题,因为我压根没改 (2)回想以前经常出现的问题,一一对照,并且做了很多尝试,依然无功而返 (3...

beanGou ⋅ 2014/03/24 ⋅ 0

blackfox/ajaxUpload

AjaxUpload @author:yangjian102621@gmail.com 插件描述: javascript异步上传插件,包含3个子项目BUpload, JUpload, TUpload. BUpload : 基于HTML5, UI仿百度编辑器的图片上传, 支持图片上...

blackfox ⋅ 2016/06/05 ⋅ 0

31天重构学习笔记24. 分解复杂判断

摘要:由于最近在做重构的项目,所以对重构又重新进行了一遍学习和整理,对31天重构最早接触是在2009年10月份,由于当时没有订阅Sean Chambers的blog,所以是在国外的社区上闲逛的时候链接过...

晨曦之光 ⋅ 2012/03/09 ⋅ 0

敏捷结果30天之第六天:周五回顾,找到三件做的好以及三件需要改善的事情

个人反思: 上周的计划执行只能给自己打70分,部分工作和学习计划执行都不到位!已经开始,本周继续努力提高 自己的执行力! 上周总结: 做的比较好的三件事: 1.跑步计划超额完成,本周继续...

长平狐 ⋅ 2013/06/17 ⋅ 0

2017年第47周总结

昨天睡得太早忘写了😰今天补起。 工作轻松 工作上比较轻松,主要就是重构购物车,顺便写了篇简书:如何处理一个tableView中同种model多种cell相同逻辑的情况? 刷街 周五晚上跟着几个小姐姐...

无夜之星辰 ⋅ 2017/11/21 ⋅ 0

没有更多内容

加载失败,请刷新页面

加载更多

下一页

volatile和synchronized的区别

volatile和synchronized的区别 在讲这个之前需要先了解下JMM(Java memory Model :java内存模型):并发过程中如何处理可见性、原子性、有序性的问题--建立JMM模型 详情请看:https://baike.b...

MarinJ_Shao ⋅ 34分钟前 ⋅ 0

深入分析Kubernetes Critical Pod(一)

Author: xidianwangtao@gmail.com 摘要:大家在部署Kubernetes集群AddOn组件的时候,经常会看到Annotation scheduler.alpha.kubernetes.io/critical-pod"="",以表示这是一个关键服务,那你知...

WaltonWang ⋅ 41分钟前 ⋅ 0

原子性 - synchronized关键词

原子性概念 原子性提供了程序的互斥操作,同一时刻只能有一个线程能对某块代码进行操作。 原子性的实现方式 在jdk中,原子性的实现方式主要分为: synchronized:关键词,它依赖于JVM,保证了同...

dotleo ⋅ 48分钟前 ⋅ 0

【2018.06.22学习笔记】【linux高级知识 14.4-15.3】

14.4 exportfs命令 14.5 NFS客户端问题 15.1 FTP介绍 15.2/15.3 使用vsftpd搭建ftp

lgsxp ⋅ 58分钟前 ⋅ 0

JeeSite 4.0 功能权限管理基础(Shiro)

Shiro是Apache的一个开源框架,是一个权限管理的框架,实现用户认证、用户授权等。 只要有用户参与一般都要有权限管理,权限管理实现对用户访问系统的控制,按照安全规则或者安全策略控制用户...

ThinkGem ⋅ 昨天 ⋅ 0

python f-string 字符串格式化

主要内容 从Python 3.6开始,f-string是格式化字符串的一种很好的新方法。与其他格式化方式相比,它们不仅更易读,更简洁,不易出错,而且速度更快! 在本文的最后,您将了解如何以及为什么今...

阿豪boy ⋅ 昨天 ⋅ 0

Python实现自动登录站点

如果我们想要实现自动登录,那么我们就需要能够驱动浏览器(比如谷歌浏览器)来实现操作,ChromeDriver 刚好能够帮助我们这一点(非谷歌浏览器的驱动有所不同)。 一、确认软件版本 首先我们...

blackfoxya ⋅ 昨天 ⋅ 0

线性回归原理和实现基本认识

一:介绍 定义:线性回归在假设特证满足线性关系,根据给定的训练数据训练一个模型,并用此模型进行预测。为了了解这个定义,我们先举个简单的例子;我们假设一个线性方程 Y=2x+1, x变量为商...

wangxuwei ⋅ 昨天 ⋅ 0

容器之查看minikue的environment——minikube的环境信息

执行如下命令 mjduan@mjduandeMacBook-Pro:~/Docker % minikube docker-envexport DOCKER_TLS_VERIFY="1"export DOCKER_HOST="tcp://192.168.99.100:2376"export DOCKER_CERT_PATH="/U......

汉斯-冯-拉特 ⋅ 昨天 ⋅ 0

mysql远程连接不上

设置了root所有hosts远程登录,可是远程登录还是失败,原因可能如下: 登录本地数据库 mysql -uroot -p123456 查询用户表 mysql> select user,host,password from mysql.user; 删除密码为空的...

冰公子 ⋅ 昨天 ⋅ 0

没有更多内容

加载失败,请刷新页面

加载更多

下一页

返回顶部
顶部