refactor prototype pattern and move to its sub acme folder

basherr · basherr · commit 8917430f322e · 2020-12-09T20:19:16.000+05:00
diff --git a/app/Controller/CreationalController.php b/app/Controller/CreationalController.php
@@ -3,16 +3,7 @@
 namespace App\Controller;
 
 use App\Patterns\Creational\Pool\Worker\WorkerPool;
-use App\Patterns\Creational\FactoryMethod\CarManufacturer\GermanFactory;
-use App\Patterns\Creational\Prototype\IAcmePrototype;
-use App\Patterns\Creational\Prototype\Marketing;
-use App\Patterns\Creational\Prototype\Engineering;
-use App\Patterns\Creational\Prototype\Management;
 use App\Patterns\Creational\SimpleFactory\SimpleFactory;
-use App\Patterns\Creational\FactoryMethod\Logger\{
-	FileLoggerFactory,
-	StdLoggerFactory
-};
 
 class CreationalController
 {
@@ -85,45 +76,9 @@ public function reverseString()
 	* @param null
 	* @return null
 	*/
-	public function prototype()
+	public function acmePrototype()
 	{
-		$marketing = new Marketing();
-		$management = new Management();
-		$engineering = new Engineering();
-
-		$smith = clone $marketing;
-		$this->setEmployee( $smith, "Tess Smith", 101, "ts101-1234");
-		$this->showEmployee($smith);
-
-		$jacob = clone $marketing;
-		$this->setEmployee( $jacob,"Jacob Jones", 102,"jj101-2234");
-		$this->showEmployee($jacob);
-
-		$ricky = clone $management;
-		$this->setEmployee($ricky,"Ricky Rodriguez", 203,"rr203-5634");
-		$this->showEmployee($ricky);
-
-		$livia = clone $engineering;
-		$this->setEmployee($livia,"Olivia Perez", 302,"op301-1278");
-		$this->showEmployee($livia);
-
-		$jhon = clone $engineering;
-		$this->setEmployee($jhon,"John Jackson", 301, "jj302-1454");
-		$this->showEmployee($jhon);
-	}
-
-	private function showEmployee(IAcmePrototype $employee)
-	{
-		echo $employee->getName() . "<br/>";
-		echo $employee->getDept() . ": " . $employee::UNIT . "<br/>";
-		echo $employee->getID() . "<p/>";
-	}
-
-	private function setEmployee(IAcmePrototype $employee,$nm,$dp,$id)
-	{
-		$employee->setName($nm);
-		$employee->setDept($dp);
-		$employee->setID($id);
+		(new \App\Patterns\Creational\Prototype\AcmePrototype\Application)->render();	
 	}
 
 	public function simpleFactory()
diff --git a/app/Patterns/Creational/Prototype/AcmePrototype/Application.php b/app/Patterns/Creational/Prototype/AcmePrototype/Application.php
@@ -0,0 +1,45 @@
+<?php namespace App\Patterns\Creational\Prototype\AcmePrototype;
+
+class Application
+{
+    public function render()
+    {
+        $marketing = new Marketing();
+		$management = new Management();
+		$engineering = new Engineering();
+
+		$smith = clone $marketing;
+		$this->setEmployee( $smith, "Tess Smith", 101, "ts101-1234");
+		$this->showEmployee($smith);
+
+		$jacob = clone $marketing;
+		$this->setEmployee( $jacob,"Jacob Jones", 102,"jj101-2234");
+		$this->showEmployee($jacob);
+
+		$ricky = clone $management;
+		$this->setEmployee($ricky,"Ricky Rodriguez", 203,"rr203-5634");
+		$this->showEmployee($ricky);
+
+		$livia = clone $engineering;
+		$this->setEmployee($livia,"Olivia Perez", 302,"op301-1278");
+		$this->showEmployee($livia);
+
+		$jhon = clone $engineering;
+		$this->setEmployee($jhon,"John Jackson", 301, "jj302-1454");
+		$this->showEmployee($jhon);
+    }
+
+    private function showEmployee(IAcmePrototype $employee)
+	{
+		echo $employee->getName() . "<br/>";
+		echo $employee->getDept() . ": " . $employee::UNIT . "<br/>";
+		echo $employee->getID() . "<p/>";
+	}
+
+	private function setEmployee(IAcmePrototype $employee,$nm,$dp,$id)
+	{
+		$employee->setName($nm);
+		$employee->setDept($dp);
+		$employee->setID($id);
+	}
+}
diff --git a/app/Patterns/Creational/Prototype/AcmePrototype/Engineering.php b/app/Patterns/Creational/Prototype/AcmePrototype/Engineering.php
@@ -1,4 +1,4 @@
-<?php namespace App\Patterns\Creational\Prototype;
+<?php namespace App\Patterns\Creational\Prototype\AcmePrototype;
 
 class Engineering extends IAcmePrototype
 {
@@ -33,5 +33,5 @@ public function getDept() : string
         return $this->dept;
     }
 
-    public function __clone(){}
+    public function __clone() {}
 }
diff --git a/app/Patterns/Creational/Prototype/AcmePrototype/IAcmePrototype.php b/app/Patterns/Creational/Prototype/AcmePrototype/IAcmePrototype.php
@@ -1,4 +1,4 @@
-<?php namespace App\Patterns\Creational\Prototype;
+<?php namespace App\Patterns\Creational\Prototype\AcmePrototype;
 
 abstract class IAcmePrototype {
 	/**
diff --git a/app/Patterns/Creational/Prototype/AcmePrototype/Management.php b/app/Patterns/Creational/Prototype/AcmePrototype/Management.php
@@ -1,4 +1,4 @@
-<?php namespace App\Patterns\Creational\Prototype;
+<?php namespace App\Patterns\Creational\Prototype\AcmePrototype;
 
 class Management extends IAcmePrototype
 {
diff --git a/app/Patterns/Creational/Prototype/AcmePrototype/Marketing.php b/app/Patterns/Creational/Prototype/AcmePrototype/Marketing.php
@@ -1,4 +1,4 @@
-<?php namespace App\Patterns\Creational\Prototype;
+<?php namespace App\Patterns\Creational\Prototype\AcmePrototype;
 
 class Marketing extends IAcmePrototype {
 	const UNIT = "Marketing";
diff --git a/app/Patterns/Creational/Prototype/AcmePrototype/README.md b/app/Patterns/Creational/Prototype/AcmePrototype/README.md
@@ -0,0 +1,2 @@
+# Acme Prototype
+![Illustration](./illustration.png)
diff --git a/app/Patterns/Creational/Prototype/AcmePrototype/illustration.png b/app/Patterns/Creational/Prototype/AcmePrototype/illustration.png
diff --git a/bootstraps/routes.php b/bootstraps/routes.php
@@ -8,7 +8,7 @@
     ['GET', '/car-manufacturer-factory-method', ['App\Controller\CreationalController', 'carManufactureFactoryMethod']],
     ['GET', '/logger-factory-method', ['App\Controller\CreationalController', 'loggerFactoryMethod']],
     ['GET', '/pool-reverse-string', ['App\Controller\CreationalController', 'reverseString']],
-    ['GET', '/proto', ['App\Controller\CreationalController', 'prototype']],
+    ['GET', '/prototype-acme', ['App\Controller\CreationalController', 'acmePrototype']],
     ['GET', '/simple', ['App\Controller\CreationalController', 'simpleFactory']],
     ['GET', '/static', ['App\Controller\CreationalController', 'staticFactory']],
     ['GET', '/structural', ['App\Controller\StructuralController', 'adapterPattern']],
diff --git a/tests/AcmePrototypeTest.php b/tests/AcmePrototypeTest.php
@@ -0,0 +1,30 @@
+<?php
+
+use PHPUnit\Framework\TestCase;
+
+use App\Patterns\Creational\Prototype\AcmePrototype\IAcmePrototype;
+use App\Patterns\Creational\Prototype\AcmePrototype\Marketing;
+
+class AcmePrototypeTest extends TestCase {
+
+    public function testShouldCloneProperly()
+    {
+        $marketing = new Marketing();
+
+		$smith = clone $marketing;
+		$this->setEmployee( $smith, "Tess Smith", 101, "ts101-1234");
+		
+		$jacob = clone $marketing;
+		$this->setEmployee( $jacob,"Jacob Jones", 102,"jj101-2234");
+
+		$this->assertEquals("Jacob Jones", $jacob->getName());
+		$this->assertEquals("Tess Smith", $smith->getName());
+    }
+
+    private function setEmployee(IAcmePrototype $employee,$nm,$dp,$id)
+	{
+		$employee->setName($nm);
+		$employee->setDept($dp);
+		$employee->setID($id);
+	}
+}

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-<?php namespace App\Patterns\Creational\Prototype;`
	`1`	`+<?php namespace App\Patterns\Creational\Prototype\AcmePrototype;`
`2`	`2`
`3`	`3`	`class Engineering extends IAcmePrototype`
`4`	`4`	`{`
`@@ -33,5 +33,5 @@ public function getDept() : string`
`33`	`33`	`return $this->dept;`
`34`	`34`	`}`
`35`	`35`
`36`		`- public function __clone(){}`
	`36`	`+ public function __clone() {}`
`37`	`37`	`}`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+# Acme Prototype`
	`2`	`+![Illustration](./illustration.png)`