cs FactoryMethod

FactoryMethod/Bicycle.php

@@ -1,9 +1,5 @@
 <?php
 
-/*
- * DesignPatternPHP
- */
-
 namespace DesignPatterns\FactoryMethod;
 
 /**
@@ -11,10 +7,18 @@ namespace DesignPatterns\FactoryMethod;
  */
 class Bicycle implements Vehicle
 {
+    /**
+     * @var string
+     */
+    protected $color;
 
+    /**
+     * sets the color of the bicycle
+     *
+     * @param string $rgb
+     */
     public function setColor($rgb)
     {
-        
+        $this->color = $rgb;
     }
-
 }

FactoryMethod/FactoryMethod.php

@@ -1,9 +1,5 @@
 <?php
 
-/*
- * DesignPatternPHP
- */
-
 namespace DesignPatterns\FactoryMethod;
 
 /**
@@ -50,5 +46,4 @@ abstract class FactoryMethod
 
         return $obj;
     }
-
 }

FactoryMethod/Ferrari.php

@@ -1,9 +1,5 @@
 <?php
 
-/*
- * DesignPatternPHP
- */
-
 namespace DesignPatterns\FactoryMethod;
 
 /**
@@ -11,10 +7,16 @@ namespace DesignPatterns\FactoryMethod;
  */
 class Ferrari implements Vehicle
 {
+    /**
+     * @var string
+     */
+    protected $color;
 
+    /**
+     * @param string $rgb
+     */
     public function setColor($rgb)
     {
-        
+        $this->color = $rgb;
     }
-
 }

FactoryMethod/GermanFactory.php

@@ -1,39 +1,31 @@
 <?php
 
-/*
- * DesignPatternPHP
- */
-
 namespace DesignPatterns\FactoryMethod;
 
 /**
- * GermanFactory is vehicle factory in Germany
+ * GermanFactory is a vehicle factory in Germany
  */
 class GermanFactory extends FactoryMethod
 {
-
     /**
-     * @inheritdoc
+     * {@inheritdoc}
      */
     protected function createVehicle($type)
     {
         switch ($type) {
-
+            case parent::CHEAP:
-            case parent::CHEAP :
                 return new Bicycle();
                 break;
-
+            case parent::FAST:
-            case parent::FAST :
                 $obj = new Porsche();
                 // we can specialize the way we want some concrete Vehicle since
                 // we know the class
                 $obj->addTuningAMG();
+
                 return $obj;
                 break;
-
             default :
                 throw new \InvalidArgumentException("$type is not a valid vehicle");
         }
     }
-
 }

FactoryMethod/ItalianFactory.php

@@ -1,9 +1,5 @@
 <?php
 
-/*
- * DesignPatternPHP
- */
-
 namespace DesignPatterns\FactoryMethod;
 
 /**
@@ -11,25 +7,20 @@ namespace DesignPatterns\FactoryMethod;
  */
 class ItalianFactory extends FactoryMethod
 {
-
     /**
-     * @inheritdoc
+     * {@inheritdoc}
      */
     protected function createVehicle($type)
     {
         switch ($type) {
-
+            case parent::CHEAP:
-            case parent::CHEAP :
                 return new Bicycle();
                 break;
-
+            case parent::FAST:
-            case parent::FAST :
                 return new Ferrari();
                 break;
-
             default :
                 throw new \InvalidArgumentException("$type is not a valid vehicle");
         }
     }
-
 }

FactoryMethod/Porsche.php

@@ -1,9 +1,5 @@
 <?php
 
-/*
- * DesignPatternPHP
- */
-
 namespace DesignPatterns\FactoryMethod;
 
 /**
@@ -11,15 +7,25 @@ namespace DesignPatterns\FactoryMethod;
  */
 class Porsche implements Vehicle
 {
+    /**
+     * @var string
+     */
+    protected $color;
 
+    /**
+     * @param string $rgb
+     */
     public function setColor($rgb)
     {
-        
+        $this->color = $rgb;
     }
 
+    /**
+     * although tuning by AMG is only offered for Mercedes Cars,
+     * this is a valid coding example ...
+     */
     public function addTuningAMG()
     {
 
     }
-
 }

FactoryMethod/Vehicle.php

@@ -1,9 +1,5 @@
 <?php
 
-/*
- * DesignPatternPHP
- */
-
 namespace DesignPatterns\FactoryMethod;
 
 /**
@@ -11,6 +7,10 @@ namespace DesignPatterns\FactoryMethod;
  */
 interface Vehicle
 {
-
+    /**
+     * sets the color of the vehicle
+     *
+     * @param string $rgb
+     */
     function setColor($rgb);
 }