3

我有一个构建器模式，其中很可能我的所有参数都是强制性的，因此我创建了一个长构造器，如下面的代码所示。

public final class ResponseHolder {

    // all below six are related to response information
    private final String response;
    private final boolean isLinking;
    private final TypeHold typeOfId;
    private final long userTimeInMs;
    private final long userLmdInDays;
    private final String maskInfo;

    // below two are related to error handling
    private final ErrorCode error;
    private final StatusCode status;

    private ResponseHolder(Builder builder) {
        this.response = builder.response;
        this.isLinking = builder.isLinking;
        this.typeOfId = builder.typeOfId;
        this.userTimeInMs = builder.userTimeInMs;
        this.userLmdInDays = builder.userLmdInDays;
        this.maskInfo = builder.maskInfo;
        this.error = builder.error;
        this.status = builder.status;
    }

    public static class Builder {
        protected final String response;
        protected final TypeHold typeOfId;
        protected final String maskInfo;
        protected final ErrorCode error;
        protected final StatusCode status;
        protected final boolean isLinking;
        protected final long userTimeInMs;
        protected final long userLmdInDays;


        public Builder(String response, TypeHold typeOfId, String maskInfo, ErrorCode error,
                StatusCode status, boolean isLinking, long userTimeInMs, long userLmdInDays) {
            this.response = response;
            this.typeOfId = typeOfId;
            this.maskInfo = maskInfo;
            this.error = error;
            this.status = status;
            this.isLinking = isLinking;
            this.userTimeInMs = userTimeInMs;
            this.userLmdInDays = userLmdInDays
        }

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

    // getters here
}

现在我很困惑，当所有参数都是强制性的，那么它有什么用呢？有没有更好的方法来表示我上面的 Builder 模式？可能在逻辑上将传递给它们自己的类的参数分组，以减少传递给构建器构造函数的参数数量？

虽然拥有单独的对象可以大大简化事情，但如果不熟悉代码，也会使事情变得有点难以理解。我可以做的一件事是将所有参数移动到它们自己的addParam(param)方法中，然后build()在运行时对方法中所需的参数执行验证？

我应该在这里遵循什么最佳实践，我可以在这里使用什么更好的方法？

4

2 回答 2

8

当有许多不同的有效参数排列允许您创建对象时，Builder 模式就会大放异彩。如果没有 Builder 模式，您将被迫创建许多丑陋且令人困惑的构造函数来处理所有可能的有效参数组合。

但是在您的情况下，只有一组有效的参数可以让您创建对象。这正是构造函数的用途。在这里使用 Builder 模式不仅大材小用，而且根本不合适。

ResponseHolder只需为您的类使用普通的构造函数。

于 2015-12-26T05:18:33.027 回答
6

构建器模式的目的是在构造目标对象之前拥有一个无参数构造函数、许多命名良好的 setter 方法和一个最终完成方法，该方法验证给定值的组合是否有效。

在您的情况下，由于所有值都是必需的，因此构建器模式的主要目的是提供一种命名参数支持形式。

此外，您的目标对象应该具有多参数构造函数，而不是将构建器对象作为参数）

因此，您的构建器应该是（假设不允许空值）：

public static class Builder {
    private String     response;
    private TypeHold   typeOfId;
    private String     maskInfo;
    private ErrorCode  error;
    private StatusCode status;
    private Boolean    isLinking;
    private Long       userTimeInMs;
    private Long       userLmdInDays;

    public Builder setResponse(String response) {
        this.response = response;
        return this;
    }
    public Builder setTypeOfId(TypeHold typeOfId) {
        this.typeOfId = typeOfId;
        return this;
    }
    public Builder setMaskInfo(String maskInfo) {
        this.maskInfo = maskInfo;
        return this;
    }
    public Builder setError(ErrorCode error) {
        this.error = error;
        return this;
    }
    public Builder setStatus(StatusCode status) {
        this.status = status;
        return this;
    }
    public Builder setIsLinking(boolean isLinking) {
        this.isLinking = isLinking;
        return this;
    }
    public Builder setUserTimeInMs(long userTimeInMs) {
        this.userTimeInMs = userTimeInMs;
        return this;
    }
    public Builder setUserLmdInDays(long userLmdInDays) {
        this.userLmdInDays = userLmdInDays;
        return this;
    }
    public ResponseHolder build() {
        if (this.response == null ||
            this.typeOfId == null ||
            this.maskInfo == null ||
            this.error == null ||
            this.status == null ||
            this.isLinking == null ||
            this.userTimeInMs == null ||
            this.userLmdInDays == null) {
            throw new IllegalStateException("Not all required values given");
        }
        return new ResponseHolder(this.response,
                                  this.typeOfId,
                                  this.maskInfo,
                                  this.error,
                                  this.status,
                                  this.isLinking,
                                  this.userTimeInMs,
                                  this.userLmdInDays);
    }
}

您现在可以像这样使用它：

ResponseHolder holder = new ResponseHolder.Builder()
                        .setResponse(response)
                        .setTypeOfId(typeOfId)
                        .setMaskInfo(maskInfo)
                        .setError(error)
                        .setStatus(status)
                        .setIsLinking(isLinking)
                        .setUserTimeInMs(userTimeInMs)
                        .setUserLmdInDays(userLmdInDays)
                        .build();
于 2015-12-26T05:27:41.053 回答