一、重构的原因
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消息映射函数;