梳理

最近在重构一段代码，发现之前的流程大概是如下图，这样的设计结构。

一、这是一个发送消息的流程，长长的一段代码，通过依赖构成整个流程的架构。整个流程依赖三个环节：适配、发送、保存，其中发送又依赖消息体生成。

二、由于消息类型不同，适配、发送、保存的具体实现不同，所以代码中大量的充斥着if ... else ...的判断语句。

这不仅导致代码可读性很差，而且还产生了大量冗余代码。

重构

针对如上问题，首先是通过组合的方式重新封装。

一、提取一个更高的抽象（Mesasge），这里组合了Sender和Saver两个行为，这里只组合行为，不组合实现。

这样的好处，是外观代码变得更加清晰，如下面代码：

Message message = adapterService.adapter(jsonMessage);
message.getSender().send();
message.getSaver().save();

之前和同事讨论这里，很形象的比如成，Sender像一张弓，Message像一支箭，getSender()可以获取不同的弓，Message也可以是不同的箭，通过不同的实现随意组合。

二、这里以服务号消息（ServiceNoMessage）为例，SamplerMessage是Message的一种具体实现，其中SamplerSender和SamplerSaver分别是Sender和Saver的具体实现。

通过组合的方式，就可以在不改变外观行为的框架下，构造针对不同特定的具体实现。

抽象封装Message，以及组合Sender和Saver可以参见如下代码：

public class SamplerMessage implements Message {

    private Sender sender;
    private Saver saver;

    private String jsonMessage;

    private MessageSenderService messageSenderService;
    private MessageBodyService messageBodyService;

    public SamplerMessage(String jsonMessage, MessageSenderService messageSenderService, MessageBodyService messageBodyService) {
        this.jsonMessage= jsonMessage;
        // 封装service到Message
        this.messageSenderService = messageSenderService;
        this.messageBodyService= messageBodyService;
    }

    public Sender getSender() throws Exception {
        sender = new SamplerSender(jsonMessage, messageSenderService, messageBodyService);
        return sender;
    }

    public Saver getSaver() {
        saver = new SamplerSaver(jsonMessage);
        return saver;
    }
}

优化

在重构中，提取一些共性的行为和属性，通过继承，在子类封装每个特性的行为和属性。

在实现MessageBodyService服务时，发现不同实现中，有着很多共性的一些，所以通过抽象类的方式，封装共性的，在具体的实现中，封装特性的。

以MessageBodyService为例，不同的实现都会调用初始init()方法，但是generate()会有所不同，如下图：

AbsMessageBodyService的抽象实现如下：

public abstract class AbsMessageBodyService implements MessageBodyService {
   
    public void init() throws Exception {
        ...
        generate();
        ...
    }

    protected abstract void generate() throws Exception;
}

ServiceNoMessageBodyServiceImpl的具体实现：

public class ServiceNoMessageBodyServiceImpl extends AbsMessageBodyService {

    protected String generate() throws Exception {
        ...
    }
}

MessageBody的调用：

public class SamplerMessageBody implements MessageBody {

    private MessageBodyService messageBodyService;

    public SamplerMessageBody() throws Exception {
        this.messageBodyService= new ServiceNoMessageBodyServiceImpl();

        initMessageBody();
    }

    private void initMessageBody() throws Exception {
        messageBodyService.init();
    }
}

本文受原创保护，未经作者授权，禁止转载。 linkedkeeper.com (文／张松然)  ©著作权归作者所有