2

我正在使用构建器模式构建一个 JavaBean(只有字段和 getter/setter)。

为了这个例子,假设这是我们的 bean:

public class Pizza {
  private int size;
  private boolean cheese;
  private boolean pepperoni;
  private boolean bacon;

  private Pizza(Builder builder) {
    size = builder.size;
    cheese = builder.cheese;
    pepperoni = builder.pepperoni;
    bacon = builder.bacon;
  }

  public static class Builder {
    //required
    private final int size;

    //optional
    private boolean cheese = false;
    private boolean pepperoni = false;
    private boolean bacon = false;

    public Builder(int size) {
      this.size = size;
    }

    public Builder cheese(boolean value) {
      cheese = value;
      return this;
    }

    public Builder pepperoni(boolean value) {
      pepperoni = value;
      return this;
    }

    public Builder bacon(boolean value) {
      bacon = value;
      return this;
    }

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

取自这里

现在我一直在尝试确保其中的所有字段Pizza都是非空的,通过反射,遍历字段Pizza并检查它们不为空,但看起来(我在这里可能是错的)我的字段是'在检查发生之前不设置。Jon Skeet 的这段代码是我为了检查我的字段的非空性而改变的(而不是计数,我抛出异常)。

然后我尝试检查构建器的字段,但构建器中有额外的字段(例如,我有一个可能为空的 XMLParser 字段)。通过比萨字段对构建器字段进行子集化不起作用,因为它们具有不同的“包路径”(?),例如vsorg.GiusepesPizzaria.pizza.size org.GiusepesPizzaria.builder.size

有没有更好的方法来检查这个?在实现反射方法之前,我使用了这种构造:

if(builder.size ==null){
    throw new BadPizzaException("Eh, what're ya doin'?"+
             " Pizza Size was not set correctly");
}else{
    size=builder.size;
}

但它最终,如果你说要检查大约 10 个字段,冗长的,杂乱无章的应该是一个简单的类。

所以这就是我尝试过的。有没有更好的方法?

4

2 回答 2

2

确保设置所有变量的有趣模式是使用Step Builder 模式,其中第一个设置器只允许您设置第二个,第二个只允许第三个,依此类推。当您在最后一步时,您可以构建类,到那时您将知道所有方法都已被调用。

该帖子的简短摘录:

Panino solePanino = PaninoStepBuilder.newBuilder()
        .paninoCalled("sole panino")
        .breadType("baguette")
        .fish("sole")
        .addVegetable("tomato")
        .addVegetable("lettece")
        .noMoreVegetablesPlease()
        .build();

您必须从 panino 的名称开始,然后使用面包类型。

于 2013-05-15T17:55:51.730 回答
0

尝试这个:

public class Pizza
{
    private final boolean bacon;
    private final boolean cheese;
    private final boolean pepperoni;
    private final int size;

    private Pizza()
    {
        throw new UnsupportedOperationException();
    }

    Pizza(
        final int theSize,
        final boolean theCheese,
        final boolean thePepperoni,
        final boolean theBacon)
    {
        bacon = theBacon;
        cheese = theCheese;
        pepperoni = thePepperoni;
        size = theSize;
    }
}

// new file.
public class PizzaBuilder
{
    private boolean bacon;
    private boolean cheese;
    private boolean pepperoni;
    private int size;

    public PizzaBuilder()
    {
        size = 9; // default size.
    }

    public void setHasBacon()
    {
        bacon = true;
    }

    public void setHasNoBacon()
    {
        bacon = false;
    }

    public void setHasCheese()
    {
        cheese = true;
    }

    public void setHasNoCheese()
    {
        cheese = false;
    }

    public void setHasPepperoni()
    {
        pepperoni = true;
    }

    public void setHasNoPepperoni()
    {
        pepperoni = false;
    }

    public void setSizeNineInch()
    {
        size = 9;
    }

    public void setSizeTwelveInch()
    {
        size = 12;
    }

    public Pizza buildPizza()
    {
        return new Pizza(size, cheese, pepperoni, bacon);
    }
}

使用上面的构建器,构建器不可能生产出无效的比萨饼。

假设:仅支持 9 和 12 英寸的比萨饼。根据需要添加更多 setSize。

构建器使用我称为 NMSeters 的东西。此样式设置器允许您设置值,但不公开所述值的实现。这似乎不是我的原创发明。

于 2013-05-15T17:26:45.617 回答