Builder::generateConditions() - Code Metrics - Inspection of "Separated Builder to just build from classMetadata" - samsonframework/container - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4bcb6e...27d771 )

by Vitaly

created 2016-08-18 11:43 UTC

Builder::generateConditions() C

↳ Parent: Builder

Complexity

Conditions	12
Paths	33

Size

Total Lines	87
Code Lines	44

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	39
CRAP Score	12

Importance

Changes

Metric	Value
dl	0
loc	87
ccs	39
cts	39
cp	1
rs	5.034
c	0
b	0
f	0
cc	12
eloc	44
nc	33
nop	2
crap	12

How to fix Long Method Complexity

<?php declare(strict_types = 1);
/**
 * Created by PhpStorm.
 * User: root
 * Date: 02.08.16
 * Time: 0:46.
 */
namespace samsonframework\container;

use samsonframework\container\metadata\ClassMetadata;
use samsonframework\container\metadata\MethodMetadata;
use samsonframework\container\metadata\PropertyMetadata;
use samsonframework\di\Container;
use samsonphp\generator\Generator;

/**
 * Container builder.
 *
 * @author Vitaly Egorov <[email protected]>
 */
class Builder
{
    /** Controller classes scope name */
    const SCOPE_CONTROLLER = 'controllers';

    /** Service classes scope name */
    const SCOPE_SERVICES = 'services';

    /** Generated resolving function name prefix */
    const DI_FUNCTION_PREFIX = 'container';

    /** Generated resolving function service static collection name */
    const DI_FUNCTION_SERVICES = '$' . self::SCOPE_SERVICES;

    /** @var string[] Collection of available container scopes */
    protected $scopes = [
        self::SCOPE_CONTROLLER => [],
        self::SCOPE_SERVICES => []
    ];

    /** @var ClassMetadata[] Collection of classes metadata */
    protected $classMetadata = [];

    /** @var array Collection of dependencies aliases */
    protected $classAliases = [];

    /**
     * @var Generator
     * @Injectable
     */
    protected $generator;

    /** @var string Resolver function name */
    protected $resolverFunction;

    /**
     * Container builder constructor.
     *
     * @param Generator       $generator     PHP code generator
     * @param ClassMetadata[] $classMetadata Collection of classes metadata for container
     */
    public function __construct(Generator $generator, array $classMetadata)
    {
        $this->generator = $generator;
        $this->classMetadata = $classMetadata;
    }

    /**
     * Build container class.
     *
     * @param string|null $containerClass Container class name
     * @param string      $namespace      Name space
     *
     * @return string Generated Container class code
     * @throws \InvalidArgumentException
     */
    public function build($containerClass = 'Container', $namespace = '')
    {
        // Build dependency injection container function name
        $this->resolverFunction = uniqid(self::DI_FUNCTION_PREFIX);

        $containerDependencies = [];
        $containerAliases = [];
        foreach ($this->classMetadata as $className => $classMetadata) {
            if ($classMetadata->alias !== null) {
                $containerAliases[$className] = $classMetadata->alias;
            }
            // Store inner dependencies
            if (array_key_exists('__construct', $classMetadata->methodsMetadata)) {
                $containerDependencies[$className] = array_values($classMetadata->methodsMetadata['__construct']->dependencies ?? []);
            }
        }

        $this->generator
            ->text('<?php declare(strict_types = 1);')
            ->newLine()
            ->defNamespace($namespace)
            ->multiComment(['Application container'])
            ->defClass($containerClass, '\\' . Container::class)
            ->defClassFunction('__construct')
            ->newLine('$this->dependencies = ')->arrayValue($containerDependencies)->text(';')
            ->newLine('$this->aliases = ')->arrayValue($containerAliases)->text(';')
            ->newLine('$this->' . self::SCOPE_SERVICES . ' = ')->arrayValue($this->scopes[self::SCOPE_SERVICES])->text(';')
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            ->endClassFunction()
            ->defClassFunction('logic', 'protected', ['$dependency'], ['{@inheritdoc}'])
            ->newLine('return $this->' . $this->resolverFunction . '($dependency);')
            ->endClassFunction();

        foreach ($this->classMetadata as $className => $classMetadata) {
            $dependencyName = $classMetadata->name ?? $className;

            // Generate camel case getter method
            $camelMethodName = 'get' . str_replace(' ', '', ucwords(ucfirst(str_replace(['\\', '_'], ' ', $dependencyName))));

            $this->generator
                ->defClassFunction($camelMethodName, 'public', [], ['@return \\' . $className . ' Get ' . $dependencyName . ' instance'])
                ->newLine('return $this->' . $this->resolverFunction . '(\'' . $dependencyName . '\');')
                ->endClassFunction();
        }

        // Build di container function and add to container class and return class code
        $this->buildDependencyResolver($this->resolverFunction);

        return $this->generator
            ->endClass()
            ->flush();
    }

    /**
     * Build dependency resolving function.
     *
     * @param string $functionName Function name
     *
     * @throws \InvalidArgumentException
     */
    protected function buildDependencyResolver($functionName)
    {
        $inputVariable = '$aliasOrClassName';
        $this->generator
            ->defClassFunction($functionName, 'protected', [$inputVariable], ['Dependency resolving function'])
            ->defVar('static ' . self::DI_FUNCTION_SERVICES . ' = []')
            ->newLine();

        // Generate all container and delegate conditions
        $this->generateConditions($inputVariable, false);

        // Add method not found
        $this->generator->endIfCondition()->endFunction();
    }

    /**
     * Generate logic conditions and their implementation for container and its delegates.
     *
     * @param string     $inputVariable Input condition parameter variable name
     * @param bool|false $started       Flag if condition branching has been started
     */
    public function generateConditions($inputVariable = '$alias', $started = false)
    {
        // Iterate all container dependencies
        foreach ($this->classMetadata as $className => $classMetadata) {
            // Generate condition statement to define if this class is needed
            $conditionFunc = !$started ? 'defIfCondition' : 'defElseIfCondition';

            // Output condition branch
            $this->generator->$conditionFunc(
                $this->buildResolverCondition($inputVariable, $className, $classMetadata->name)
            );

            // Define if this class has service scope
            $isService = in_array($className, $this->scopes[self::SCOPE_SERVICES], true);

            /** @var MethodMetadata[] Gather only valid method for container */
            $classValidMethods = $this->getValidClassMethodsMetadata($classMetadata->methodsMetadata);

            /** @var PropertyMetadata[] Gather only valid property for container */
            $classValidProperties = $this->getValidClassPropertiesMetadata($classMetadata->propertiesMetadata);

            // Define class or service variable
            $staticContainerName = $isService
                ? self::DI_FUNCTION_SERVICES . '[\'' . $classMetadata->name . '\']'
                : '$temp';

            if ($isService) {
                // Check if dependency was instantiated
                $this->generator->defIfCondition('!array_key_exists(\'' . $className . '\', ' . self::DI_FUNCTION_SERVICES . ')');
            }

            if (count($classValidMethods) || count($classValidProperties)) {
                $this->generator->newLine($staticContainerName . ' = ');
                $this->buildResolvingClassDeclaration($className);
                $this->buildConstructorDependencies($classMetadata->methodsMetadata);

                // Internal scope reflection variable
                $reflectionVariable = '$reflectionClass';

                $this->buildReflectionClass($className, $classValidProperties, $classValidMethods, $reflectionVariable);

                // Process class properties
                foreach ($classValidProperties as $property) {
                    // If such property has the dependency
                    if ($property->dependency) {
                        // Set value via refection
                        $this->buildResolverPropertyDeclaration(
                            $property->name,
                            $property->dependency,
                            $staticContainerName,
                            $reflectionVariable,
                            $property->isPublic
                        );
                    }
                }

                /** @var MethodMetadata $methodMetadata */
                foreach ($classValidMethods as $methodName => $methodMetadata) {
                    $this->buildResolverMethodDeclaration(
                        $methodMetadata->dependencies,
                        $methodName,
                        $staticContainerName,
                        $reflectionVariable,
                        $methodMetadata->isPublic
                    );
                }

                if ($isService) {
                    $this->generator->endIfCondition();
                }

                $this->generator->newLine()->newLine('return ' . $staticContainerName . ';');
            } else {
                $this->generator->newLine('return ');
                $this->buildResolvingClassDeclaration($className);
                $this->buildConstructorDependencies($classMetadata->methodsMetadata);

                if ($isService) {
                    $this->generator->endIfCondition()->newLine('return ' . $staticContainerName . ';');
                }


            }

            // Set flag that condition is started
            $started = true;
        }
    }

    /**
     * Build resolving function condition.
     *
     * @param string      $inputVariable Condition variable
     * @param string      $className
     * @param string|null $alias
     *
     * @return string Condition code
     */
    protected function buildResolverCondition(string $inputVariable, string $className, string $alias = null) : string
    {
        // Create condition branch
        $condition = $inputVariable . ' === \'' . $className . '\'';

        if ($alias !== null && $alias !== $className) {
            $condition .= '||' . $this->buildResolverCondition($inputVariable, $alias);
        }

        return $condition;
    }

    /**
     * Get valid class methods metadata.
     *
     * @param MethodMetadata[] $classMethodsMetadata All class methods metadata
     *
     * @return array Valid class methods metadata
     */
    protected function getValidClassMethodsMetadata(array $classMethodsMetadata)
    {
        /** @var MethodMetadata[] Gather only valid method for container */
        $classValidMethods = [];
        foreach ($classMethodsMetadata as $methodName => $methodMetadata) {
            // Skip constructor method and empty dependencies
            if ($methodName !== '__construct' && count($methodMetadata->dependencies) > 0) {

                $classValidMethods[$methodName] = $methodMetadata;
            }
        }

        return $classValidMethods;
    }

    /**
     * Get valid class properties metadata.
     *
     * @param PropertyMetadata[] $classPropertiesMetadata All class properties metadata
     *
     * @return array Valid class properties metadata
     */
    protected function getValidClassPropertiesMetadata(array $classPropertiesMetadata)
    {
        /** @var PropertyMetadata[] Gather only valid property for container */
        $classValidProperties = [];
        foreach ($classPropertiesMetadata as $propertyName => $propertyMetadata) {
            // Skip constructor method and empty dependencies
            if ($propertyMetadata->dependency) {
                $classValidProperties[$propertyName] = $propertyMetadata;
            }
        }

        return $classValidProperties;
    }

    /**
     * Build resolving function class block.
     *
     * @param string $className Class name for new instance creation
     */
    protected function buildResolvingClassDeclaration(string $className)
    {
        $this->generator->text('new \\' . ltrim($className, '\\') . '(');
    }

    /**
     * Build constructor arguments injection.
     *
     * @param MethodMetadata[] $methodsMetaData
     */
    protected function buildConstructorDependencies(array $methodsMetaData)
    {
        // Process constructor dependencies
        $argumentsCount = 0;
        if (array_key_exists('__construct', $methodsMetaData)) {
            $constructorArguments = $methodsMetaData['__construct']->dependencies ?? [];
            $argumentsCount = count($constructorArguments);
            $i = 0;

            // Add indentation to move declaration arguments
            $this->generator->tabs++;

            // Process constructor arguments
            foreach ($constructorArguments as $argument => $dependency) {
                $this->buildResolverArgument($dependency);

                // Add comma if this is not last dependency
                if (++$i < $argumentsCount) {
                    $this->generator->text(',');
                }
            }

            // Restore indentation
            $this->generator->tabs--;
        }

        // Close declaration block, multiline if we have dependencies
        $argumentsCount ? $this->generator->newLine(');') : $this->generator->text(');');
    }

    /**
     * Build resolving function dependency argument.
     *
     * @param mixed $argument Dependency argument
     */
    protected function buildResolverArgument($argument, $textFunction = 'newLine')
    {
        // This is a dependency which invokes resolving function
        if (array_key_exists($argument, $this->classMetadata)) {
            // Call container logic for this dependency
            $this->generator->$textFunction('$this->' . $this->resolverFunction . '(\'' . $argument . '\')');
        } elseif (array_key_exists($argument, $this->classAliases)) {
            // Call container logic for this dependency
            $this->generator->$textFunction('$this->' . $this->resolverFunction . '(\'' . $argument . '\')');
        } elseif (is_string($argument)) { // String variable
            $this->generator->$textFunction()->stringValue($argument);
        } elseif (is_array($argument)) { // Dependency value is array
            $this->generator->$textFunction()->arrayValue($argument);
        }
    }

    /**
     * Generate reflection class for private/protected methods or properties
     * in current scope.
     *
     * @param string             $className          Reflection class source class name
     * @param PropertyMetadata[] $propertiesMetadata Properties metadata
     * @param MethodMetadata[]   $methodsMetadata    Methods metadata
     * @param string             $reflectionVariable Reflection class variable name
     */
    protected function buildReflectionClass(string $className, array $propertiesMetadata, array $methodsMetadata, string $reflectionVariable)
    {
        /** @var bool $reflectionClassCreated Flag showing that reflection class already created in current scope */
        $reflectionClassCreated = false;

        /**
         * Iterate all properties and create internal scope reflection class instance if
         * at least one property in not public
         */
        foreach ($propertiesMetadata as $propertyMetadata) {

            if (!$propertyMetadata->isPublic) {
                $this->generator
                    ->comment('Create reflection class for injecting private/protected properties and methods')
                    ->newLine($reflectionVariable . ' = new \ReflectionClass(\'' . $className . '\');')
                    ->newLine();

                $reflectionClassCreated = true;

                break;
            }
        }

        /**
         * Iterate all properties and create internal scope reflection class instance if
         * at least one property in not public
         */
        if (!$reflectionClassCreated) {
            foreach ($methodsMetadata as $methodMetadata) {

                if (!$methodMetadata->isPublic) {
                    $this->generator
                        ->comment('Create reflection class for injecting private/protected properties and methods')
                        ->newLine($reflectionVariable . ' = new \ReflectionClass(\'' . $className . '\');')
                        ->newLine();

                    break;
                }
            }
        }
    }

    /**
     * Build resolving property injection declaration.
     *
     * @param string $propertyName       Target property name
     * @param string $dependency         Dependency class name
     * @param string $containerVariable  Container declaration variable name
     * @param string $reflectionVariable Reflection class variable name
     * @param bool   $isPublic           Flag if property is public
     */
    protected function buildResolverPropertyDeclaration(
        string $propertyName,
        string $dependency,
        string $containerVariable,
        string $reflectionVariable,
        bool $isPublic
    )
    {

        if ($isPublic) {
            $this->generator
                ->comment('Inject public dependency for $' . $propertyName)
                ->newLine($containerVariable . '->' . $propertyName . ' = ');
            $this->buildResolverArgument($dependency, 'text');
            $this->generator->text(';');
        } else {
            $this->generator
                ->comment('Inject private dependency for $' . $propertyName)
                ->newLine('$property = ' . $reflectionVariable . '->getProperty(\'' . $propertyName . '\');')
                ->newLine('$property->setAccessible(true);')
                ->newLine('$property->setValue(')
                ->increaseIndentation()
                ->newLine($containerVariable . ',');

            $this->buildResolverArgument($dependency);

            $this->generator
                ->decreaseIndentation()
                ->newLine(');')
                ->newLine('$property->setAccessible(false);')
                ->newLine();
        }
    }

    /**
     * Build resolving method injection declaration.
     *
     * @param array  $dependencies       Collection of method dependencies
     * @param string $methodName         Method name
     * @param string $containerVariable  Container declaration variable name
     * @param string $reflectionVariable Reflection class variable name
     * @param bool   $isPublic           Flag if method is public
     */
    protected function buildResolverMethodDeclaration(
        array $dependencies,
        string $methodName,
        string $containerVariable,
        string $reflectionVariable,
        bool $isPublic
    )
    {

        // Get method arguments
        $argumentsCount = count($dependencies);

        $this->generator->comment('Invoke ' . $methodName . '() and pass dependencies(y)');

        if ($isPublic) {
            $this->generator->newLine($containerVariable . '->' . $methodName . '(')->increaseIndentation();
        } else {
            $this->generator
                ->newLine('$method = ' . $reflectionVariable . '->getMethod(\'' . $methodName . '\');')
                ->newLine('$method->setAccessible(true);')
                ->newLine('$method->invoke(')
                ->increaseIndentation()
                ->newLine($containerVariable . ',');
        }

        $i = 0;
        // Iterate method arguments
        foreach ($dependencies as $argument => $dependency) {
            // Add dependencies
            $this->buildResolverArgument($dependency);

            // Add comma if this is not last dependency
            if (++$i < $argumentsCount) {
                $this->generator->text(',');
            }
        }

        $this->generator->decreaseIndentation()->newLine(');');
    }
}


1			<?php declare(strict_types = 1);
2			/**
3			* Created by PhpStorm.
4			* User: root
5			* Date: 02.08.16
6			* Time: 0:46.
7			*/
8			namespace samsonframework\container;
9
10			use samsonframework\container\metadata\ClassMetadata;
11			use samsonframework\container\metadata\MethodMetadata;
12			use samsonframework\container\metadata\PropertyMetadata;
13			use samsonframework\di\Container;
14			use samsonphp\generator\Generator;
15
16			/**
17			* Container builder.
18			*
19			* @author Vitaly Egorov <[email protected]>
20			*/
21			class Builder
22			{
23			/** Controller classes scope name */
24			const SCOPE_CONTROLLER = 'controllers';
25
26			/** Service classes scope name */
27			const SCOPE_SERVICES = 'services';
28
29			/** Generated resolving function name prefix */
30			const DI_FUNCTION_PREFIX = 'container';
31
32			/** Generated resolving function service static collection name */
33			const DI_FUNCTION_SERVICES = '$' . self::SCOPE_SERVICES;
34
35			/** @var string[] Collection of available container scopes */
36			protected $scopes = [
37			self::SCOPE_CONTROLLER => [],
38			self::SCOPE_SERVICES => []
39			];
40
41			/** @var ClassMetadata[] Collection of classes metadata */
42			protected $classMetadata = [];
43
44			/** @var array Collection of dependencies aliases */
45			protected $classAliases = [];
46
47			/**
48			* @var Generator
49			* @Injectable
50			*/
51			protected $generator;
52
53			/** @var string Resolver function name */
54			protected $resolverFunction;
55
56			/**
57			* Container builder constructor.
58			*
59			* @param Generator $generator PHP code generator
60			* @param ClassMetadata[] $classMetadata Collection of classes metadata for container
61			*/
62	12		public function __construct(Generator $generator, array $classMetadata)
63			{
64	12		$this->generator = $generator;
65	12		$this->classMetadata = $classMetadata;
66	12		}
67
68			/**
69			* Build container class.
70			*
71			* @param string\|null $containerClass Container class name
72			* @param string $namespace Name space
73			*
74			* @return string Generated Container class code
75			* @throws \InvalidArgumentException
76			*/
77	8		public function build($containerClass = 'Container', $namespace = '')
78			{
79			// Build dependency injection container function name
80	8		$this->resolverFunction = uniqid(self::DI_FUNCTION_PREFIX);
81
82	8		$containerDependencies = [];
83	8		$containerAliases = [];
84	8		foreach ($this->classMetadata as $className => $classMetadata) {
85	8		if ($classMetadata->alias !== null) {
86			$containerAliases[$className] = $classMetadata->alias;
87			}
88			// Store inner dependencies
89	8		if (array_key_exists('__construct', $classMetadata->methodsMetadata)) {
90	8		$containerDependencies[$className] = array_values($classMetadata->methodsMetadata['__construct']->dependencies ?? []);
91			}
92			}
93
94	8		$this->generator
95	8		->text('<?php declare(strict_types = 1);')
96	8		->newLine()
97	8		->defNamespace($namespace)
98	8		->multiComment(['Application container'])
99	8		->defClass($containerClass, '\\' . Container::class)
100	8		->defClassFunction('__construct')
101	8		->newLine('$this->dependencies = ')->arrayValue($containerDependencies)->text(';')
102	8		->newLine('$this->aliases = ')->arrayValue($containerAliases)->text(';')
103	8		->newLine('$this->' . self::SCOPE_SERVICES . ' = ')->arrayValue($this->scopes[self::SCOPE_SERVICES])->text(';')
			0 ignored issues – show Documentation introduced 2016-08-14 07:49 UTC by Report Bug Copy Issue Report `$this->scopes[self::SCOPE_SERVICES]` is of type `string`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
104	8		->endClassFunction()
105	8		->defClassFunction('logic', 'protected', ['$dependency'], ['{@inheritdoc}'])
106	8		->newLine('return $this->' . $this->resolverFunction . '($dependency);')
107	8		->endClassFunction();
108
109	8		foreach ($this->classMetadata as $className => $classMetadata) {
110	8		$dependencyName = $classMetadata->name ?? $className;
111
112			// Generate camel case getter method
113	8		$camelMethodName = 'get' . str_replace(' ', '', ucwords(ucfirst(str_replace(['\\', '_'], ' ', $dependencyName))));
114
115	8		$this->generator
116	8		->defClassFunction($camelMethodName, 'public', [], ['@return \\' . $className . ' Get ' . $dependencyName . ' instance'])
117	8		->newLine('return $this->' . $this->resolverFunction . '(\'' . $dependencyName . '\');')
118	8		->endClassFunction();
119			}
120
121			// Build di container function and add to container class and return class code
122	8		$this->buildDependencyResolver($this->resolverFunction);
123
124	8		return $this->generator
125	8		->endClass()
126	8		->flush();
127			}
128
129			/**
130			* Build dependency resolving function.
131			*
132			* @param string $functionName Function name
133			*
134			* @throws \InvalidArgumentException
135			*/
136	8		protected function buildDependencyResolver($functionName)
137			{
138	8		$inputVariable = '$aliasOrClassName';
139	8		$this->generator
140	8		->defClassFunction($functionName, 'protected', [$inputVariable], ['Dependency resolving function'])
141	8		->defVar('static ' . self::DI_FUNCTION_SERVICES . ' = []')
142	8		->newLine();
143
144			// Generate all container and delegate conditions
145	8		$this->generateConditions($inputVariable, false);
146
147			// Add method not found
148	8		$this->generator->endIfCondition()->endFunction();
149	8		}
150
151			/**
152			* Generate logic conditions and their implementation for container and its delegates.
153			*
154			* @param string $inputVariable Input condition parameter variable name
155			* @param bool\|false $started Flag if condition branching has been started
156			*/
157	8		public function generateConditions($inputVariable = '$alias', $started = false)
158			{
159			// Iterate all container dependencies
160	8		foreach ($this->classMetadata as $className => $classMetadata) {
161			// Generate condition statement to define if this class is needed
162	8		$conditionFunc = !$started ? 'defIfCondition' : 'defElseIfCondition';
163
164			// Output condition branch
165	8		$this->generator->$conditionFunc(
166	8		$this->buildResolverCondition($inputVariable, $className, $classMetadata->name)
167			);
168
169			// Define if this class has service scope
170	8		$isService = in_array($className, $this->scopes[self::SCOPE_SERVICES], true);
171
172			/** @var MethodMetadata[] Gather only valid method for container */
173	8		$classValidMethods = $this->getValidClassMethodsMetadata($classMetadata->methodsMetadata);
174
175			/** @var PropertyMetadata[] Gather only valid property for container */
176	8		$classValidProperties = $this->getValidClassPropertiesMetadata($classMetadata->propertiesMetadata);
177
178			// Define class or service variable
179	8		$staticContainerName = $isService
180	8		? self::DI_FUNCTION_SERVICES . '[\'' . $classMetadata->name . '\']'
181	8		: '$temp';
182
183	8		if ($isService) {
184			// Check if dependency was instantiated
185	8		$this->generator->defIfCondition('!array_key_exists(\'' . $className . '\', ' . self::DI_FUNCTION_SERVICES . ')');
186			}
187
188	8		if (count($classValidMethods) \|\| count($classValidProperties)) {
189	8		$this->generator->newLine($staticContainerName . ' = ');
190	8		$this->buildResolvingClassDeclaration($className);
191	8		$this->buildConstructorDependencies($classMetadata->methodsMetadata);
192
193			// Internal scope reflection variable
194	8		$reflectionVariable = '$reflectionClass';
195
196	8		$this->buildReflectionClass($className, $classValidProperties, $classValidMethods, $reflectionVariable);
197
198			// Process class properties
199	8		foreach ($classValidProperties as $property) {
200			// If such property has the dependency
201	8		if ($property->dependency) {
202			// Set value via refection
203	8		$this->buildResolverPropertyDeclaration(
204	8		$property->name,
205	8		$property->dependency,
206			$staticContainerName,
207			$reflectionVariable,
208	8		$property->isPublic
209			);
210			}
211			}
212
213			/** @var MethodMetadata $methodMetadata */
214	8		foreach ($classValidMethods as $methodName => $methodMetadata) {
215	8		$this->buildResolverMethodDeclaration(
216	8		$methodMetadata->dependencies,
217			$methodName,
218			$staticContainerName,
219			$reflectionVariable,
220	8		$methodMetadata->isPublic
221			);
222			}
223
224	8		if ($isService) {
225	7		$this->generator->endIfCondition();
226			}
227
228	8		$this->generator->newLine()->newLine('return ' . $staticContainerName . ';');
229			} else {
230	8		$this->generator->newLine('return ');
231	8		$this->buildResolvingClassDeclaration($className);
232	8		$this->buildConstructorDependencies($classMetadata->methodsMetadata);
233
234	8		if ($isService) {
235	1		$this->generator->endIfCondition()->newLine('return ' . $staticContainerName . ';');
236			}
237
			0 ignored issues – show Coding Style introduced 2016-08-18 11:47 UTC by Report Bug Copy Issue Report Blank line found at end of control structure Loading history...
238			}
239
240			// Set flag that condition is started
241	8		$started = true;
242			}
243	8		}
244
245			/**
246			* Build resolving function condition.
247			*
248			* @param string $inputVariable Condition variable
249			* @param string $className
250			* @param string\|null $alias
251			*
252			* @return string Condition code
253			*/
254	8		protected function buildResolverCondition(string $inputVariable, string $className, string $alias = null) : string
255			{
256			// Create condition branch
257	8		$condition = $inputVariable . ' === \'' . $className . '\'';
258
259	8		if ($alias !== null && $alias !== $className) {
260	8		$condition .= '\|\|' . $this->buildResolverCondition($inputVariable, $alias);
261			}
262
263	8		return $condition;
264			}
265
266			/**
267			* Get valid class methods metadata.
268			*
269			* @param MethodMetadata[] $classMethodsMetadata All class methods metadata
270			*
271			* @return array Valid class methods metadata
272			*/
273	8		protected function getValidClassMethodsMetadata(array $classMethodsMetadata)
274			{
275			/** @var MethodMetadata[] Gather only valid method for container */
276	8		$classValidMethods = [];
277	8		foreach ($classMethodsMetadata as $methodName => $methodMetadata) {
278			// Skip constructor method and empty dependencies
279	8		if ($methodName !== '__construct' && count($methodMetadata->dependencies) > 0) {
			0 ignored issues – show Unused Code Bug introduced 2016-08-18 11:47 UTC by Report Bug Copy Issue Report The strict comparison `!==` seems to always evaluate to `true` as the types of `$methodName` (`integer`) and `'__construct'` (`string`) can never be identical. Maybe you want to use a loose comparison `!=` instead? Loading history...
280	8		$classValidMethods[$methodName] = $methodMetadata;
281			}
282			}
283
284	8		return $classValidMethods;
285			}
286
287			/**
288			* Get valid class properties metadata.
289			*
290			* @param PropertyMetadata[] $classPropertiesMetadata All class properties metadata
291			*
292			* @return array Valid class properties metadata
293			*/
294	8		protected function getValidClassPropertiesMetadata(array $classPropertiesMetadata)
295			{
296			/** @var PropertyMetadata[] Gather only valid property for container */
297	8		$classValidProperties = [];
298	8		foreach ($classPropertiesMetadata as $propertyName => $propertyMetadata) {
299			// Skip constructor method and empty dependencies
300	8		if ($propertyMetadata->dependency) {
301	8		$classValidProperties[$propertyName] = $propertyMetadata;
302			}
303			}
304
305	8		return $classValidProperties;
306			}
307
308			/**
309			* Build resolving function class block.
310			*
311			* @param string $className Class name for new instance creation
312			*/
313	8		protected function buildResolvingClassDeclaration(string $className)
314			{
315	8		$this->generator->text('new \\' . ltrim($className, '\\') . '(');
316	8		}
317
318			/**
319			* Build constructor arguments injection.
320			*
321			* @param MethodMetadata[] $methodsMetaData
322			*/
323	8		protected function buildConstructorDependencies(array $methodsMetaData)
324			{
325			// Process constructor dependencies
326	8		$argumentsCount = 0;
327	8		if (array_key_exists('__construct', $methodsMetaData)) {
328	8		$constructorArguments = $methodsMetaData['__construct']->dependencies ?? [];
329	8		$argumentsCount = count($constructorArguments);
330	8		$i = 0;
331
332			// Add indentation to move declaration arguments
333	8		$this->generator->tabs++;
334
335			// Process constructor arguments
336	8		foreach ($constructorArguments as $argument => $dependency) {
337	8		$this->buildResolverArgument($dependency);
338
339			// Add comma if this is not last dependency
340	8		if (++$i < $argumentsCount) {
341	8		$this->generator->text(',');
342			}
343			}
344
345			// Restore indentation
346	8		$this->generator->tabs--;
347			}
348
349			// Close declaration block, multiline if we have dependencies
350	8		$argumentsCount ? $this->generator->newLine(');') : $this->generator->text(');');
351	8		}
352
353			/**
354			* Build resolving function dependency argument.
355			*
356			* @param mixed $argument Dependency argument
357			*/
358	8		protected function buildResolverArgument($argument, $textFunction = 'newLine')
359			{
360			// This is a dependency which invokes resolving function
361	8		if (array_key_exists($argument, $this->classMetadata)) {
362			// Call container logic for this dependency
363	8		$this->generator->$textFunction('$this->' . $this->resolverFunction . '(\'' . $argument . '\')');
364	1		} elseif (array_key_exists($argument, $this->classAliases)) {
365			// Call container logic for this dependency
366	1		$this->generator->$textFunction('$this->' . $this->resolverFunction . '(\'' . $argument . '\')');
367	1		} elseif (is_string($argument)) { // String variable
368	1		$this->generator->$textFunction()->stringValue($argument);
369			} elseif (is_array($argument)) { // Dependency value is array
370			$this->generator->$textFunction()->arrayValue($argument);
371			}
372	8		}
373
374			/**
375			* Generate reflection class for private/protected methods or properties
376			* in current scope.
377			*
378			* @param string $className Reflection class source class name
379			* @param PropertyMetadata[] $propertiesMetadata Properties metadata
380			* @param MethodMetadata[] $methodsMetadata Methods metadata
381			* @param string $reflectionVariable Reflection class variable name
382			*/
383	8		protected function buildReflectionClass(string $className, array $propertiesMetadata, array $methodsMetadata, string $reflectionVariable)
384			{
385			/** @var bool $reflectionClassCreated Flag showing that reflection class already created in current scope */
386	8		$reflectionClassCreated = false;
387
388			/**
389			* Iterate all properties and create internal scope reflection class instance if
390			* at least one property in not public
391			*/
392	8	View Code Duplication	foreach ($propertiesMetadata as $propertyMetadata) {
			0 ignored issues – show Duplication introduced 2016-08-09 17:42 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
393	8		if (!$propertyMetadata->isPublic) {
394	8		$this->generator
395	8		->comment('Create reflection class for injecting private/protected properties and methods')
396	8		->newLine($reflectionVariable . ' = new \ReflectionClass(\'' . $className . '\');')
397	8		->newLine();
398
399	8		$reflectionClassCreated = true;
400
401	8		break;
402			}
403			}
404
405			/**
406			* Iterate all properties and create internal scope reflection class instance if
407			* at least one property in not public
408			*/
409	8		if (!$reflectionClassCreated) {
410	8	View Code Duplication	foreach ($methodsMetadata as $methodMetadata) {
			0 ignored issues – show Duplication introduced 2016-08-09 17:42 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
411	8		if (!$methodMetadata->isPublic) {
412			$this->generator
413			->comment('Create reflection class for injecting private/protected properties and methods')
414			->newLine($reflectionVariable . ' = new \ReflectionClass(\'' . $className . '\');')
415			->newLine();
416
417	8		break;
418			}
419			}
420			}
421	8		}
422
423			/**
424			* Build resolving property injection declaration.
425			*
426			* @param string $propertyName Target property name
427			* @param string $dependency Dependency class name
428			* @param string $containerVariable Container declaration variable name
429			* @param string $reflectionVariable Reflection class variable name
430			* @param bool $isPublic Flag if property is public
431			*/
432	8		protected function buildResolverPropertyDeclaration(
433			string $propertyName,
434			string $dependency,
435			string $containerVariable,
436			string $reflectionVariable,
437			bool $isPublic
438			)
439			{
			0 ignored issues – show Coding Style introduced 2016-08-18 11:47 UTC by Report Bug Copy Issue Report The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line Loading history...
440	8		if ($isPublic) {
441	7		$this->generator
442	7		->comment('Inject public dependency for $' . $propertyName)
443	7		->newLine($containerVariable . '->' . $propertyName . ' = ');
444	7		$this->buildResolverArgument($dependency, 'text');
445	7		$this->generator->text(';');
446			} else {
447	8		$this->generator
448	8		->comment('Inject private dependency for $' . $propertyName)
449	8		->newLine('$property = ' . $reflectionVariable . '->getProperty(\'' . $propertyName . '\');')
450	8		->newLine('$property->setAccessible(true);')
451	8		->newLine('$property->setValue(')
452	8		->increaseIndentation()
453	8		->newLine($containerVariable . ',');
454
455	8		$this->buildResolverArgument($dependency);
456
457	8		$this->generator
458	8		->decreaseIndentation()
459	8		->newLine(');')
460	8		->newLine('$property->setAccessible(false);')
461	8		->newLine();
462			}
463	8		}
464
465			/**
466			* Build resolving method injection declaration.
467			*
468			* @param array $dependencies Collection of method dependencies
469			* @param string $methodName Method name
470			* @param string $containerVariable Container declaration variable name
471			* @param string $reflectionVariable Reflection class variable name
472			* @param bool $isPublic Flag if method is public
473			*/
474	8		protected function buildResolverMethodDeclaration(
475			array $dependencies,
476			string $methodName,
477			string $containerVariable,
478			string $reflectionVariable,
479			bool $isPublic
480			)
481			{
			0 ignored issues – show Coding Style introduced 2016-08-18 11:47 UTC by Report Bug Copy Issue Report The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line Loading history...
482			// Get method arguments
483	8		$argumentsCount = count($dependencies);
484
485	8		$this->generator->comment('Invoke ' . $methodName . '() and pass dependencies(y)');
486
487	8		if ($isPublic) {
488	8		$this->generator->newLine($containerVariable . '->' . $methodName . '(')->increaseIndentation();
489			} else {
490	7		$this->generator
491	7		->newLine('$method = ' . $reflectionVariable . '->getMethod(\'' . $methodName . '\');')
492	7		->newLine('$method->setAccessible(true);')
493	7		->newLine('$method->invoke(')
494	7		->increaseIndentation()
495	7		->newLine($containerVariable . ',');
496			}
497
498	8		$i = 0;
499			// Iterate method arguments
500	8		foreach ($dependencies as $argument => $dependency) {
501			// Add dependencies
502	8		$this->buildResolverArgument($dependency);
503
504			// Add comma if this is not last dependency
505	8		if (++$i < $argumentsCount) {
506	8		$this->generator->text(',');
507			}
508			}
509
510	8		$this->generator->decreaseIndentation()->newLine(');');
511	8		}
512			}
513

samsonframework / container

Push — master ( 4bcb6e...27d771 )

Builder::generateConditions() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like