NullObjectDependency::toNull() - Code Metrics - Inspection of "Null Object binding" - ray-di/Ray.Di - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — 2.x (#237)

by Akihito

created 2021-02-12 13:08 UTC

NullObjectDependency::toNull() A

↳ Parent: NullObjectDependency

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	9
rs	9.9666
c	0
b	0
f	0
cc	1
nc	1
nop	1

<?php

declare(strict_types=1);

namespace Ray\Di;

use Koriym\NullObject\NullObject;
use Ray\Compiler\FilePutContents;
use Ray\Di\Annotation\ScriptDir;
use ReflectionClass;

use function assert;
use function class_exists;
use function interface_exists;
use function is_dir;
use function is_string;
use function sprintf;
use function str_replace;

/**
 * @codeCoverageIgnore
 */
final class NullObjectDependency implements DependencyInterface
{
    /** @var class-string */
    private $interface;

    /**
     * @param class-string $interface

     */
    public function __construct(string $interface)
    {
        $this->interface = $interface;
    }

    /**
     * {@inheritdoc}
     */
    public function __toString(): string
    {
        return '';
    }

    /**
     * {@inheritdoc}
     */
    public function inject(Container $container)
    {
        return null;
    }

    /**
     * {@inheritdoc}
     *
     * @return void
     */
    public function register(array &$container, Bind $bind)
    {
        $container[(string) $bind] = $bind->getBound();
    }

    /**
     * {@inheritdoc}
     */
    public function setScope($scope)
    {
    }

    public function toNull(string $scriptDir): Dependency
    {
        assert(is_dir($scriptDir));
        $nullObject = new NullObject();
        $class = $nullObject->save($this->interface, $scriptDir);
        $file = sprintf('%s/%s.php', $scriptDir, str_replace('\\', '_', $class));
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        return new Dependency(new NewInstance(new ReflectionClass($class), new SetterMethods([])));
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Ray\Di;
6
7			use Koriym\NullObject\NullObject;
8			use Ray\Compiler\FilePutContents;
9			use Ray\Di\Annotation\ScriptDir;
10			use ReflectionClass;
11
12			use function assert;
13			use function class_exists;
14			use function interface_exists;
15			use function is_dir;
16			use function is_string;
17			use function sprintf;
18			use function str_replace;
19
20			/**
21			* @codeCoverageIgnore
22			*/
23			final class NullObjectDependency implements DependencyInterface
24			{
25			/** @var class-string */
26			private $interface;
27
28			/**
29			* @param class-string $interface
			0 ignored issues – show Documentation introduced 2021-02-10 15:53 UTC by Report Bug Copy Issue Report The doc-type `class-string` could not be parsed: Unknown type name "class-string" at position 0. (view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
30			*/
31			public function __construct(string $interface)
32			{
33			$this->interface = $interface;
34			}
35
36			/**
37			* {@inheritdoc}
38			*/
39			public function __toString(): string
40			{
41			return '';
42			}
43
44			/**
45			* {@inheritdoc}
46			*/
47			public function inject(Container $container)
48			{
49			return null;
50			}
51
52			/**
53			* {@inheritdoc}
54			*
55			* @return void
56			*/
57			public function register(array &$container, Bind $bind)
58			{
59			$container[(string) $bind] = $bind->getBound();
60			}
61
62			/**
63			* {@inheritdoc}
64			*/
65			public function setScope($scope)
66			{
67			}
68
69			public function toNull(string $scriptDir): Dependency
70			{
71			assert(is_dir($scriptDir));
72			$nullObject = new NullObject();
73			$class = $nullObject->save($this->interface, $scriptDir);
74			$file = sprintf('%s/%s.php', $scriptDir, str_replace('\\', '_', $class));
			0 ignored issues – show Unused Code introduced 2021-02-12 12:57 UTC by Report Bug Copy Issue Report `$file` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
75
76			return new Dependency(new NewInstance(new ReflectionClass($class), new SetterMethods([])));
77			}
78			}
79

ray-di / Ray.Di

Pull Request — 2.x (#237)

NullObjectDependency::toNull() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like