LambdaExpHandler::evaluate() - Code Metrics - mattwellss/sphpeme - Measure and Improve Code Quality continuously with Scrutinizer

LambdaExpHandler::evaluate() A
last analyzed 2019-11-08 19:48 UTC

↳ Parent: LambdaExpHandler

Complexity

Conditions	3
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	3

Importance

Changes

Metric	Value
cc	3
nc	1
nop	3
dl	0
loc	16
ccs	10
cts	10
cp	1
crap	3
rs	9.7333
c	0
b	0
f	0

<?php

namespace Sphpeme\ExpHandler;


use Sphpeme\Env;
use function Sphpeme\env_extend;
use Sphpeme\EnvExtender;
use Sphpeme\Evaluator;
use Sphpeme\Symbol;

class LambdaExpHandler implements ExpHandler
{
    private $lambdaSymbol;
    /**
     * @var EnvExtender
     */
    private $envExtender;

    public function __construct(EnvExtender $envExtender)
    {
        $this->lambdaSymbol = Symbol::make('lambda');
        $this->envExtender = $envExtender;
    }

    public function evaluate($exp, Env\EnvInterface $env, Evaluator $evaluate)
    {
        list($lambda, $params) = $exp;
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;
        $body = \array_slice($exp, 2);
        return function (...$args) use ($env, $body, $params, $evaluate) {
            if (\count($params)) {
                $env = ($this->envExtender)($env, new Env\SimpleEnv(array_combine($params, $args)));
$x = 1;
$callable = function() use ($x) {
    $x = 2; // Not visible in outer scope. If you would like this, how
            // about using a different variable name than $x?
};

$callable();
var_dump($x); // integer(1)
            }

            while (\count($body) > 1) {
                $evaluate(array_shift($body), $env);
            }

            return $evaluate(array_shift($body), $env);
        };
    }

    public function handles($exp): bool
    {
        return $exp[0] === $this->lambdaSymbol;
    }
}

GitHub Access Token became invalid

LambdaExpHandler::evaluate() A
last analyzed 2019-11-08 19:48 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Change not visible in outer-scope

Change visible in outer-scope

1		<?php
2
3		namespace Sphpeme\ExpHandler;
4
5
6		use Sphpeme\Env;
7		use function Sphpeme\env_extend;
8		use Sphpeme\EnvExtender;
9		use Sphpeme\Evaluator;
10		use Sphpeme\Symbol;
11
12		class LambdaExpHandler implements ExpHandler
13		{
14		private $lambdaSymbol;
15		/**
16		* @var EnvExtender
17		*/
18		private $envExtender;
19
20	3	public function __construct(EnvExtender $envExtender)
21		{
22	3	$this->lambdaSymbol = Symbol::make('lambda');
23	3	$this->envExtender = $envExtender;
24	3	}
25
26	2	public function evaluate($exp, Env\EnvInterface $env, Evaluator $evaluate)
27		{
28	2	list($lambda, $params) = $exp;
		0 ignored issues – show Unused Code introduced 2017-12-23 06:54 UTC by Report Bug Copy Issue Report The assignment to `$lambda` is unused. Consider omitting it like so `list($first,,$third)`. This checks looks for assignemnts to variables using the `list(...)` function, where not all assigned variables are subsequently used. Consider the following code example. <?php function returnThreeValues() { return array('a', 'b', 'c'); } list($a, $b, $c) = returnThreeValues(); print $a . " - " . $c; Only the variables `$a` and `$c` are used. There was no need to assign `$b`. Instead, the list call could have been. list($a,, $c) = returnThreeValues(); Loading history...
29	2	$body = \array_slice($exp, 2);
30	2	return function (...$args) use ($env, $body, $params, $evaluate) {
31	2	if (\count($params)) {
32	1	$env = ($this->envExtender)($env, new Env\SimpleEnv(array_combine($params, $args)));
		0 ignored issues – show Bug introduced 2017-12-23 06:54 UTC by Report Bug Copy Issue Report Consider using a different name than the imported variable `$env`, or did you forget to import by reference? It seems like you are assigning to a variable which was imported through a `use` statement which was not imported by reference. For clarity, we suggest to use a different name or import by reference depending on whether you would like to have the change visibile in outer-scope. Change not visible in outer-scope $x = 1; $callable = function() use ($x) { $x = 2; // Not visible in outer scope. If you would like this, how // about using a different variable name than $x? }; $callable(); var_dump($x); // integer(1) Change visible in outer-scope $x = 1; $callable = function() use (&$x) { $x = 2; }; $callable(); var_dump($x); // integer(2) Loading history...
33		}
34
35	2	while (\count($body) > 1) {
36	2	$evaluate(array_shift($body), $env);
37		}
38
39	2	return $evaluate(array_shift($body), $env);
40	2	};
41		}
42
43	1	public function handles($exp): bool
44		{
45	1	return $exp[0] === $this->lambdaSymbol;
46		}
47		}

mattwellss / sphpeme

GitHub Access Token became invalid

LambdaExpHandler::evaluate() A last analyzed 2019-11-08 19:48 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Change not visible in outer-scope

Change visible in outer-scope

Duplication Side-by-Side

Filter issues like

LambdaExpHandler::evaluate() A
last analyzed 2019-11-08 19:48 UTC