Parameter::typeDoesNotCoincideWithDocBlock() - Code Metrics - marcosh/php-type-checker - Measure and Improve Code Quality continuously with Scrutinizer

Parameter::typeDoesNotCoincideWithDocBlock() B
last analyzed 2017-05-16 15:10 UTC

↳ Parent: Parameter

Complexity

Conditions	11
Paths	67

Size

Total Lines	17
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	17
rs	7.1162
c	0
b	0
f	0
cc	11
eloc	13
nc	67
nop	0

How to fix Complexity

<?php

declare(strict_types=1);

namespace Marcosh\PhpTypeChecker\Check;

use phpDocumentor\Reflection\Type;
use phpDocumentor\Reflection\Types\Array_;
use phpDocumentor\Reflection\Types\Mixed;
use phpDocumentor\Reflection\Types\Object_;
use phpDocumentor\Reflection\Types\Self_;
use Roave\BetterReflection\Reflection\ReflectionParameter;

final class Parameter
{
    /**
     * @var ReflectionParameter
     */
    private $parameter;

    public function __construct(ReflectionParameter $parameter)
    {
        $this->parameter = $parameter;
    }

    /**
     * @return Type[]
     */
    private function docBlockTypes(): array
    {
        $docBlockTypes = [];

        try {
            $docBlockTypes = $this->parameter->getDocBlockTypes();
        } catch (\InvalidArgumentException $e) {
            // we need this here to prevent reflection-bocblock errors on @see invalid Fqsen
        }

        return $docBlockTypes;
    }

    public function typeIsMissing(): bool
    {
        return null === $this->parameter->getTypeHint() &&
            empty($this->docBlockTypes());
    }

    public function typeIsMissingWithDocBlock(): bool
    {
        return null === $this->parameter->getTypeHint() &&
            !empty($this->docBlockTypes()) &&
            !$this->docBlockTypes()[0] instanceof Mixed &&
            !$this->docBlockTypes()[0] instanceof Object_;
    }

    public function typeDoesNotCoincideWithDocBlock(): bool
    {
        $docBlockTypes = $this->docBlockTypes();

        return !empty($docBlockTypes) &&
            $this->parameter->getTypeHint() instanceof Type &&
            !in_array($this->parameter->getTypeHint(), $docBlockTypes, false) &&
            !($this->parameter->getTypeHint() instanceof Array_ && $docBlockTypes[0] instanceof Array_) &&
            !($this->parameter->getTypeHint() instanceof Self_ &&
                $docBlockTypes[0] instanceof Object_ &&
                $this->parameter->getDeclaringClass()->getName() === ltrim((string) $docBlockTypes[0]->getFqsen(), '\\')
            ) &&
            !($this->parameter->getTypeHint() instanceof Object_ &&
                $docBlockTypes[0] instanceof Self_ &&
                $this->parameter->getDeclaringClass()->getName() === ltrim((string) $this->parameter->getTypeHint()->getFqsen(), '\\')
            );
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Marcosh\PhpTypeChecker\Check;
6
7			use phpDocumentor\Reflection\Type;
8			use phpDocumentor\Reflection\Types\Array_;
9			use phpDocumentor\Reflection\Types\Mixed;
10			use phpDocumentor\Reflection\Types\Object_;
11			use phpDocumentor\Reflection\Types\Self_;
12			use Roave\BetterReflection\Reflection\ReflectionParameter;
13
14			final class Parameter
15			{
16			/**
17			* @var ReflectionParameter
18			*/
19			private $parameter;
20
21			public function __construct(ReflectionParameter $parameter)
22			{
23			$this->parameter = $parameter;
24			}
25
26			/**
27			* @return Type[]
28			*/
29			private function docBlockTypes(): array
30			{
31			$docBlockTypes = [];
32
33			try {
34			$docBlockTypes = $this->parameter->getDocBlockTypes();
35			} catch (\InvalidArgumentException $e) {
36			// we need this here to prevent reflection-bocblock errors on @see invalid Fqsen
37			}
38
39			return $docBlockTypes;
40			}
41
42			public function typeIsMissing(): bool
43			{
44			return null === $this->parameter->getTypeHint() &&
45			empty($this->docBlockTypes());
46			}
47
48			public function typeIsMissingWithDocBlock(): bool
49			{
50			return null === $this->parameter->getTypeHint() &&
51			!empty($this->docBlockTypes()) &&
52			!$this->docBlockTypes()[0] instanceof Mixed &&
53			!$this->docBlockTypes()[0] instanceof Object_;
54			}
55
56			public function typeDoesNotCoincideWithDocBlock(): bool
57			{
58			$docBlockTypes = $this->docBlockTypes();
59
60			return !empty($docBlockTypes) &&
61			$this->parameter->getTypeHint() instanceof Type &&
62			!in_array($this->parameter->getTypeHint(), $docBlockTypes, false) &&
63			!($this->parameter->getTypeHint() instanceof Array_ && $docBlockTypes[0] instanceof Array_) &&
64			!($this->parameter->getTypeHint() instanceof Self_ &&
65			$docBlockTypes[0] instanceof Object_ &&
66			$this->parameter->getDeclaringClass()->getName() === ltrim((string) $docBlockTypes[0]->getFqsen(), '\\')
67			) &&
68			!($this->parameter->getTypeHint() instanceof Object_ &&
69			$docBlockTypes[0] instanceof Self_ &&
70			$this->parameter->getDeclaringClass()->getName() === ltrim((string) $this->parameter->getTypeHint()->getFqsen(), '\\')
71			);
72			}
73			}
74

marcosh / php-type-checker

Parameter::typeDoesNotCoincideWithDocBlock() B last analyzed 2017-05-16 15:10 UTC

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like

Parameter::typeDoesNotCoincideWithDocBlock() B
last analyzed 2017-05-16 15:10 UTC