separate service providers to a new class

Author: harryolx

index.php

@@ -8,25 +8,43 @@
 
 require __DIR__ . '/vendor/autoload.php';
 
+/**
+ * instantiate the dependency injection $container object
+ */
 $container = new League\Container\Container;
 
-$serviceProviders = new \PhpBootstrap\ServiceProviders();
-$controllerProviders = new \PhpBootstrap\ControllerProviders();
-$container->addServiceProvider($serviceProviders);
-$container->addServiceProvider($controllerProviders);
+/**
+ * Register all service providers to $container
+ */
+\PhpBootstrap\ServiceProviders::register($container);
 
+/**
+ * instantiate $route object
+ */
 $route = new League\Route\RouteCollection($container);
 
+/**
+ * Register all routes mapping to $route object
+ */
 \PhpBootstrap\Routes::collections($route, $container);
 
 try {
+    /**
+     * process the request
+     */
     $response = $route->dispatch(
         $container->get('request'),
         $container->get(\PhpBootstrap\Contracts\Response::class)
     );
 } catch (\League\Route\Http\Exception\NotFoundException $exception) {
+    /**
+     * handle 404
+     */
     $response = new \PhpBootstrap\Services\Response();
     $response = $response->errorNotFound();
 }
 
+/**
+ * Display the response
+ */
 $container->get('emitter')->emit($response);

src/ControllerProviders.php renamed to src/Providers/Controller.php

@@ -2,11 +2,11 @@
 /**
  * Created by PhpStorm.
  * User: harry
- * Date: 2/13/18
- * Time: 11:53 AM
+ * Date: 2/15/18
+ * Time: 12:47 PM
  */
 
-namespace PhpBootstrap;
+namespace PhpBootstrap\Providers;
 
 use League\Container\ServiceProvider\AbstractServiceProvider;
 use PhpBootstrap\Controller\HelloWorld;
@@ -16,7 +16,7 @@
  * @package PhpBootstrap
  * Handle all your controller dependency injection here
  */
-class ControllerProviders extends AbstractServiceProvider
+class Controller extends AbstractServiceProvider
 {
     protected $provides = [
         'helloworldcontroller'

src/Providers/Library.php

@@ -0,0 +1,51 @@
+<?php
+/**
+ * Created by PhpStorm.
+ * User: harry
+ * Date: 2/15/18
+ * Time: 12:48 PM
+ */
+
+namespace PhpBootstrap\Providers;
+
+use League\Container\ServiceProvider\AbstractServiceProvider;
+use PhpBootstrap\Services\Hello;
+use PhpBootstrap\Contracts\Response as ResponseInterface;
+use Zend\Diactoros\ServerRequestFactory;
+use PhpBootstrap\Services\Response;
+
+/**
+ * Class ServiceProviders
+ * @package PhpBootstrap
+ * All your library/services need to be registered in the container
+ */
+class Library extends AbstractServiceProvider
+{
+    protected $provides = [
+        ResponseInterface::class,
+        'request',
+        'emitter',
+        \PhpBootstrap\Contracts\Hello::class
+    ];
+
+    public function register()
+    {
+        $this->getContainer()->share(ResponseInterface::class, Response::class);
+
+        $this->getContainer()->share(
+            'request',
+            function () {
+                return ServerRequestFactory::fromGlobals(
+                    $_SERVER, $_GET, $_POST, $_COOKIE, $_FILES
+                );
+            }
+        );
+
+        $this->getContainer()->share('emitter', \Zend\Diactoros\Response\SapiEmitter::class);
+
+        // by registering the helloworld implementation as an alias of it's interface it
+        // is easy to swap out for other implementations
+        $this->getContainer()
+            ->add(\PhpBootstrap\Contracts\Hello::class, Hello::class);
+    }
+}

src/Routes.php

@@ -10,7 +10,6 @@
 
 
 use League\Container\Container;
-use League\Route\Strategy\JsonStrategy;
 use PhpBootstrap\Middleware\ExampleMiddleware;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\ServerRequestInterface;

src/ServiceProviders.php

@@ -8,40 +8,17 @@
 
 namespace PhpBootstrap;
 
-use League\Container\ServiceProvider\AbstractServiceProvider;
-use PhpBootstrap\Services\Hello;
-use PhpBootstrap\Contracts\Response as ResponseInterface;
-use Zend\Diactoros\ServerRequestFactory;
-use PhpBootstrap\Services\Response;
+use League\Container\Container;
 
-
-class ServiceProviders extends AbstractServiceProvider
+class ServiceProviders
 {
-    protected $provides = [
-        ResponseInterface::class,
-        'request',
-        'emitter',
-        \PhpBootstrap\Contracts\Hello::class
-    ];
-
-    public function register()
+    /**
+     * Register all service provider here
+     * @param Container $container
+     */
+    public static function register(Container $container)
     {
-        $this->getContainer()->share(ResponseInterface::class, Response::class);
-
-        $this->getContainer()->share(
-            'request',
-            function () {
-                return ServerRequestFactory::fromGlobals(
-                    $_SERVER, $_GET, $_POST, $_COOKIE, $_FILES
-                );
-            }
-        );
-
-        $this->getContainer()->share('emitter', \Zend\Diactoros\Response\SapiEmitter::class);
-
-        // by registering the helloworld implementation as an alias of it's interface it
-        // is easy to swap out for other implementations
-        $this->getContainer()
-            ->add(\PhpBootstrap\Contracts\Hello::class, Hello::class);
+        $container->addServiceProvider(new \PhpBootstrap\Providers\Library());
+        $container->addServiceProvider(new \PhpBootstrap\Providers\Controller());
     }
 }