Builder Pattern implementieren

[Jasmin589]

Hallo ich habe noch eine Frage dazu wie ich das Builder Pattern implementieren soll.

Das UML Diagramm dazu ist im Anhang.

Ich habe nun folgenden Code geschrieben:

public class Builder implements Director {

public buildPart () {}
}

Public Interface Director {

Public ConstructObjects () {}

Public class concreteBuilder 1 extends Builder {

public buildPart() {}

public concreteBuilder1 getResult Product () { }

Public class concreteBuilder2 extends Builder {

public buildPart() {}

public concreteBuilder2 getResult Product () { }

Hab ich da jetzt Fehler drin?

 

[Hutzli]

Hey

Also der Builder implementiert den Director nicht. Das ist eine Aggregation, was bedeutet, dass nach deinem UML der Builder ein Teil des Directors ist, aber selbst auch ausserhalb des Directors existieren kann. Lies diesen Beitrag durch: http://www.ibm.com/developerworks/rational/library/content/RationalEdge/sep04/bell/

Bei sämtlichen Methoden fehlen weiterhin die Rückgabewert, und public ist ein Schlüsselwort und wird immer klein geschrieben.

Ich habe mich beim folgenden Beispiel nicht zu 100% an dein UML gehalten, das, was ich hier schreibe, macht für
mich am meisten Sinn.

// Director
public class Director
{
   private Builder mBuilder1;
   private Builder mBuilder2;

   public Director()
   {
      mBuilder1 = new ConcreteBuilder1();
      mBuilder2 = new ConcreteBuilder2();
   }

   public Product constructObjects1()
   {
      mBuilder1.buildPart();
      return mBuilder1.getResult();
   }

   public Product constructObjects2()
   {
      mBuilder2.buildPart();
      return mBuilder2.getResult();
   }
}

// Builder
public abstract class Builder
{
      private Product mProduct;

      public abstract void buildPart();

      // Ich wuerde hier getResult implementieren,
      // ansonsten muesste man im Director casten.
      public Product getResult()
      {
         return mProduct;
      }

      // Uebergabe des Products
      protected void setProduct( Product pProduct )
      {
         this.mProduct = pProduct;
      }
}

// Concrete Builder 1
public class ConcreteBuilder1 extends Builder
{
   @Override
   public void buildPart()
   {
      Product lProd = new Product();
      // Kreiere Product

      super.setProduct( lProd );
   }
}

// Concrete Builder 2
public class ConcreteBuilder2 extends Builder
{
   @Override
   public void buildPart()
   {
      Product lProd = new Product();
      // Kreiere Product

      super.setProduct( lProd );
   }
}

// Product
public class Product()
{
   // Irgendwelche member sowie setter und getter
}

Ich hoffe, dass ich dir ein wenig helfen konnte,

Hutzli

 

[Jasmin589]

Oh stimmt, die Rückgabewerte habe ich vergessen. Genauso hatte ich die Pfeile vertauscht der Interface Pfeil ist ja so gestrichelt und nicht durchgezogen. Das mit der klein schreibung trainiere ich mir an, aber die Gewohnheit am Anfang Groß zu schreiben überwiegt noch. Zum Glück meckert dann immer Eclipse und ich seh das dann.

Ich versteh nur nicht ganz wie ich dann diese Aggregation in Java implementiere.

 

[Hutzli]

Ich versteh nur nicht ganz wie ich dann diese Aggregation in Java implementiere.

Ja, dazu habe ich einen kleinen Fehler gemacht.
Bei meinem Bespiel ist es eine Komposition, da gar keine Möglichkeit besteht, auf den Builder zuzugreifen.
Das heisst, der Builder lebt und stirbt mit dem Objekt Director.
Eine Aggregation sieht folgendermassen aus:

public class Director
{
   private Builder mBuilder;

   public Director()
   {
      // Hier koenntest du einen Default Builder
      // instanziieren.
      this.mBuilder = new ConcreteBuilder1();
   }

   public Director( Builder pBuilder )
   {
      this.mBuilder = pBuilder;
   }

   public void setBuilder( Builder pBuilder )
   {
      this.mBuilder = pBuilder;
   }

   public Builder getBuilder()
   {
      return mBuilder;
   }

   public Product constructObjects()
   {
      mBuilder.buildPart();
      return mBuilder.getResult();
   }
}

Für eine bessere Erklärung empfehle ich dir, den folgenden Thread durchzulesen:
http://stackoverflow.com/a/12431359