CodeBlock::getCode() - Code Metrics - Inspection of "Add support for extending examples, close #12" - hanneskod/readme-tester - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( abec3a...7e1653 )

by Hannes

created 2017-01-07 13:08 UTC

CodeBlock::getCode() A

↳ Parent: CodeBlock

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	0

<?php

namespace hanneskod\readmetester;

/**
 * Wrapps an executable block of code
 */
class CodeBlock
{
    /**
     * @var string The contained code
     */
    private $code;

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

    /**
     * Grab contained code
     *
     * @return string
     */
    public function getCode()
    {
        return $this->code;
    }

    /**
     * Execute code block
     *
     * @return Result The result of the executed code
     */
    public function execute()
    {
        $returnValue = '';
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $exception = null;

        ob_start();

        try {
            $returnValue = eval($this->code);

        } catch (\Exception $e) {
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
            $exception = $e;
        }

        return new Result($returnValue, ob_get_clean(), $exception);
    }
}


1			<?php
2
3			namespace hanneskod\readmetester;
4
5			/**
6			* Wrapps an executable block of code
7			*/
8			class CodeBlock
9			{
10			/**
11			* @var string The contained code
12			*/
13			private $code;
14
15			/**
16			* @param string $code
17			*/
18			public function __construct($code)
19			{
20			$this->code = $code;
21			}
22
23			/**
24			* Grab contained code
25			*
26			* @return string
27			*/
28			public function getCode()
29			{
30			return $this->code;
31			}
32
33			/**
34			* Execute code block
35			*
36			* @return Result The result of the executed code
37			*/
38			public function execute()
39			{
40			$returnValue = '';
			0 ignored issues – show Unused Code introduced 2015-12-29 12:46 UTC by Report Bug Copy Issue Report `$returnValue` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
41			$exception = null;
42
43			ob_start();
44
45			try {
46			$returnValue = eval($this->code);
			0 ignored issues – show Coding Style introduced 2015-12-29 12:46 UTC by Report Bug Copy Issue Report It is generally not recommended to use `eval` unless absolutely required. On one hand, `eval` might be exploited by malicious users if they somehow manage to inject dynamic content. On the other hand, with the emergence of faster PHP runtimes like the HHVM, `eval` prevents some optimization that they perform. Loading history...
47			} catch (\Exception $e) {
			0 ignored issues – show Unused Code introduced 2015-12-29 12:46 UTC by Report Bug Copy Issue Report `catch (\Exception $e) { $exception = $e; }` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
48			$exception = $e;
49			}
50
51			return new Result($returnValue, ob_get_clean(), $exception);
52			}
53			}
54

hanneskod / readme-tester

Push — master ( abec3a...7e1653 )

CodeBlock::getCode() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like