本节书摘来自华章计算机《Web前端开发最佳实践》一书中的第2章,第2.2节,作者:党 建 更多章节内容可以访问云栖社区“华章计算机”公众号查看。
2.2 前端代码重构
代码重构是业内经常讨论的一个热门话题。重构指的是在不改变代码外部行为的情况下进行源代码修改,重构之前需要考虑的是重构后如何才能保证外部行为不改变。对于后端代码来说,可以通过大量的自动化测试来确保重构后的代码逻辑,可对于普遍缺乏自动化测试的前端代码来说,重构之前一定要考虑再三才能下手。
我曾经有一次不算太成功的前端重构经历,所幸的是没有导致太大的问题,但教训是惨痛的。此次重构的项目本身没有足够的自动化测试,尤其是针对前端的自动化测试,其实在重构之前也预想到了重构的风险。先来介绍重构之前项目存在的问题:项目的前端代码没有统一的规范,不同人员开发的模块对应的代码风格完全不同,即使有相同的逻辑也是由完全不同的代码实现的;而且,不同模块的代码写到了相同的代码文件中,使得文件的代码量很大;项目的功能较多,前端代码量大,代码的文件也多,又由于需求变更频繁,因此代码中的无用代码量较大;网站首页加载时间过长,平均加载时间超过5秒;部分动态模块的加载会导致页面“假死”。总体来说,项目代码存在的问题就是维护性差和性能差,如果要持续地添加新功能,就有必要重构现有的代码。
鉴于项目代码的实际情况,并且考虑到缺乏足够的自动化测试,因此,我权衡再权衡,最终决定先尝试对前端的部分代码进行重构。重构的过程主要分为如下几个步骤。
1)删除无用代码,精简代码。无用的代码主要集中在CSS和Javascript文件中,包括已经不起作用的CSS样式和废弃的Javascript函数。这个重构过程是比较麻烦的,因为修改的过程需要小心翼翼,不停地查看函数和变量在上下文的调用关系,以免错删代码。总体来说,这个阶段的重构还是利大于弊,虽然后续发现了一些由于误删而导致的Bug,但是删除了废弃的代码,提高了后续的可维护性。
2)前端代码规范化。页面的HTML标签中还有大量的内联CSS样式,有些页面的head部分也有CSS样式,需要把这些CSS放到独立的文件中;调整代码的层次缩进格式,不同层级按照4个空格来缩进;更改标准已不推荐的标签,如
、等,改为由CSS样式控制;统一命名规则,这里主要涉及HTML中的id和class名称;另外,在Javascript中集中定义局部变量,并把部分全局变量转变为局部变量,缩小变量的作用域。
3)整理基础类库。网站初期为了加快开发的进度而引入了多个框架,其中包括Ext JS、jQuery以及多个jQuery插件。Ext JS和jQuery中重复的功能较多,项目中很多相同的实现使用的是不同类库中的功能,如DOM选择、AJAX请求等。部分jQuery的UI插件的功能也和Ext JS中的功能重复。鉴于项目的状况,只能保守选择同时保留Ext JS和jQuery。重构的内容是统一UI插件的使用、统一基础方法的使用。
4)前端代码模块化。按照模块归类CSS代码,放到模块对应的单独CSS文件中;按照模块分离Javascript代码,按照模块定义不同的命名空间;将Javascript代码中的公共方法归类到独立的共通文件中,同时引入面向对象的思想来重构Javascript代码,进一步明确公有接口和私有接口。
5)提高页面加载性能。将部分不影响首页展示的Javascript文件延迟到页面加载后加载;删除页面中初始隐藏的区域,改为通过Javascript按需动态生成;页面中的部分图片延迟加载;调整CSS和Javascript文件的引用顺序,即CSS在前Javascript在后;给静态文件设置缓存;使用CSS Sprite,合并首页背景图;合并和压缩发布后的CSS和Javascript代码文件。
从上面的重构内容可以看到,这次重构的目的是明确的,即提高前端代码的可维护性和性能。从中也暴露了重构过程中的一些问题:最主要的问题还是低估了重构的风险,导致后续发现了不少新Bug;没有足够的自动化测试,而冒然去修改代码,很难保证最终修改代码的正确性;其次是对于性能优化,事先没有量化,没有做任何性能方面的监控,没有抓住性能主要的瓶颈,所以性能部分的重构就显得比较盲目。
有了这次的重构实践经历,再加上学习业内的一些经验,我归纳了如下的前端重构最佳实践。 - 重构前一定要预估风险,如果没有足够的自动化测试,最好是先完善自动化测试代码。
- 重构的目的和范围要明确,切忌盲目修改。前端代码的重构目的主要是提高代码的可维护性、可读性和性能。
- 最好是先易后难,循序渐进。首先修改诸如命名、格式等不涉及具体逻辑的内容,然后考虑模块化和性能提升等与具体逻辑相关的内容。
- 重构过程中要持续测试,在多个浏览器中测试,确保重构的部分功能正确。切忌在大量重构后再进行测试,因为大量重构后基本很难记得重构的逻辑,也就有可能遗漏部分测试用例。
- 如果是性能提升,要事先检测网站的整体性能并量化,找出性能瓶颈。重构过程中要持续监控性能,并对比性能提升的效果。