SetterMethod::__invoke() - Code Metrics - Inspection of "Implements visitor pattern" - ray-di/Ray.Di - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — 2.x (#289)

by Akihito

created 2024-05-22 22:16 UTC

SetterMethod::__invoke() A

↳ Parent: SetterMethod

Complexity

Conditions	4
Paths	4

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	10
dl	0
loc	18
c	0
b	0
f	0
rs	9.9332
cc	4
nc	4
nop	2

<?php

declare(strict_types=1);

namespace Ray\Di;

use Exception;
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php
use LogicException;
use Ray\Di\Exception\Unbound;
use ReflectionMethod;

use function call_user_func_array;
use function is_callable;

final class SetterMethod implements AcceptInterface
{
    /** @var string */
    private $method;

    /** @var Arguments */
    private $arguments;

    /**
     * Is optional binding ?
     *
     * @var bool
     */
    private $isOptional = false;

    public function __construct(ReflectionMethod $method, Name $name)
    {
        $this->method = $method->name;
        $this->arguments = new Arguments($method, $name);
    }

    /**
     * @param object $instance
     *
     * @throws Unbound
     * @throws Exception
     */
    public function __invoke($instance, Container $container): void
    {
        try {
            $parameters = $this->arguments->inject($container);
        } catch (Unbound $e) {
            if ($this->isOptional) {
                return;
            }

            throw $e;
        }

        $callable = [$instance, $this->method];
        if (! is_callable($callable)) {
            throw new LogicException(); // @codeCoverageIgnore
        }

        call_user_func_array($callable, $parameters);
    }

    public function setOptional(): void
    {
        $this->isOptional = true;
    }

    /** @inheritDoc */
    public function accept(VisitorInterface $visitor)
    {
        $visitor->visitSetterMethod($this->method, $this->arguments);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Ray\Di;
6
7			use Exception;
			0 ignored issues – show Bug introduced 2021-10-31 07:12 UTC by Report Bug Copy Issue Report This use statement conflicts with another class in this namespace, `Ray\Di\Exception`. Consider defining an alias. Let?s assume that you have a directory layout like this: . \|-- OtherDir \| \|-- Bar.php \| `-- Foo.php `-- SomeDir `-- Foo.php and let?s assume the following content of `Bar.php`: // Bar.php namespace OtherDir; use SomeDir\Foo; // This now conflicts the class OtherDir\Foo If both files `OtherDir/Foo.php` and `SomeDir/Foo.php` are loaded in the same runtime, you will see a PHP error such as the following: PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php However, as `OtherDir/Foo.php` does not necessarily have to be loaded and the error is only triggered if it is loaded before `OtherDir/Bar.php`, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias: // Bar.php namespace OtherDir; use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore. Loading history...
8			use LogicException;
9			use Ray\Di\Exception\Unbound;
10			use ReflectionMethod;
11
12			use function call_user_func_array;
13			use function is_callable;
14
15			final class SetterMethod implements AcceptInterface
16			{
17			/** @var string */
18			private $method;
19
20			/** @var Arguments */
21			private $arguments;
22
23			/**
24			* Is optional binding ?
25			*
26			* @var bool
27			*/
28			private $isOptional = false;
29
30			public function __construct(ReflectionMethod $method, Name $name)
31			{
32			$this->method = $method->name;
33			$this->arguments = new Arguments($method, $name);
34			}
35
36			/**
37			* @param object $instance
38			*
39			* @throws Unbound
40			* @throws Exception
41			*/
42			public function __invoke($instance, Container $container): void
43			{
44			try {
45			$parameters = $this->arguments->inject($container);
46			} catch (Unbound $e) {
47			if ($this->isOptional) {
48			return;
49			}
50
51			throw $e;
52			}
53
54			$callable = [$instance, $this->method];
55			if (! is_callable($callable)) {
56			throw new LogicException(); // @codeCoverageIgnore
57			}
58
59			call_user_func_array($callable, $parameters);
60			}
61
62			public function setOptional(): void
63			{
64			$this->isOptional = true;
65			}
66
67			/** @inheritDoc */
68			public function accept(VisitorInterface $visitor)
69			{
70			$visitor->visitSetterMethod($this->method, $this->arguments);
71			}
72			}
73

ray-di / Ray.Di

Pull Request — 2.x (#289)

SetterMethod::__invoke() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like