CompilerBuffer::select() - Code Metrics - Inspection of "Add some basic error handling components" - vanilla/ebi - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#10)

by Todd

created 2017-10-19 19:27 UTC

CompilerBuffer::select() A

↳ Parent: CompilerBuffer

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	8
cts	8
cp	1
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	7
nc	2
nop	1
crap	2

<?php
/**
 * @author Todd Burry <[email protected]>
 * @copyright 2009-2017 Vanilla Forums Inc.
 * @license MIT
 */

namespace Ebi;


class CompilerBuffer {
    const STYLE_JOIN = 'join';
    const STYLE_ARRAY = 'array';

    /**
     * @var ComponentBuffer[]
     */
    private $buffers;

    /**
     * @var ComponentBuffer
     */
    private $current;

    private $currentName;

    private $basename;

    private $style = self::STYLE_JOIN;

    /**
     * @var array
     */
    private $defaults;

    /**
     * @var \SplObjectStorage
     */
    private $nodeProps;

    public function __construct($style = self::STYLE_JOIN, array $defaults = []) {
        $defaults += [
            'baseIndent' => 0
        ];

        $this->buffers = [];
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->nodeProps = new \SplObjectStorage();
        $this->style = $style;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->defaults = $defaults;
        $this->select('');
    }

    /**
     * Select a specific component buffer.
     * @param $component
     */
    public function select($component) {

        $previous = $this->currentName;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->currentName = $component;

        if (!array_key_exists($component, $this->buffers)) {
            $this->buffers[$component] = $buffer = new ComponentBuffer($this->defaults);
$myVar = 'Value';
$higher = false;

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

        $this->current =& $this->buffers[$component];

        return $previous;
    }

    public function echoLiteral($value) {
        $this->current->echoLiteral($value);
    }

    public function echoCode($php) {
        $this->current->echoCode($php);
    }

    public function appendCode($php) {
        $this->current->appendCode($php);
    }

    public function indent($add) {
        $this->current->indent($add);
    }

    public function depth($add = 1) {
        $this->current->depth($add);
    }

    public function depthName($name, $add = 0) {

        return $this->current->depthName($name, $add);
    }

    public function pushScope(array $vars) {
        $this->current->pushScope($vars);
    }

    public function popScope() {
        $this->current->popScope();
    }

    public function getScopeVariables() {
        return $this->current->getScopeVariables();
    }

    public function flush() {
        switch ($this->getStyle()) {
            case self::STYLE_ARRAY:
                return $this->flushArray();
            default:
                return $this->flushJoin();
        }
    }

    private function flushJoin() {
        return implode("\n\n", array_map(function ($buffer) {
            /* @var ComponentBuffer $buffer */
            return $buffer->flush();
        }, $this->buffers));
    }

    private function flushArray() {
        $result = [];

        foreach ($this->buffers as $name => $buffer) {
            $flushed = $buffer->flush();
            if (empty($flushed)) {
                continue;
            }

            if ($name === '') {

                $result[] = $flushed;
            } else {
                $result[] = var_export($name, true).' => '.$flushed;
            }
        }

        if (empty($result)) {
            return '[]';
        } else {
            return "[\n".implode(",\n\n", $result)."\n".$this->px().']';
        }
    }

    protected function px($add = 0) {
        return str_repeat(' ', ($this->defaults['baseIndent'] + $add) * 4);
    }

    /**
     * Get the style.
     *
     * @return string Returns the style.
     */
    public function getStyle() {
        return $this->style;
    }

    /**
     * Set the style.
     *
     * @param string $style One of the **STYLE_*** constants.
     * @return $this
     */
    public function setStyle($style) {
        $this->style = $style;
        return $this;
    }

    /**
     * Get the basename.
     *
     * @return string Returns the basename.
     */
    public function getBasename() {
        return $this->basename;
    }

    /**
     * Set the basename.
     *
     * @param string $basename
     * @return $this
     */
    public function setBasename($basename) {
        $this->basename = $basename;
        return $this;
    }

    public function getNodeProp(\DOMNode $node, $name, $default = null) {

        if (!$this->nodeProps->contains($node) || !array_key_exists($name, $this->nodeProps[$node])) {
            return $default;
        }
        return $this->nodeProps[$node][$name];
    }

    public function setNodeProp(\DOMNode $node = null, $name, $value) {
        if ($node === null) {
            return $this;
        }

        if (!$this->nodeProps->contains($node)) {
            $this->nodeProps->attach($node, [$name => $value]);
        }

        $this->nodeProps[$node] = [$name => $value] + $this->nodeProps[$node];
        return $this;
    }

    public function getIndent() {
        return $this->current->getIndent();
    }

    public function getDepth() {
        return $this->current->getDepth();
    }

    public function getScope() {
        return $this->current->getScope();
    }

    public function getAllScopes() {
        return $this->current->getAllScopes();
    }
}


1		<?php
2		/**
3		* @author Todd Burry <[email protected]>
4		* @copyright 2009-2017 Vanilla Forums Inc.
5		* @license MIT
6		*/
7
8		namespace Ebi;
9
10
11		class CompilerBuffer {
12		const STYLE_JOIN = 'join';
13		const STYLE_ARRAY = 'array';
14
15		/**
16		* @var ComponentBuffer[]
17		*/
18		private $buffers;
19
20		/**
21		* @var ComponentBuffer
22		*/
23		private $current;
24
25		private $currentName;
26
27		private $basename;
28
29		private $style = self::STYLE_JOIN;
30
31		/**
32		* @var array
33		*/
34		private $defaults;
35
36		/**
37		* @var \SplObjectStorage
38		*/
39		private $nodeProps;
40
41	38	public function __construct($style = self::STYLE_JOIN, array $defaults = []) {
42		$defaults += [
43		'baseIndent' => 0
44	38	];
45
46	38	$this->buffers = [];
		0 ignored issues – show Coding Style introduced 2017-10-18 17:47 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
47	38	$this->nodeProps = new \SplObjectStorage();
48	38	$this->style = $style;
		0 ignored issues – show Coding Style introduced 2017-10-18 17:47 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
49	38	$this->defaults = $defaults;
50	38	$this->select('');
51	38	}
52
53		/**
54		* Select a specific component buffer.
55		* @param $component
56		*/
57	38	public function select($component) {
		0 ignored issues – show Documentation introduced 2017-10-18 17:47 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
58	38	$previous = $this->currentName;
		0 ignored issues – show Coding Style introduced 2017-10-18 17:47 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 10 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
59	38	$this->currentName = $component;
60
61	38	if (!array_key_exists($component, $this->buffers)) {
62	38	$this->buffers[$component] = $buffer = new ComponentBuffer($this->defaults);
		0 ignored issues – show Unused Code introduced 2017-10-18 17:47 UTC by Report Bug Copy Issue Report `$buffer` 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...
63	38	}
64
65	38	$this->current =& $this->buffers[$component];
66
67	38	return $previous;
68		}
69
70	38	public function echoLiteral($value) {
71	38	$this->current->echoLiteral($value);
72	38	}
73
74	26	public function echoCode($php) {
75	26	$this->current->echoCode($php);
76	26	}
77
78	38	public function appendCode($php) {
79	38	$this->current->appendCode($php);
80	38	}
81
82	38	public function indent($add) {
83	38	$this->current->indent($add);
84	38	}
85
86	13	public function depth($add = 1) {
87	13	$this->current->depth($add);
88	13	}
89
90	13	public function depthName($name, $add = 0) {
		0 ignored issues – show Documentation introduced 2017-10-18 17:47 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
91	13	return $this->current->depthName($name, $add);
92		}
93
94	38	public function pushScope(array $vars) {
95	38	$this->current->pushScope($vars);
96	38	}
97
98	38	public function popScope() {
99	38	$this->current->popScope();
100	38	}
101
102	34	public function getScopeVariables() {
103	34	return $this->current->getScopeVariables();
104		}
105
106	38	public function flush() {
107	38	switch ($this->getStyle()) {
108	38	case self::STYLE_ARRAY:
109	9	return $this->flushArray();
110	38	default:
111	38	return $this->flushJoin();
112	38	}
113		}
114
115		private function flushJoin() {
116	38	return implode("\n\n", array_map(function ($buffer) {
117		/* @var ComponentBuffer $buffer */
118	38	return $buffer->flush();
119	38	}, $this->buffers));
120		}
121
122	9	private function flushArray() {
123	9	$result = [];
124
125	9	foreach ($this->buffers as $name => $buffer) {
126	9	$flushed = $buffer->flush();
127	9	if (empty($flushed)) {
128	7	continue;
129		}
130
131	2	if ($name === '') {
		0 ignored issues – show Unused Code Bug introduced 2017-10-18 17:47 UTC by Report Bug Copy Issue Report The strict comparison `===` seems to always evaluate to `false` as the types of `$name` (`integer`) and `''` (`string`) can never be identical. Maybe you want to use a loose comparison `==` instead? Loading history...
132	2	$result[] = $flushed;
133	2	} else {
134	1	$result[] = var_export($name, true).' => '.$flushed;
135		}
136	9	}
137
138	9	if (empty($result)) {
139	7	return '[]';
140		} else {
141	2	return "[\n".implode(",\n\n", $result)."\n".$this->px().']';
142		}
143		}
144
145	2	protected function px($add = 0) {
146	2	return str_repeat(' ', ($this->defaults['baseIndent'] + $add) * 4);
147		}
148
149		/**
150		* Get the style.
151		*
152		* @return string Returns the style.
153		*/
154	38	public function getStyle() {
155	38	return $this->style;
156		}
157
158		/**
159		* Set the style.
160		*
161		* @param string $style One of the STYLE_* constants.
162		* @return $this
163		*/
164		public function setStyle($style) {
165		$this->style = $style;
166		return $this;
167		}
168
169		/**
170		* Get the basename.
171		*
172		* @return string Returns the basename.
173		*/
174		public function getBasename() {
175		return $this->basename;
176		}
177
178		/**
179		* Set the basename.
180		*
181		* @param string $basename
182		* @return $this
183		*/
184	38	public function setBasename($basename) {
185	38	$this->basename = $basename;
186	38	return $this;
187		}
188
189	38	public function getNodeProp(\DOMNode $node, $name, $default = null) {
		0 ignored issues – show Documentation introduced 2017-10-18 17:47 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
190	38	if (!$this->nodeProps->contains($node) \|\| !array_key_exists($name, $this->nodeProps[$node])) {
191	38	return $default;
192		}
193	4	return $this->nodeProps[$node][$name];
194		}
195
196	19	public function setNodeProp(\DOMNode $node = null, $name, $value) {
197	19	if ($node === null) {
198	15	return $this;
199		}
200
201	4	if (!$this->nodeProps->contains($node)) {
202	4	$this->nodeProps->attach($node, [$name => $value]);
203	4	}
204
205	4	$this->nodeProps[$node] = [$name => $value] + $this->nodeProps[$node];
206	4	return $this;
207		}
208
209	9	public function getIndent() {
210	9	return $this->current->getIndent();
211		}
212
213	9	public function getDepth() {
214	9	return $this->current->getDepth();
215		}
216
217		public function getScope() {
218		return $this->current->getScope();
219		}
220
221	9	public function getAllScopes() {
222	9	return $this->current->getAllScopes();
223		}
224		}
225

vanilla / ebi

Pull Request — master (#10)

CompilerBuffer::select() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like