最近在项目中的重构实践：代码重构的时机和步骤

一、重构的原因

1，一期功能已基本完成，二期开发时间较为充裕；

2，一期代码是基于一个已有老项目进行的，代码风格混乱，缺少注释，过期、冗余的代码和有效代码混在一起，可读性较差；一期开发时间有限，赶了进度，没有时间对老代码进行整改，新老代码交杂在一起，不利于后期继续开发；

3，二期功能需求已经较为明确，无论从逻辑上还是数据上都要比一起复杂的多，在一期代码基础上做不太容易；

 

二、重构的计划和步骤

1，统一代码风格：执行编码规范、去除冗余注释、增加必要注释、整理排版等；

2，函数级重构：提取公共函数、抽取内联函数、控制函数规模、调整临时变量/参数/返回值等；

3，对象级重构：梳理类对象结构，设计接口和继承，代码结构松耦合；

4，引入异步消息机制：提取功能模块，实现服务虚接口；增加异步任务队列，功能模块处理时只接收异步消息和抛出异步任务，进一步使得功能代码松散耦合；

 

三、重构记录

1，代码风格统一：

  # 所有头文件增加文件头注释、类名注释，统一数据成员和成员函数的命名规范（匈牙利命名法）、代码排版等；

  # 用常量代替宏，并将所有常量集中到Const.h头文件，统一命名风格；

  # 抽取公共函数如日志记录，集中到CommFunc类，并声明为static，更新原调用方式；

  # 提取所有全局变量至Global.h/cpp，去除原有的定义加extern的C风格引用；

2，函数级重构：

  # 用空对象（NULL OBJECT）替代空值判断；

  # 拆分大函数逻辑；

  # 用多态替代功能switch；

  # 函数内联；

3，对象级重构：

  # 删除无效或冗余的数据成员和代码，同时添加必要的注释；

  # 数据成员/成员函数的可见性控制整理；

  # 封装集合操作到类内部，提供访问接口；

  # 用代理模式（delegate）和唯一实例（singelon）替代原先的部分全局变量的处理；

  # 用组合和继承分化类功能，使之单一化；

4，异步机制：

  # 异步服务调度机制（后续发文详解）：维护一个全局异步任务队列，然后启动多个任务处理线程，当需要进行较为耗时或者跨模块的服务调用时，就抛出一个异步任务，放到任务队列里；多个任务处理线程分别从任务队列中取出待处理的任务，各自处理；程序退出时，清空队列和线程。

  # 异步服务分类：窗体消息和服务处理；

  # 功能模块初始化时注册自己能够处理的服务消息（SERVICE），窗体MESSAGE_MAP中定义自己能够处理的windows消息映射函数；