Compiler::addFunction() - Code Metrics - Inspection of "Merge pull request #90 from ddmler/argument-unpack..." - ovr/phpsa - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( b59b39...7e1c48 )

by Дмитрий

created 2016-09-06 04:29 UTC

Compiler::addFunction() A

↳ Parent: Compiler

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	2
c	0
b	0
f	0
nc	1
nop	1
dl	0
loc	4
rs	10
ccs	3
cts	3
cp	1
crap	1

<?php
/**
 * @author Patsura Dmitry https://github.com/ovr <[email protected]>
 */

namespace PHPSA;

use PHPSA\Definition\ClassDefinition;
use PHPSA\Definition\FunctionDefinition;
use PHPSA\Definition\RuntimeClassDefinition;
use ReflectionClass;

class Compiler
{
    /**
     * @var ClassDefinition[]
     */
    protected $classes = array();

    /**
     * @var FunctionDefinition[]
     */
    protected $functions = array();

    /**
     * @param ClassDefinition $class
     */
    public function addClass(ClassDefinition $class)
    {
        $this->classes[implode('\\', [$class->getNamespace(), $class->getName()])] = $class;
    }

    /**
     * @param FunctionDefinition $function
     */
    public function addFunction(FunctionDefinition $function)
    {
        $this->functions[] = $function;
    }

    /**
     * @param Context $context
     */
    public function compile(Context $context)
    {
        $context->scopePointer = null;

        /**
         * @todo Implement class map...
         */
        foreach ($this->classes as $class) {
            $extends = $class->getExtendsClass();
            if ($extends) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
                if (isset($this->classes[$extends])) {
                    $class->setExtendsClassDefinition($this->classes[$extends]);
                } else {
                    if (class_exists($extends, true)) {
                        $class->setExtendsClassDefinition(
                            new RuntimeClassDefinition(
                                new ReflectionClass(
                                    $extends
                                )
                            )
                        );
                    }
                }
            }
        }

        foreach ($this->functions as $function) {
            /**
             * @todo Configuration
             *
             * Ignore functions compiling from vendor
             */
            $checkVendor = strpos($function->getFilepath(), './vendor');
            if ($checkVendor !== false && $checkVendor < 3) {
                continue;
            }

            $function->compile($context);
        }

        foreach ($this->classes as $class) {
            /**
             * @todo Configuration
             *
             * Ignore Classes compiling from vendor
             */
            $checkVendor = strpos($class->getFilepath(), './vendor');
            if ($checkVendor !== false && $checkVendor < 3) {
                continue;
            }

            $class->compile($context);
        }
    }

    /**
     * Try to find function with $namespace from pre-compiled function(s)
     *
     * @param string $name
     * @param string|null $namespace
     * @return bool|FunctionDefinition
     */
    public function getFunctionNS($name, $namespace = null)
    {
        foreach ($this->functions as $function) {
            if ($function->getName() == $name && $function->getNamespace() == $namespace) {
                return $function;
            }
        }

        return false;
    }

    /**
     * Try to find function from pre-compiled function(s)
     *
     * @param string $name
     * @return bool|FunctionDefinition
     */
    public function getFunction($name)
    {
        foreach ($this->functions as $function) {
            if ($function->getName() == $name) {
                return $function;
            }
        }

        return false;
    }
}


1		<?php
2		/**
3		* @author Patsura Dmitry https://github.com/ovr <[email protected]>
4		*/
5
6		namespace PHPSA;
7
8		use PHPSA\Definition\ClassDefinition;
9		use PHPSA\Definition\FunctionDefinition;
10		use PHPSA\Definition\RuntimeClassDefinition;
11		use ReflectionClass;
12
13		class Compiler
14		{
15		/**
16		* @var ClassDefinition[]
17		*/
18		protected $classes = array();
19
20		/**
21		* @var FunctionDefinition[]
22		*/
23		protected $functions = array();
24
25		/**
26		* @param ClassDefinition $class
27		*/
28	10	public function addClass(ClassDefinition $class)
29		{
30	10	$this->classes[implode('\\', [$class->getNamespace(), $class->getName()])] = $class;
31	10	}
32
33		/**
34		* @param FunctionDefinition $function
35		*/
36	1	public function addFunction(FunctionDefinition $function)
37		{
38	1	$this->functions[] = $function;
39	1	}
40
41		/**
42		* @param Context $context
43		*/
44	10	public function compile(Context $context)
45		{
46	10	$context->scopePointer = null;
47
48		/**
49		* @todo Implement class map...
50		*/
51	10	foreach ($this->classes as $class) {
52	10	$extends = $class->getExtendsClass();
53	10	if ($extends) {
		0 ignored issues – show Bug Best Practice introduced 2015-11-22 15:25 UTC by Report Bug Copy Issue Report The expression `$extends` of type `string\|null` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` 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...
54		if (isset($this->classes[$extends])) {
55		$class->setExtendsClassDefinition($this->classes[$extends]);
56		} else {
57		if (class_exists($extends, true)) {
58		$class->setExtendsClassDefinition(
59		new RuntimeClassDefinition(
60		new ReflectionClass(
61		$extends
62		)
63		)
64		);
65		}
66		}
67		}
68	10	}
69
70	10	foreach ($this->functions as $function) {
71		/**
72		* @todo Configuration
73		*
74		* Ignore functions compiling from vendor
75		*/
76	1	$checkVendor = strpos($function->getFilepath(), './vendor');
77	1	if ($checkVendor !== false && $checkVendor < 3) {
78		continue;
79		}
80
81	1	$function->compile($context);
82	10	}
83
84	10	foreach ($this->classes as $class) {
85		/**
86		* @todo Configuration
87		*
88		* Ignore Classes compiling from vendor
89		*/
90	10	$checkVendor = strpos($class->getFilepath(), './vendor');
91	10	if ($checkVendor !== false && $checkVendor < 3) {
92		continue;
93		}
94
95	10	$class->compile($context);
96	10	}
97	10	}
98
99		/**
100		* Try to find function with $namespace from pre-compiled function(s)
101		*
102		* @param string $name
103		* @param string\|null $namespace
104		* @return bool\|FunctionDefinition
105		*/
106	8	public function getFunctionNS($name, $namespace = null)
107		{
108	8	foreach ($this->functions as $function) {
109	1	if ($function->getName() == $name && $function->getNamespace() == $namespace) {
110		return $function;
111		}
112	8	}
113
114	8	return false;
115		}
116
117		/**
118		* Try to find function from pre-compiled function(s)
119		*
120		* @param string $name
121		* @return bool\|FunctionDefinition
122		*/
123	1	public function getFunction($name)
124		{
125	1	foreach ($this->functions as $function) {
126	1	if ($function->getName() == $name) {
127		return $function;
128		}
129	1	}
130
131	1	return false;
132		}
133		}
134

ovr / phpsa

Push — master ( b59b39...7e1c48 )

Compiler::addFunction() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like