重构 Android

移动应用遗留系统重构（4）-分析篇

前言

上一篇移动应用遗留系统重构（3）-示例篇我们介绍了CloudDisk的业务及代码现状。分享了“理想”（未来的架构设计）与“现实”（目前的代码现状），接下来在我们开始动手进行重构时，我们首先得知道往理想的设计架构演化，中间存在多少问题。一方面作为开始重构的输入，另外一方面我们有数据指标，也能更好评估工作量及衡量进度。

接下来我们将根据架构篇团队采用的架构设计，结合目前的代码，总结分析工具及方法。

架构设计

我们先回忆一下架构篇里团队采用的架构设计。

代码复用

公共能力复用，有一层专门统一管理应用公用的基础能力，如图片、网络、存储能力、安全等
公用业务能力复用，有一层专门统一管理应用的业务通用组件，如分享、推送、登录等

低耦合

业务模块间通过API方式依赖，不依赖具体的模块实现
依赖方向清晰，上层模块依赖下层模块

并行研发

业务模块支持独立编译调试
业务模块独立发布

结合该4层架构、已有的代码，以及业务的后续演化，团队设计的新架构如下

分析工具

ArchUnit

有了架构设计后，我们就能识别代码的边界，这里我们可以通过Archunit进行边界约束描述。我们可以得到2条通用的守护规则。

垂直方向，下层模块不能反向依赖上层
横向方向，组件之间不能存在相互的依赖

转化为ArchUnit的测试用例如下:

  @ArchTest
public static final ArchRule architecture_layer_should_has_right_dependency =layeredArchitecture()

            .layer("Library").definedBy("..cloud.disk.library..")

            .layer("PlatForm").definedBy("..cloud.disk.platform..")

            .layer("FileBundle").definedBy("..cloud.disk.bundle.file..")

            .layer("DynamicBundle").definedBy("..cloud.disk.bundle.dynamic..")

            .layer("UserBundle").definedBy("..cloud.disk.bundle.user..")

            .layer("AllBundle").definedBy("..cloud.disk.bundle..")

            .layer("App").definedBy("..cloud.disk.app..")

            .whereLayer("App").mayOnlyBeAccessedByLayers()

            .whereLayer("FileBundle").mayOnlyBeAccessedByLayers("App")

            .whereLayer("DynamicBundle").mayOnlyBeAccessedByLayers("App")

            .whereLayer("UserBundle").mayOnlyBeAccessedByLayers("App")

            .whereLayer("PlatForm").mayOnlyBeAccessedByLayers("App","AllBundle")

            .whereLayer("Library").mayOnlyBeAccessedByLayers("App","AllBundle","PlatForm");

复制代码