使用没有内部静态类的流畅的构建器模式进行工作流

debugcn 发表于 Dev

spakai

我对此有很好的反馈，我相信我已经改进了设计。整个代码在https://github.com/spakai/flow_input_builder

要求很简单：-

我需要使用以前的工作流程中的一个或多个输出来为不同的工作流程构建一组输入

我有一组接口

public interface SwfInput {

}

public interface SwfOutput {

}

public interface Workflow<I extends SwfInput, O extends SwfOutput> {

    public O execute(I input);
}

public interface Builder<I extends SwfInput> {
    public I build();

}

现在，假设我有3个流，它们按顺序FlowA-> FlowB-> FlowC执行

FlowC需要FlowB的强制输出，但仅可选地需要FlowA

所以我有一个FlowCBuilder的实现

public class FlowCInputBuilder implements Builder<FlowCInput> {

    private final FlowBOutput mandatoryflowBOutput;
    private FlowAOutput optionalflowAOutput;

    public FlowAOutput getOptionalflowAOutput() {
        return optionalflowAOutput;
    }

    public FlowCInputBuilder setOptionalflowAOutput(FlowAOutput optionalflowAOutput) {
        this.optionalflowAOutput = optionalflowAOutput;
        return this;
    }

    public FlowCInputBuilder(FlowBOutput mandatoryflowBOutput) {
        this.mandatoryflowBOutput = mandatoryflowBOutput;
    }

    @Override
    public FlowCInput build() {

        FlowCInput input = new FlowCInput();
        input.setMandatoryFromFlowB(mandatoryflowBOutput.getOutput1FromB());

        if (optionalflowAOutput != null) {
            input.setOptionalFromFlowA(optionalflowAOutput.getOutput2FromA());
        }

        return input;       

    }


}

我写的一个测试显示了一个示例用法

 FlowBOutput mandatoryflowBOutput = new FlowBOutput();

 mandatoryflowBOutput.setOutput1FromB("iNeedThis");
 FlowAOutput optionalflowAOutput = new FlowAOutput();
 FlowCInput input =  new FlowCInputBuilder(mandatoryflowBOutput)
                .setOptionalflowAOutput(optionalflowAOutput)
                .build();

我没有为Builder模式使用静态内部类。

任何建议都欢迎。

Nghia Bui

您应该使用静态内部类。使用这种方法的关键在于，内部可以直接访问正在构造的对象的私有属性。这有助于消除重复的代码，因为构建器不需要维护一长串的临时状态。因此，您的代码可以像这样重写：

public class FlowCInput {
    private int output1FromB; // suppose that it is int
    private String output2FromA; // suppose that it is String
    private FlowCInput() { }
    //...

    public static class FlowCInputBuilder implements Builder<FlowCInput> {
        private final FlowCInput result;

        public FlowCInputBuilder(FlowBOutput mandatoryflowBOutput) {
            result = new FlowCInput();
            // output1FromB is private but still accessed from here
            result.output1FromB = mandatoryflowBOutput.getOutput1FromB();
        }

        public FlowCInputBuilder setOptionalflowAOutput(FlowAOutput optionalflowAOutput) {
            // same for output2FromA
            result.output2FromA = optionalflowAOutput.getOutput2FromA();
            return this;
        }

        @Override
        public FlowCInput build() {
            return result;
        }
    }
}

正如你看到的，建筑商现在只保存FlowCInput对象时，它不会不必要地持有mandatoryflowBOutput和optionalflowAOutput以前一样。

本文收集自互联网，转载请注明来源。

如有侵权，请联系[email protected] 删除。