ClosureDefinition::run() - Code Metrics - ovr/phpsa - Measure and Improve Code Quality continuously with Scrutinizer

ClosureDefinition::run() A
last analyzed 2019-02-27 06:44 UTC

↳ Parent: ClosureDefinition

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

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

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

namespace PHPSA\Definition;

use PHPSA\CompiledExpression;
use PHPSA\Compiler\SymbolTable;
use PHPSA\Context;
use PhpParser\Node;
use PHPSA\Compiler\Parameter;
use PHPSA\Compiler\Types;
use PHPSA\Variable;

/**
 * Closure Definition
 *
 * @package PHPSA\Definition
 */
class ClosureDefinition extends ParentDefinition
{
    /**
     * @todo Use Finder
     *
     * @var string
     */
    protected $filepath;

    /**
     * @var Node\Expr\Closure
     */
    protected $statement;

    /**
     * @var int
     */
    protected $returnTypes = CompiledExpression::MIXED;

    /**
     * @var array
     */
    protected $possibleReturnTypes = [];

    /**
     * @var SymbolTable
     */
    protected $symbolTable;

    /**
     * @param Node\Expr\Closure $statement
     */
    public function __construct(Node\Expr\Closure $statement)
    {
        $this->symbolTable = new SymbolTable();
        $this->statement = $statement;
    }

    /**
     * @param Context $context
     */
    public function preCompile(Context $context)
    {
        if ($this->statement->uses) {

            /**
             * Store variables from User to next restore Context
             */
            foreach ($this->statement->uses as $variable) {
                $variable = $context->getSymbol($variable->var);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
                if ($variable) {
                    $variable->incGets();

                    $this->symbolTable->add(clone $variable);
                }
            }
        }
    }

    /**
     * Compile function to check it
     *
     * @param Context $context
     * @return bool
     */
    public function compile(Context $context)

    {
        if ($this->compiled) {
            return true;
        }

        $context->setFilepath($this->filepath);
        $this->compiled = true;

        $context->clearSymbols();

        // @todo rewrite when we will use symbol table in all places!
        foreach ($this->symbolTable->getVariables() as $variable) {
            $context->addVariable($variable);
        }

        if ($context->scopePointer) {
            if ($context->scopePointer->isClassMethod()) {
                /** @var \PHPSA\Definition\ClassMethod $method */
                $method = $context->scopePointer->getObject();

                if (!$method->isStatic()) {
                    $thisPtr = new Variable('this', $this, CompiledExpression::OBJECT);
                    $thisPtr->incGets();

                    $this->symbolTable->add($thisPtr);
                    $context->addVariable($thisPtr);
                }
            } elseif ($context->scopePointer->isClosure()) {
                /** @var \PHPSA\Definition\ClosureDefinition $closure */
                $closure = $context->scopePointer->getObject();

                $thisPtr = $closure->getSymbolTable()->get('this');
                if ($thisPtr) {
                    $thisPtr->incGets();
                    $context->addVariable($thisPtr);
                }
            }
        }

        $context->scopePointer = $this->getPointer();
        $context->setScope(null);

        if (count($this->statement->stmts) == 0) {
            return $context->notice(
                'not-implemented-function',
                sprintf('Closure %s() is not implemented', $this->name),
                $this->statement
            );
        }

        if (count($this->statement->params) > 0) {
            /** @var  Node\Param $parameter */
            foreach ($this->statement->params as $parameter) {
                $type = CompiledExpression::UNKNOWN;

                if ($parameter->type) {
                    if (is_string($parameter->type)) {
                        $type = Types::getType($parameter->type);
                    } elseif ($parameter->type instanceof Node\Name) {
                        $type = CompiledExpression::OBJECT;
                    }
                }

                $context->addVariable(
                    new Parameter($parameter->name, null, $type, $parameter->byRef)
                );
            }
        }

        foreach ($this->statement->stmts as $st) {
            \PHPSA\nodeVisitorFactory($st, $context);
        }

        return true;
    }

    /**
     * @param CompiledExpression[] $arguments
     * @param Context $context
     * @return CompiledExpression
     */
    public function run(array $arguments, Context $context)

    {
        return new CompiledExpression();
    }

    /**
     * @return string
     */
    public function getFilepath()
    {
        return $this->filepath;
    }

    /**
     * @param string $filepath
     */
    public function setFilepath($filepath)
    {
        $this->filepath = $filepath;
    }

    /**
     * @return string
     */
    public function getNamespace()
    {
        return $this->namespace;
    }

    /**
     * @return SymbolTable
     */
    public function getSymbolTable()
    {
        return $this->symbolTable;
    }
}


1			<?php
2			/**
3			* @author Patsura Dmitry https://github.com/ovr <[email protected]>
4			*/
5
6			namespace PHPSA\Definition;
7
8			use PHPSA\CompiledExpression;
9			use PHPSA\Compiler\SymbolTable;
10			use PHPSA\Context;
11			use PhpParser\Node;
12			use PHPSA\Compiler\Parameter;
13			use PHPSA\Compiler\Types;
14			use PHPSA\Variable;
15
16			/**
17			* Closure Definition
18			*
19			* @package PHPSA\Definition
20			*/
21			class ClosureDefinition extends ParentDefinition
22			{
23			/**
24			* @todo Use Finder
25			*
26			* @var string
27			*/
28			protected $filepath;
29
30			/**
31			* @var Node\Expr\Closure
32			*/
33			protected $statement;
34
35			/**
36			* @var int
37			*/
38			protected $returnTypes = CompiledExpression::MIXED;
39
40			/**
41			* @var array
42			*/
43			protected $possibleReturnTypes = [];
44
45			/**
46			* @var SymbolTable
47			*/
48			protected $symbolTable;
49
50			/**
51			* @param Node\Expr\Closure $statement
52			*/
53			public function __construct(Node\Expr\Closure $statement)
54			{
55			$this->symbolTable = new SymbolTable();
56			$this->statement = $statement;
57			}
58
59			/**
60			* @param Context $context
61			*/
62			public function preCompile(Context $context)
63			{
64			if ($this->statement->uses) {
			0 ignored issues – show Bug Best Practice introduced 2016-06-11 13:21 UTC by Report Bug Copy Issue Report The expression `$this->statement->uses` of type `PhpParser\Node\Expr\ClosureUse[]` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
65			/**
66			* Store variables from User to next restore Context
67			*/
68			foreach ($this->statement->uses as $variable) {
69			$variable = $context->getSymbol($variable->var);
			0 ignored issues – show Bug introduced 2016-06-11 13:21 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$variable` is correct as `$context->getSymbol($variable->var)` (which targets `PHPSA\Context::getSymbol()`) seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
70			if ($variable) {
71			$variable->incGets();
72
73			$this->symbolTable->add(clone $variable);
74			}
75			}
76			}
77			}
78
79			/**
80			* Compile function to check it
81			*
82			* @param Context $context
83			* @return bool
84			*/
85			public function compile(Context $context)
			0 ignored issues – show Complexity introduced 2017-08-04 15:23 UTC by Report Bug Copy Issue Report This operation has 576 execution paths which exceeds the configured maximum of 200. A high number of execution paths generally suggests many nested conditional statements and make the code less readible. This can usually be fixed by splitting the method into several smaller methods. You can also find more information in the “Code” section of your repository. Loading history...
86			{
87			if ($this->compiled) {
88			return true;
89			}
90
91			$context->setFilepath($this->filepath);
92			$this->compiled = true;
93
94			$context->clearSymbols();
95
96			// @todo rewrite when we will use symbol table in all places!
97			foreach ($this->symbolTable->getVariables() as $variable) {
98			$context->addVariable($variable);
99			}
100
101			if ($context->scopePointer) {
102			if ($context->scopePointer->isClassMethod()) {
103			/** @var \PHPSA\Definition\ClassMethod $method */
104			$method = $context->scopePointer->getObject();
105
106			if (!$method->isStatic()) {
107			$thisPtr = new Variable('this', $this, CompiledExpression::OBJECT);
108			$thisPtr->incGets();
109
110			$this->symbolTable->add($thisPtr);
111			$context->addVariable($thisPtr);
112			}
113			} elseif ($context->scopePointer->isClosure()) {
114			/** @var \PHPSA\Definition\ClosureDefinition $closure */
115			$closure = $context->scopePointer->getObject();
116
117			$thisPtr = $closure->getSymbolTable()->get('this');
118			if ($thisPtr) {
119			$thisPtr->incGets();
120			$context->addVariable($thisPtr);
121			}
122			}
123			}
124
125			$context->scopePointer = $this->getPointer();
126			$context->setScope(null);
127
128			if (count($this->statement->stmts) == 0) {
129			return $context->notice(
130			'not-implemented-function',
131			sprintf('Closure %s() is not implemented', $this->name),
132			$this->statement
133			);
134			}
135
136			if (count($this->statement->params) > 0) {
137			/** @var Node\Param $parameter */
138			foreach ($this->statement->params as $parameter) {
139			$type = CompiledExpression::UNKNOWN;
140
141			if ($parameter->type) {
142			if (is_string($parameter->type)) {
143			$type = Types::getType($parameter->type);
144			} elseif ($parameter->type instanceof Node\Name) {
145			$type = CompiledExpression::OBJECT;
146			}
147			}
148
149			$context->addVariable(
150			new Parameter($parameter->name, null, $type, $parameter->byRef)
151			);
152			}
153			}
154
155			foreach ($this->statement->stmts as $st) {
156			\PHPSA\nodeVisitorFactory($st, $context);
157			}
158
159			return true;
160			}
161
162			/**
163			* @param CompiledExpression[] $arguments
164			* @param Context $context
165			* @return CompiledExpression
166			*/
167			public function run(array $arguments, Context $context)
			0 ignored issues – show Unused Code introduced 2017-08-04 15:23 UTC by Report Bug Copy Issue Report The parameter `$arguments` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2017-08-04 15:23 UTC by Report Bug Copy Issue Report The parameter `$context` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
168			{
169			return new CompiledExpression();
170			}
171
172			/**
173			* @return string
174			*/
175			public function getFilepath()
176			{
177			return $this->filepath;
178			}
179
180			/**
181			* @param string $filepath
182			*/
183			public function setFilepath($filepath)
184			{
185			$this->filepath = $filepath;
186			}
187
188			/**
189			* @return string
190			*/
191			public function getNamespace()
192			{
193			return $this->namespace;
194			}
195
196			/**
197			* @return SymbolTable
198			*/
199			public function getSymbolTable()
200			{
201			return $this->symbolTable;
202			}
203			}
204

ovr / phpsa

ClosureDefinition::run() A last analyzed 2019-02-27 06:44 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

ClosureDefinition::run() A
last analyzed 2019-02-27 06:44 UTC