代码改变世界

记我的一次重构——希望对新人有所帮助

2013-06-03 11:07  麦舒  阅读(2479)  评论(28编辑  收藏  举报

看过博客园里几篇关于重构的文章,感觉都不怎么实在。下面我来谈谈自己的一次重构经历,希望对新人能有所帮助。

ALinq 这个产品维护了将近五年的时间,最近对它进行了一次重构。为什么要重构?主要是为了适应业务的发展需要。产品是服务于业务,而重构是服务于产品,归根到底,重构是服务于业务。所以我一直强调,好的架构不是想出来的,而是做来的,经验积累下来的。很多时候业务的发展,往往会超出你出初的预期,所以在产品的前期想设计出一个完美的架构是不可能的事。

这次的重构,出现了很多问题,一直陆续推出了好几个版本,才开始稳定下来,还好我的用户都是些忠实的用户,用ALinq好几年了,一直耐着性子,一个版本一个版本地安装,然后试用,真心感谢他们。

重构的代价,其实还是挺大的,所以在重构之前,尽可能多做一些工作,使得付出的成本能尽可能地降到最低。重构前要做工作有下面几个方面:

1、测试用例。这个是一定要的,并且,测试用例要全面。对于ALinq来说,就是单元测试+文档了。在重构完成后,先跑单元测试,然后再按着文档的描述,进行操作。界面的操作是很难写单元测试,即使可以,付出代价也太大了,远大于人工测试。

2、划分各个功能模块,每个模块的职责,也就是要做些什么。

3、提炼接口,这是必须的。

下面我们进一步谈谈为什么要进行重构,为了让大家能够更理解这次的重构,先来介绍简单一下产品的功能以及产品的发展线路。

现有功能

先来看一下图一,图一展示的功能主要有显示用户的授权信息、检查产品的更新,显示产品的信息。图二展示的功能有,将修改后的模型更新到数据库,将修改后的数据库更新到模型,就是模型与数据库的同步,如图三所示。

发展线路

1、增强用户的体验。主要有:

  1. 检查更新,然后再到网站下载,接着卸载旧版本,接着再装新的版本。我希望将来是一点菜单,就把一系列的动作全部完成。
  2. 授权信息,简单点说就是输入注册码。现在是,由于产品是一年内免费更新的,如果这一年内,产品升级了,或者验证码的算法改了,那么就得重新发送一次验证码到用户的邮箱,并且,用户还得保管好注册码,如果万一不记得了,还得找客服(客服就是我了,当然作者也是我)重新要回注册码。我希望是用户一点菜单,然后输入邮箱就可以自动取回相应版本的注册码,如果再输入密码,就能从网上下载注册码,然后自动完成注册。
  3. 向客户推送一些使用技巧的提示,或者文章。这是新的功能,菜单上没有。

关于用户体验要做的改进,现在想到的,暂时就是这些,当然,不限于这些。

2、数据库与模型的同步,这方面的功能现在太弱了。准备加强建模以及建库的功能,对于建模功能,未来有可能向PowerDesigner的方面发展。而建库的功能,就是能直接在界面上很方便对数据库进行修改。

那么现在面临的问题是什么呢?这部份代码是夹杂在视图模块里面的。而视图模块这部份的功能,是很稳定的了。视图模块,指的是把表从数据库里拖出来,生成实体类,或者从工具栏里,拖个图标出来,生成实体。如果不进行重构,易变的代码和稳定的代码交织在一起,那么易变代码的修改,势必会影响到已经相对稳定的模块。我对重构的理解是把变和不变的分开,对架构的理解是封装变化。

那么为什么会造成这种结果?

1、业务的发展

2、没能预期到业务的发展

话说,几年前的东西,怎么可能预测今天要弄些啥呢,哥是人,不是神。

到这里,明白为什么要重构了,以及重构的意义了吧。重构,不是你捧着本书,对着上面的教条,看哪不顺眼,就来耍耍。架构,不是你看了设计模式的书,然后依样画葫芦就弄得出来的了,而是必须依据业务的发展。是否重构,必须站在业务的层面去看待,而不仅是代码层面。如果系统很稳定,业务也很稳定,一般情况下是没有重构的必要的,哪怕代码写得再烂。

架构其实就是经验的积累,然后加上思考。当你的经验积累到一定的程度,架构自然就会有了。所以,建议刚入门程序员,应该多花时间代码的阅读、算法上。可别让那些重构、模式的书都误导了,随便看看就行了,不要花太多的时间。

如果我的这篇文章让你对重构有个更深的认识,那么我的目的也就达到了,谢谢阅读。读完请勿忘点击“推荐”,再次感谢。^_^

 

(图一)

 

(图二)

 

 

 

(图三)