从重构到吐血 - 我是如何删掉 6 万行代码并且不删减原有功能的

主要重构工作如下

删除没用到的第三方库
删除不合理的第三方库，使用系统自带的或者自己造轮子
删除定义好但是没有用到的变量
删除 import 进来但是没有用到的头文件
删除更旧项目留下来的用不到的逻辑
Controller 层不合理的层级结构重构，无用代码清理
View 层不合理的结构重构
Service 层冗余的写法重构
Model 层不合理的写法重构
拆开不合理的耦合
耦合一个类别的模块
修复了多处内存泄露
修复了多处循环引用
优化编译速度
消除项目中的 warning

关于删除代码，在某个项目里，Pods 文件夹那些第三方库的代码删了 9 万多行（那个目录没有被 git ignore 掉），项目里面删除了大约 4 万行，其中大量代码是该项目之前的项目里面留下来的东西，只不过没人清理。在删了 4 万行之后，程序仍然能完整的跑。

接下来是做了部分重构，把一些第三方库删掉，自己造轮子，在这个过程中，累计删除了 1.2 万行代码，增加了 1100 行左右。

整个重构工作下来，编译速度从 2-3 分钟减小到了 40 多秒（ 13 寸 2 核心机器），warning 从 70 多减少到了 0，第三方库的数量从 51 个减少到了 13 个，安装包从 22.1M 减小到了 3.7M ，功能反而比之前还要多。

原文发表在我的博客近期重构工作的一点收获

pupboss

2017-12-10 16:56:16 +08:00

@WangYanjie 不是很会用单元测试，所以靠的是人肉 debug，其实如果遵循规范的话，比如布局全部用 AutoLayout，基本上不会出问题。测试的过程中发现部分 UI 不对劲，是因为那个 UITableVIew 的 cell，没有用 AutoLayout 算高度。

功能方面，3.7M 的安装包已经很能说明问题了，一共就 20 多个页面，不是像淘宝那样复杂，重构的过程又是基本相当于重写，一个个页面挨着测过一遍，功能就很正确了，没有很复杂的页面，最多也就是下单页面动态加载用户优惠券这个级别的

pupboss

2017-12-10 17:22:57 +08:00

@Loyalsoldier 有的，我是基于 https://github.com/kelyvin/caffeine-theme 修改，因为修改的项目很多，和我自己耦合太强不适合分享，你可以基于他这个再定制你自己的，这是我当初优化的点供参考：[本博客优化点总结]( https://www.pupboss.com/summary-of-my-blog-optimization/)

pupboss

2017-12-10 17:39:33 +08:00

@firefox12 哈哈，多谢前辈指教，重构的绝大部分还是成功的吧，因为每次重构，都是只针对一个模块，不会说一下把所有东西都重写一遍，再就是可能前辈公司的业务复杂度比较高，就我们这个项目而言，登录注册录音识别查看商品下单收藏，功能还是比较单一的，所以会出问题的地方不多

@jpmorn Go lang 还有这么个功能，下周我也会去看看哈哈哈哈

@RorschachZZZ 对的，我在上家公司，页面的业务非常复杂，重构过一个页面，感觉也是提心吊胆，重构完一个页面，有别处又用到这里面一个 View，一般都会出布局错误😂

这是一个专为移动设备优化的页面（即为了让你能够在 Google 搜索结果里秒开这个页面），如果你希望参与 V2EX 社区的讨论，你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/413522