DesignPatternsPHP#226 fixed Builder

Author: Dominik Liebler

Creational/Builder/BikeBuilder.php

@@ -15,6 +15,7 @@ public function addDoors()
     {
         $this->car->setPart('rightDoor', new Parts\Door());
         $this->car->setPart('leftDoor', new Parts\Door());
+        $this->car->setPart('trunkLid', new Parts\Door());
     }
 
     public function addEngine()

Creational/Builder/CarBuilder.php

@@ -2,6 +2,6 @@
 
 namespace DesignPatterns\Creational\Builder\Parts;
 
-class Bike extends Vehicle
+class Truck extends Vehicle
 {
 }

Creational/Builder/Parts/Bike.php Creational/Builder/Parts/Truck.php

@@ -44,9 +44,9 @@ BuilderInterface.php
    :language: php
    :linenos:
 
-BikeBuilder.php
+TruckBuilder.php
 
-.. literalinclude:: BikeBuilder.php
+.. literalinclude:: TruckBuilder.php
    :language: php
    :linenos:
 
@@ -62,9 +62,9 @@ Parts/Vehicle.php
    :language: php
    :linenos:
 
-Parts/Bike.php
+Parts/Truck.php
 
-.. literalinclude:: Parts/Bike.php
+.. literalinclude:: Parts/Truck.php
    :language: php
    :linenos:
 

Creational/Builder/README.rst

@@ -2,18 +2,18 @@
 
 namespace DesignPatterns\Creational\Builder\Tests;
 
-use DesignPatterns\Creational\Builder\BikeBuilder;
+use DesignPatterns\Creational\Builder\TruckBuilder;
 use DesignPatterns\Creational\Builder\CarBuilder;
 use DesignPatterns\Creational\Builder\Director;
 
 class DirectorTest extends \PHPUnit_Framework_TestCase
 {
     public function testCanBuildBike()
     {
-        $bikeBuilder = new BikeBuilder();
+        $bikeBuilder = new TruckBuilder();
         $newVehicle = (new Director())->build($bikeBuilder);
 
-        $this->assertInstanceOf('DesignPatterns\Creational\Builder\Parts\Bike', $newVehicle);
+        $this->assertInstanceOf('DesignPatterns\Creational\Builder\Parts\Truck', $newVehicle);
     }
 
     public function testCanBuildCar()

Creational/Builder/Tests/DirectorTest.php

@@ -0,0 +1,44 @@
+<?php
+
+namespace DesignPatterns\Creational\Builder;
+
+use DesignPatterns\Creational\Builder\Parts\Vehicle;
+
+class TruckBuilder implements BuilderInterface
+{
+    /**
+     * @var Parts\Truck
+     */
+    private $truck;
+
+    public function addDoors()
+    {
+        $this->truck->setPart('rightDoor', new Parts\Door());
+        $this->truck->setPart('leftDoor', new Parts\Door());
+    }
+
+    public function addEngine()
+    {
+        $this->truck->setPart('truckEngine', new Parts\Engine());
+    }
+
+    public function addWheel()
+    {
+        $this->truck->setPart('wheel1', new Parts\Wheel());
+        $this->truck->setPart('wheel2', new Parts\Wheel());
+        $this->truck->setPart('wheel3', new Parts\Wheel());
+        $this->truck->setPart('wheel4', new Parts\Wheel());
+        $this->truck->setPart('wheel5', new Parts\Wheel());
+        $this->truck->setPart('wheel6', new Parts\Wheel());
+    }
+
+    public function createVehicle()
+    {
+        $this->truck = new Parts\Truck();
+    }
+
+    public function getVehicle(): Vehicle
+    {
+        return $this->truck;
+    }
+}