【问题标题】:Usage of abstract class in the builder pattern?在构建器模式中使用抽象类?
【发布时间】:2016-12-06 08:24:45
【问题描述】:

我有两种来自上游的有效负载:PayloadAPayloadBPayloadAPayloadB 之间有一些公共字段,因此我使用这些公共字段创建了 Payload 类,其余的我为每个有效负载创建了两个构建器类。

下面是PayloadA 的构建器类:

public final class PayloadA {
  private final String clientId;
  private final String langid;
  private final String deviceId;
  private final Map<String, String> applicationPayload;

  private PayloadA(Builder builder) {
    this.clientId = builder.clientId;
    this.langid = builder.langid;
    this.deviceId = builder.deviceId;
    this.applicationPayload = builder.applicationPayload.build();
  }

  public static class Builder {
    protected final String deviceId;
    protected String clientId;
    protected String langid;
    protected ImmutableMap.Builder<String, String> applicationPayload = ImmutableMap.builder();

    public Builder(String deviceId) {
      this.deviceId = deviceId;
    }

    public Builder setClientId(String clientId) {
      this.clientId = clientId;
      return this;
    }

    public Builder setLangid(String langid) {
      this.langid = langid;
      return this;
    }

    public Builder setPayload(Map<String, String> payload) {
      this.applicationPayload.putAll(payload);
      return this;
    }

    public PayloadA build() {
      return new PayloadA(this);
    }
  }

    // getters and to string here
}

下面是PayloadB的类:

public final class PayloadB {
  private final String clientid;
  private final String type;
  private final String payId;

  private PayloadB(Builder builder) {
    this.clientid = builder.clientid;
    this.type = builder.type;
    this.payId = builder.payId;
  }

  public static class Builder {
    protected final String type;
    protected String payId;
    protected String clientid;

    public Builder(String type) {
      this.type = type;
    }

    public Builder setPayId(String payId) {
      this.payId = payId;
      return this;
    }

    public Builder setClientId(String clientid) {
      this.clientid = clientid;
      return this;
    }

    public PayloadB build() {
      return new PayloadB(this);
    }
  }

    // getters and to string here
}

现在我创建了另一个类 Payload 类(这必须是抽象类吗?),其中我有 PayloadAPayloadB 的所有公共字段,所以我必须将这些字段设置为不知何故,我不确定如何使用下面的课程:

public abstract class Payload {
  private long createTimestamp;
  private String partition;
  private String key;
  // some other fields here

  // getters and setters here
}

问题:

  • 现在假设如果我们从上游获得PayloadB,那么我希望Payload 类中的key 字段是PayloadB 类中type 的值,全部小写,如果我们从上游获取PayloadA,然后我希望密钥为world
  • 如果我们从上游获得PayloadB,并且如果设置了clientId,那么我希望partition 成为15,如果我们从上游获得PayloadA,并且如果设置了clientId,那么我希望partition 成为15,但如果它没有设置并且langId 在那里,那么我希望partition 成为17
  • 我还想设置createTimestamp,这是在构建有效负载对象后必须做的。因此,例如我已经构建了 PayloadA 对象,它将被传递给其他一些类,我需要在 PayloadA 对象上设置 createTimestamp 值。不知道该怎么做?我必须克隆一些东西吗?

如何在构建器模式中使用Payload 类?我将得到两个不同的有效载荷,它们之间的共同点很少,所以我将它们分开在一个抽象类中。

我应该使用包含所有内容的大型构建器模式类还是扩展某些东西的多个构建器模式?

【问题讨论】:

  • 首先,您为什么要使用构建器?看起来代码被过度设计了......
  • 一般来说,所有这些 Payload 都有很多字段。为了使示例更易于理解,我这样做了。
  • 好的,所以你介绍了构建器,因为类很大。您是否已经考虑过在较小的类中进行重构,从而解决问题的root?目前你的问题对我来说听起来像XYProblem:你问如何在构建器中使用抽象类(这是非常罕见的)而不是问自己“为什么我最终会出现这样一个奇怪的结构?”
  • hmmm 那么我应该如何解决这个问题呢?
  • 您需要PayloadX 的目的是什么?他们是 DTO 还是 BO ?它们实际上包含一些业务逻辑还是只是一堆吸气剂?您发布完整代码(至少所有属性)是否现实?

标签: java design-patterns fluent builder-pattern


【解决方案1】:
  • 我不会将builder 实例传递给PayloadX 构造函数。将值作为单独的构造函数参数传递或调用 setter。
  • 您可以定义Payload.Builder,它将包含PayloadAPayloadB 的公共字段。这个类将是一个抽象类,声明了一个抽象的build 方法。
  • PayloadA.BuilderPayloadB.Builder 将扩展 Payload.Builder,实现 build 方法。
  • 在这个build 方法中,您可以实现任何您需要创建的自定义逻辑,并设置PayloadX 的字段。

似乎你想让你的类不可变(顺便提一下applicationPayload)。在这种情况下,你不能真正“设置”任何东西。您只能生成一个新实例。有很多方法可以做到这一点,例如你可以实现PayloadX withTimestamp(...) 方法。或者您可以扩展您的构建以接受 PayloadX 并在那里设置时间戳,从而产生类似 new PayloadX.Builder(payloadXInstance).setTimestamp(...).build() 的内容。

【讨论】:

  • 你能举一个例子来说明你上面的所有建议吗?它将帮助我更好地理解。到目前为止,我很困惑如何使用您的建议。
  • @david 我不会,抱歉,这超出了我准备贡献的时间。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2016-04-25
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多