ArrayDefinition::setCallMethods() - Code Metrics - Inspection of "New array definition syntax" - yiisoft/factory - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#78)

by Alexander

created 2021-04-12 17:53 UTC

ArrayDefinition::setCallMethods() B

↳ Parent: ArrayDefinition

Complexity

Conditions	7
Paths	7

Size

Total Lines	33
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	10.6588

Importance

Changes

Metric	Value
cc	7
eloc	19
nc	7
nop	1
dl	0
loc	33
ccs	11
cts	19
cp	0.5789
crap	10.6588
rs	8.8333
c	0
b	0
f	0

<?php

declare(strict_types=1);

namespace Yiisoft\Factory\Definitions;

use Psr\Container\ContainerInterface;
use Yiisoft\Factory\Exceptions\InvalidConfigException;

use Yiisoft\Factory\Exceptions\NotInstantiableException;

use function array_key_exists;
use function get_class;
use function gettype;
use function is_array;
use function is_int;
use function is_object;
use function is_string;

/**
 * Builds object by array config
 */
class ArrayDefinition implements DefinitionInterface
{
    public const CLASS_NAME = 'class';
    public const CONSTRUCTOR = '__construct()';

    /**
     * @psalm-var class-string
     */
    private string $class;
    private array $constructorParameters;

    /**
     * @psalm-var array<string, array>
     */
    private array $callMethods;

    /**
     * @psalm-var array<string, mixed>
     */
    private array $setProperties;

    /**
     * @psalm-param array{
     *   class: class-string,
     *   constructor?: array,
     *   callMethods?: array,
     *   setProperties?: array,
     * } $config
     *
     * @throws InvalidConfigException
     */
    public function __construct(array $config)
    {
        $this->setClass($config);
        $this->setConstructorParameters($config);
        $this->setCallMethods($config);
        $this->setSetProperties($config);
    }

    /**
     * @throws InvalidConfigException
     */
    private function setClass(array $config): void
    {
        if (!array_key_exists(self::CLASS_NAME, $config)) {
            throw new InvalidConfigException('Invalid definition: no class name specified.');
        }

        /** @var mixed */
        $class = $config[self::CLASS_NAME];

        if (!is_string($class)) {
            throw new InvalidConfigException(sprintf('Invalid definition: invalid class name "%s".', (string)$class));
        }

        if ($class === '') {
            throw new InvalidConfigException('Invalid definition: empty class name.');
        }

        if (!class_exists($class)) {
            throw new InvalidConfigException(sprintf('Invalid definition: class "%s" does not exist.', $class));
        }

        $this->class = $class;
    }

    /**
     * @throws InvalidConfigException
     */
    private function setConstructorParameters(array $config): void
    {
        $parameters = $config[self::CONSTRUCTOR] ?? [];

        if (!is_array($parameters)) {
            throw new InvalidConfigException(
                sprintf(
                    'Invalid definition: incorrect constructor parameters. Expected array, got %s.',
                    $this->getType($parameters)
                )
            );
        }

        $this->constructorParameters = $parameters;
    }

    /**
     * @throws InvalidConfigException
     */
    private function setCallMethods(array $config): void
    {
        $items = $this->extractMethods($config);

        if (!is_array($items)) {

            throw new InvalidConfigException(
                sprintf('Invalid definition: incorrect method calls. Expected array, got %s.', $this->getType($items))
            );
        }

        $callMethods = [];
        foreach ($items as $key => $value) {
            if (is_int($key)) {
                if (!is_string($value)) {
                    throw new InvalidConfigException(
                        sprintf('Invalid definition: expected method name, got %s.', $this->getType($value))
                    );
                }
                if ($value === '') {
                    throw new InvalidConfigException('Invalid definition: expected method name, got empty string.');
                }
                $callMethods[$value] = [];
            } else {
                if (!is_array($value)) {
                    throw new InvalidConfigException(
                        sprintf('Invalid definition: incorrect method parameters. Expected array, got %s.', $this->getType($value))
                    );
                }
                $callMethods[$key] = $value;
            }
        }

        $this->callMethods = $callMethods;
    }

    private function extractMethods(array $config): array
    {
        $methods = [];

        foreach ($config as $key => $value) {
            if ($key !== self::CONSTRUCTOR && substr($key, -2) === '()') {
                $methods[substr($key, 0, -2)] = $value;
            }
        }

        return $methods;
    }

    /**
     * @throws InvalidConfigException
     */
    private function setSetProperties(array $config): void
    {
        $properties = $this->extractProperties($config);

        if (!is_array($properties)) {

            throw new InvalidConfigException(
                sprintf('Invalid definition: incorrect properties to set. Expected array, got %s.', $this->getType($properties))
            );
        }

        foreach ($properties as $key => $_value) {
            if (!is_string($key)) {
                throw new InvalidConfigException(
                    sprintf('Invalid definition: expected property name, got %s.', $this->getType($key))
                );
            }
        }

        /** @psalm-var array<string,mixed> $properties */

        $this->setProperties = $properties;
    }

    private function extractProperties(array $config): array
    {
        $properties = [];
        foreach ($config as $key => $value) {
            if (substr($key, 0, 1) === '@') {
                $properties[substr($key, 1)] = $value;
            }
        }

        return $properties;
    }

    /**
     * @psalm-return class-string
     */
    public function getClass(): string
    {
        return $this->class;
    }

    public function getConstructorParameters(): array
    {
        return $this->constructorParameters;
    }

    /**
     * @psalm-return array<string, array>
     */
    public function getCallMethods(): array
    {
        return $this->callMethods;
    }

    /**
     * @psalm-return array<string, mixed>
     */
    public function getSetProperties(): array
    {
        return $this->setProperties;
    }

    /**
     * @throws NotInstantiableException
     * @throws InvalidConfigException
     */
    public function resolve(ContainerInterface $container): object
    {
        return ArrayDefinitionBuilder::getInstance()->build($container, $this);
    }

    public function merge(self $other): self
    {
        $callMethods = $this->getCallMethods();
        foreach ($other->getCallMethods() as $method => $parameters) {
            $callMethods[$method] = isset($callMethods[$method])
                ? $this->mergeParameters($callMethods[$method], $parameters)
                : $parameters;
        }
        $mergedMethods = [];
        foreach ($callMethods as $key => $value) {
            $mergedMethods[$key . '()'] = $value;
        }

        $properties = array_merge($this->getSetProperties(), $other->getSetProperties());
        $mergedProperties = [];
        foreach ($properties as $key => $value) {
            $mergedProperties['@' . $key] = $value;
        }

        return new self(array_merge([
            self::CLASS_NAME => $other->getClass(),
            self::CONSTRUCTOR => $this->mergeParameters(
                $this->getConstructorParameters(),
                $other->getConstructorParameters()
            ),
        ], $mergedMethods, $mergedProperties));
    }

    private function mergeParameters(array $selfParameters, array $otherParameters): array
    {
        /** @var mixed $parameter */
        foreach ($otherParameters as $index => $parameter) {
            /** @var mixed */
            $selfParameters[$index] = $parameter;
        }

        return $selfParameters;
    }

    /**
     * @param mixed $value
     */
    private function getType($value): string
    {
        return is_object($value) ? get_class($value) : gettype($value);
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Yiisoft\Factory\Definitions;
6
7		use Psr\Container\ContainerInterface;
8		use Yiisoft\Factory\Exceptions\InvalidConfigException;
9
10		use Yiisoft\Factory\Exceptions\NotInstantiableException;
11
12		use function array_key_exists;
13		use function get_class;
14		use function gettype;
15		use function is_array;
16		use function is_int;
17		use function is_object;
18		use function is_string;
19
20		/**
21		* Builds object by array config
22		*/
23		class ArrayDefinition implements DefinitionInterface
24		{
25		public const CLASS_NAME = 'class';
26		public const CONSTRUCTOR = '__construct()';
27
28		/**
29		* @psalm-var class-string
30		*/
31		private string $class;
32		private array $constructorParameters;
33
34		/**
35		* @psalm-var array<string, array>
36		*/
37		private array $callMethods;
38
39		/**
40		* @psalm-var array<string, mixed>
41		*/
42		private array $setProperties;
43
44		/**
45		* @psalm-param array{
46		* class: class-string,
47		* constructor?: array,
48		* callMethods?: array,
49		* setProperties?: array,
50		* } $config
51		*
52		* @throws InvalidConfigException
53		*/
54	47	public function __construct(array $config)
55		{
56	47	$this->setClass($config);
57	40	$this->setConstructorParameters($config);
58	39	$this->setCallMethods($config);
59	38	$this->setSetProperties($config);
60	38	}
61
62		/**
63		* @throws InvalidConfigException
64		*/
65	47	private function setClass(array $config): void
66		{
67	47	if (!array_key_exists(self::CLASS_NAME, $config)) {
68	1	throw new InvalidConfigException('Invalid definition: no class name specified.');
69		}
70
71		/** @var mixed */
72	46	$class = $config[self::CLASS_NAME];
73
74	46	if (!is_string($class)) {
75	1	throw new InvalidConfigException(sprintf('Invalid definition: invalid class name "%s".', (string)$class));
76		}
77
78	45	if ($class === '') {
79	1	throw new InvalidConfigException('Invalid definition: empty class name.');
80		}
81
82	44	if (!class_exists($class)) {
83	6	throw new InvalidConfigException(sprintf('Invalid definition: class "%s" does not exist.', $class));
84		}
85
86	40	$this->class = $class;
87	40	}
88
89		/**
90		* @throws InvalidConfigException
91		*/
92	40	private function setConstructorParameters(array $config): void
93		{
94	40	$parameters = $config[self::CONSTRUCTOR] ?? [];
95
96	40	if (!is_array($parameters)) {
97	1	throw new InvalidConfigException(
98	1	sprintf(
99	1	'Invalid definition: incorrect constructor parameters. Expected array, got %s.',
100	1	$this->getType($parameters)
101		)
102		);
103		}
104
105	39	$this->constructorParameters = $parameters;
106	39	}
107
108		/**
109		* @throws InvalidConfigException
110		*/
111	39	private function setCallMethods(array $config): void
112		{
113	39	$items = $this->extractMethods($config);
114
115	39	if (!is_array($items)) {
		0 ignored issues – show introduced 2021-04-12 16:50 UTC by Report Bug Copy Issue Report The condition `is_array($items)` is always `true`. Loading history...
116		throw new InvalidConfigException(
117		sprintf('Invalid definition: incorrect method calls. Expected array, got %s.', $this->getType($items))
118		);
119		}
120
121	39	$callMethods = [];
122	39	foreach ($items as $key => $value) {
123	10	if (is_int($key)) {
124		if (!is_string($value)) {
125		throw new InvalidConfigException(
126		sprintf('Invalid definition: expected method name, got %s.', $this->getType($value))
127		);
128		}
129		if ($value === '') {
130		throw new InvalidConfigException('Invalid definition: expected method name, got empty string.');
131		}
132		$callMethods[$value] = [];
133		} else {
134	10	if (!is_array($value)) {
135	1	throw new InvalidConfigException(
136	1	sprintf('Invalid definition: incorrect method parameters. Expected array, got %s.', $this->getType($value))
137		);
138		}
139	9	$callMethods[$key] = $value;
140		}
141		}
142
143	38	$this->callMethods = $callMethods;
144	38	}
145
146	39	private function extractMethods(array $config): array
147		{
148	39	$methods = [];
149
150	39	foreach ($config as $key => $value) {
151	39	if ($key !== self::CONSTRUCTOR && substr($key, -2) === '()') {
152	10	$methods[substr($key, 0, -2)] = $value;
153		}
154		}
155
156	39	return $methods;
157		}
158
159		/**
160		* @throws InvalidConfigException
161		*/
162	38	private function setSetProperties(array $config): void
163		{
164	38	$properties = $this->extractProperties($config);
165
166	38	if (!is_array($properties)) {
		0 ignored issues – show introduced 2021-04-12 16:50 UTC by Report Bug Copy Issue Report The condition `is_array($properties)` is always `true`. Loading history...
167		throw new InvalidConfigException(
168		sprintf('Invalid definition: incorrect properties to set. Expected array, got %s.', $this->getType($properties))
169		);
170		}
171
172	38	foreach ($properties as $key => $_value) {
173	2	if (!is_string($key)) {
174		throw new InvalidConfigException(
175		sprintf('Invalid definition: expected property name, got %s.', $this->getType($key))
176		);
177		}
178		}
179
180		/** @psalm-var array<string,mixed> $properties */
181
182	38	$this->setProperties = $properties;
183	38	}
184
185	38	private function extractProperties(array $config): array
186		{
187	38	$properties = [];
188	38	foreach ($config as $key => $value) {
189	38	if (substr($key, 0, 1) === '@') {
190	2	$properties[substr($key, 1)] = $value;
191		}
192		}
193
194	38	return $properties;
195		}
196
197		/**
198		* @psalm-return class-string
199		*/
200	38	public function getClass(): string
201		{
202	38	return $this->class;
203		}
204
205	38	public function getConstructorParameters(): array
206		{
207	38	return $this->constructorParameters;
208		}
209
210		/**
211		* @psalm-return array<string, array>
212		*/
213	35	public function getCallMethods(): array
214		{
215	35	return $this->callMethods;
216		}
217
218		/**
219		* @psalm-return array<string, mixed>
220		*/
221	35	public function getSetProperties(): array
222		{
223	35	return $this->setProperties;
224		}
225
226		/**
227		* @throws NotInstantiableException
228		* @throws InvalidConfigException
229		*/
230	38	public function resolve(ContainerInterface $container): object
231		{
232	38	return ArrayDefinitionBuilder::getInstance()->build($container, $this);
233		}
234
235	2	public function merge(self $other): self
236		{
237	2	$callMethods = $this->getCallMethods();
238	2	foreach ($other->getCallMethods() as $method => $parameters) {
239	1	$callMethods[$method] = isset($callMethods[$method])
240	1	? $this->mergeParameters($callMethods[$method], $parameters)
241		: $parameters;
242		}
243	2	$mergedMethods = [];
244	2	foreach ($callMethods as $key => $value) {
245	2	$mergedMethods[$key . '()'] = $value;
246		}
247
248	2	$properties = array_merge($this->getSetProperties(), $other->getSetProperties());
249	2	$mergedProperties = [];
250	2	foreach ($properties as $key => $value) {
251		$mergedProperties['@' . $key] = $value;
252		}
253
254	2	return new self(array_merge([
255	2	self::CLASS_NAME => $other->getClass(),
256	2	self::CONSTRUCTOR => $this->mergeParameters(
257	2	$this->getConstructorParameters(),
258	2	$other->getConstructorParameters()
259		),
260		], $mergedMethods, $mergedProperties));
261		}
262
263	2	private function mergeParameters(array $selfParameters, array $otherParameters): array
264		{
265		/** @var mixed $parameter */
266	2	foreach ($otherParameters as $index => $parameter) {
267		/** @var mixed */
268	1	$selfParameters[$index] = $parameter;
269		}
270
271	2	return $selfParameters;
272		}
273
274		/**
275		* @param mixed $value
276		*/
277	2	private function getType($value): string
278		{
279	2	return is_object($value) ? get_class($value) : gettype($value);
280		}
281		}
282

yiisoft / factory

Pull Request — master (#78)

ArrayDefinition::setCallMethods() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like