CodeGenerator::getDeclarationLine() - Code Metrics - mtymek/class-dumper - Measure and Improve Code Quality continuously with Scrutinizer

CodeGenerator::getDeclarationLine() C
last analyzed 2016-06-26 12:14 UTC

↳ Parent: CodeGenerator

Complexity

Conditions	13
Paths	96

Size

Total Lines	41
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	41
rs	5.1234
cc	13
eloc	30
nc	96
nop	1

How to fix Complexity

<?php
/**
 * Class Dumper
 *
 * @link      https://github.com/mtymek/class-dumper
 * @copyright Copyright (c) 2015 Mateusz Tymek
 * @license   BSD 2-Clause
 */

namespace ClassDumper;

use Zend\Code\Reflection\ClassReflection;

class CodeGenerator
{
    /**
     * @param ClassReflection $class
     * @return array
     */
    private function getUseArray(ClassReflection $class)
    {
        $usesNames = [];
        foreach ($class->getDeclaringFile()->getUses() as $use) {
            $usesNames[$use['use']] = $use['as'];
        }
        return $usesNames;
    }

    /**
     * @param ClassReflection $class
     * @return string
     */
    public function getUseLines(ClassReflection $class)
    {
        $useLines = [];
        foreach ($this->getUseArray($class) as $name => $as) {
            $useString = "use $name";
            if ($as) {
                $useString .= " as $as";
            }
            $useLines[] = $useString . ';';
        }
        return $useLines;
    }

    /**
     * @param ClassReflection $class
     * @param string $namespaceName
     * @param array $useArray
     * @return string
     */
    private function getClassNameInContext(ClassReflection $class, $namespaceName, $useArray)
    {
        if (!$namespaceName) {
            return '\\' . $class->getName();

        }
        return array_key_exists($class->getName(), $useArray)

            ? ($useArray[$class->getName()] ?: $class->getShortName())

            : ((0 === strpos($class->getName(), $namespaceName))

                ? substr($class->getName(), strlen($namespaceName) + 1)

                : '\\' . $class->getName());

    }

    /**
     * @param ClassReflection $class
     * @return string
     */
    public function getDeclarationLine(ClassReflection $class)
    {
        $usesNames = $this->getUseArray($class);

        $declaration = '';
        if ($class->isAbstract() && !$class->isInterface() && !$class->isTrait()) {
            $declaration .= 'abstract ';
        }
        if ($class->isFinal()) {
            $declaration .= 'final ';
        }
        if ($class->isInterface()) {
            $declaration .= 'interface ';
        } elseif ($class->isTrait()) {
            $declaration .= 'trait ';
        } else {
            $declaration .= 'class ';
        }
        $declaration .= $class->getShortName();
        $parentName = false;
        if ($parent = $class->getParentClass()) {
            $parentName = $this->getClassNameInContext($parent, $class->getNamespaceName(), $usesNames);
        }
        if ($parentName) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $declaration .= " extends {$parentName}";
        }
        $interfaces = array_diff($class->getInterfaceNames(), $parent ? $parent->getInterfaceNames() : array());
        if (count($interfaces)) {
            foreach ($interfaces as $interface) {
                $iReflection = new ClassReflection($interface);
                $interfaces  = array_diff($interfaces, $iReflection->getInterfaceNames());
            }
            $declaration .= $class->isInterface() ? ' extends ' : ' implements ';
            $declaration .= implode(', ', array_map(function($interface) use ($usesNames, $class) {

                $iReflection = new ClassReflection($interface);
                return $this->getClassNameInContext($iReflection, $class->getNamespaceName(), $usesNames);
            }, $interfaces));
        }

        return $declaration;
    }
}


1			<?php
2			/**
3			* Class Dumper
4			*
5			* @link https://github.com/mtymek/class-dumper
6			* @copyright Copyright (c) 2015 Mateusz Tymek
7			* @license BSD 2-Clause
8			*/
9
10			namespace ClassDumper;
11
12			use Zend\Code\Reflection\ClassReflection;
13
14			class CodeGenerator
15			{
16			/**
17			* @param ClassReflection $class
18			* @return array
19			*/
20			private function getUseArray(ClassReflection $class)
21			{
22			$usesNames = [];
23			foreach ($class->getDeclaringFile()->getUses() as $use) {
24			$usesNames[$use['use']] = $use['as'];
25			}
26			return $usesNames;
27			}
28
29			/**
30			* @param ClassReflection $class
31			* @return string
32			*/
33			public function getUseLines(ClassReflection $class)
34			{
35			$useLines = [];
36			foreach ($this->getUseArray($class) as $name => $as) {
37			$useString = "use $name";
38			if ($as) {
39			$useString .= " as $as";
40			}
41			$useLines[] = $useString . ';';
42			}
43			return $useLines;
44			}
45
46			/**
47			* @param ClassReflection $class
48			* @param string $namespaceName
49			* @param array $useArray
50			* @return string
51			*/
52			private function getClassNameInContext(ClassReflection $class, $namespaceName, $useArray)
53			{
54			if (!$namespaceName) {
55			return '\\' . $class->getName();
			0 ignored issues – show Bug introduced 2016-06-26 12:14 UTC by Report Bug Copy Issue Report Consider using `$class->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
56			}
57			return array_key_exists($class->getName(), $useArray)
			0 ignored issues – show Bug introduced 2016-06-26 12:14 UTC by Report Bug Copy Issue Report Consider using `$class->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
58			? ($useArray[$class->getName()] ?: $class->getShortName())
			0 ignored issues – show Bug introduced 2016-06-26 12:14 UTC by Report Bug Copy Issue Report Consider using `$class->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
59			: ((0 === strpos($class->getName(), $namespaceName))
			0 ignored issues – show Bug introduced 2016-06-26 12:14 UTC by Report Bug Copy Issue Report Consider using `$class->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
60			? substr($class->getName(), strlen($namespaceName) + 1)
			0 ignored issues – show Bug introduced 2016-06-26 12:14 UTC by Report Bug Copy Issue Report Consider using `$class->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
61			: '\\' . $class->getName());
			0 ignored issues – show Bug introduced 2016-06-26 12:14 UTC by Report Bug Copy Issue Report Consider using `$class->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
62			}
63
64			/**
65			* @param ClassReflection $class
66			* @return string
67			*/
68			public function getDeclarationLine(ClassReflection $class)
69			{
70			$usesNames = $this->getUseArray($class);
71
72			$declaration = '';
73			if ($class->isAbstract() && !$class->isInterface() && !$class->isTrait()) {
74			$declaration .= 'abstract ';
75			}
76			if ($class->isFinal()) {
77			$declaration .= 'final ';
78			}
79			if ($class->isInterface()) {
80			$declaration .= 'interface ';
81			} elseif ($class->isTrait()) {
82			$declaration .= 'trait ';
83			} else {
84			$declaration .= 'class ';
85			}
86			$declaration .= $class->getShortName();
87			$parentName = false;
88			if ($parent = $class->getParentClass()) {
89			$parentName = $this->getClassNameInContext($parent, $class->getNamespaceName(), $usesNames);
90			}
91			if ($parentName) {
			0 ignored issues – show Bug Best Practice introduced 2016-06-26 12:14 UTC by Report Bug Copy Issue Report The expression `$parentName` of type `string\|false` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== false` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
92			$declaration .= " extends {$parentName}";
93			}
94			$interfaces = array_diff($class->getInterfaceNames(), $parent ? $parent->getInterfaceNames() : array());
95			if (count($interfaces)) {
96			foreach ($interfaces as $interface) {
97			$iReflection = new ClassReflection($interface);
98			$interfaces = array_diff($interfaces, $iReflection->getInterfaceNames());
99			}
100			$declaration .= $class->isInterface() ? ' extends ' : ' implements ';
101			$declaration .= implode(', ', array_map(function($interface) use ($usesNames, $class) {
			0 ignored issues – show Coding Style introduced 2016-06-26 12:14 UTC by Report Bug Copy Issue Report Expected 1 space after FUNCTION keyword; 0 found Loading history...
102			$iReflection = new ClassReflection($interface);
103			return $this->getClassNameInContext($iReflection, $class->getNamespaceName(), $usesNames);
104			}, $interfaces));
105			}
106
107			return $declaration;
108			}
109			}
110

mtymek / class-dumper

CodeGenerator::getDeclarationLine() C last analyzed 2016-06-26 12:14 UTC

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like

CodeGenerator::getDeclarationLine() C
last analyzed 2016-06-26 12:14 UTC