Types

Author: Wazabii

Container.php

@@ -125,6 +125,7 @@ public function get(string $identifier, array $args = [])
             } else {
                 if (empty($this->getter[$identifier])) {
                     if (is_string($service)) {
+                        /** @var string $service */
                         $reflect = new Reflection($service);
                         if (!is_null($args)) {
                             $reflect->setArgs($args);
@@ -135,8 +136,6 @@ public function get(string $identifier, array $args = [])
                     }
                 }
             }
-
-
             return $this->getter[$identifier];
         } else {
             throw new NotFoundException("Tring to get a container ({$identifier}) that does not exists", 1);
@@ -158,7 +157,7 @@ public function fetch(string $identifier)
             $arr = Arr::value($this->services)->wildcardSearch($identifier)->get();
             if (count($arr) > 0) {
                 $new = array();
-                foreach ($arr as $key => $unusedValues) {
+                foreach ($arr as $key => $_unusedValues) {
                     $new[$key] = $this->get($key);
                 }
                 return $new;

Interfaces/ContainerInterface.php

@@ -12,25 +12,25 @@ interface ContainerInterface
     /**
      * Finds an entry of the container by its identifier and returns it.
      *
-     * @param string $id Identifier of the entry to look for.
+     * @param string $identifier Identifier of the entry to look for.
      *
      * @throws NotFoundExceptionInterface  No entry was found for **this** identifier.
      * @throws ContainerExceptionInterface Error while retrieving the entry.
      *
      * @return mixed Entry.
      */
-    public function get(string $id);
+    public function get(string $identifier);
 
     /**
      * Returns true if the container can return an entry for the given identifier.
      * Returns false otherwise.
      *
-     * `has($id)` returning true does not mean that `get($id)` will not throw an exception.
-     * It does however mean that `get($id)` will not throw a `NotFoundExceptionInterface`.
+     * `has($identifier)` returning true does not mean that `get($identifier)` will not throw an exception.
+     * It does however mean that `get($identifier)` will not throw a `NotFoundExceptionInterface`.
      *
-     * @param string $id Identifier of the entry to look for.
+     * @param string $identifier Identifier of the entry to look for.
      *
      * @return bool
      */
-    public function has(string $id): bool;
+    public function has(string $identifier): bool;
 }

Interfaces/FactoryInterface.php

@@ -12,11 +12,11 @@ interface FactoryInterface
     /**
      * Add a single factory.
      *
-     * @param string $id The id
+     * @param string $identifier The id
      * @param callable $factory The callable
      * @param bool $factory The callable
      * @param bool|boolean $overwrite Will throw exception if already been defined if not arg is set to TRUE.
      * @return void
      */
-    public function factory(string $id, callable $factory, bool $overwrite = false): ContainerInterface;
+    public function factory(string $identifier, callable $factory, bool $overwrite = false): ContainerInterface;
 }

Reflection.php

@@ -37,9 +37,10 @@ public function __construct($data)
     /**
      * If the dependency injector tries to read an interface in controller it
      * will search for the object in interfaceFactory.
-     * @return controller
+     * @param callable $call
+     * @return void
      */
-    public static function interfaceFactory($call): void
+    public static function interfaceFactory(callable $call): void
     {
         self::$interfaceFactory[] = function ($short, $class, $reflect) use ($call) {
             //self::$interfaceProtocol[$short] = $call($class, $short, $reflect);
@@ -59,9 +60,9 @@ public function allowInterfaces(bool $bool): void
 
     /**
      * Call dependency injector
-     * @return controller
+     * @return object
      */
-    public function dependencyInjector()
+    public function dependencyInjector(): object
     {
         $params = $this->reflect->getConstructor()->getParameters();
         $this->injectRecursion($params, $this->reflect->getName());
@@ -99,13 +100,14 @@ private function initReclusiveReflect(string $className, string $fromClass): Ref
 
     /**
      * Recursion inject dependancies
-     * @param  array  $params
-     * @param  array  $args
+     * @param  array    $params
+     * @param  string   $fromClass
+     * @param  array    $_args
      * @return array
      */
-    private function injectRecursion(array $params, string $fromClass, array $args = array())
+    private function injectRecursion(array $params, string $fromClass, array $_args = array()): array
     {
-        $args = array();
+        $_args = array();
         foreach ($params as $param) {
             if ($param->getType() && !$param->getType()->isBuiltin()) {
                 $classNameA = $param->getType()->getName();
@@ -118,23 +120,23 @@ private function injectRecursion(array $params, string $fromClass, array $args =
                 }
 
                 if (count($reflectParam) > 0) {
-                    $args = $this->injectRecursion($reflectParam, $inst->getName(), $args);
+                    $_args = $this->injectRecursion($reflectParam, $inst->getName(), $_args);
 
                     // Will make it posible to set same instance in multiple nested classes
-                    $args = $this->insertMultipleNestedClasses($inst, $constructor, $classNameA, $reflectParam);
+                    $_args = $this->insertMultipleNestedClasses($inst, $constructor, $classNameA, $reflectParam);
                 } else {
                     if ($inst->isInterface()) {
                         $this->insertInterfaceClasses($inst, $classNameA);
                     } else {
                         if (empty(self::$class[$classNameA])) {
-                            self::$class[$classNameA] = $this->newInstance($inst, (bool)$constructor, $args);
+                            self::$class[$classNameA] = $this->newInstance($inst, (bool)$constructor, $_args);
                         }
                     }
-                    $args[] = self::$class[$classNameA];
+                    $_args[] = self::$class[$classNameA];
                 }
             }
         }
-        return $args;
+        return $_args;
     }
 
     /**
@@ -252,9 +254,9 @@ public static function getClassList()
 
     /**
      * Load dependencyInjector / or just a container
-     * @return instance
+     * @return object
      */
-    private function getClass()
+    private function getClass(): object
     {
         if (!is_null($this->args)) {
             $inst = $this->reflect->newInstanceArgs($this->args);