SubCodeHandler::scanSeparators() - Code Metrics - Inspection of "Added integration with Markdown" - XaBerr/JATE - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9ad54b...895d73 )

by Federico

created 2016-08-26 18:08 UTC

SubCodeHandler::scanSeparators() C

↳ Parent: SubCodeHandler

Complexity

Conditions	8
Paths	1

Size

Total Lines	27
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	8
eloc	16
c	1
b	0
f	1
nc	1
nop	2
dl	0
loc	27
rs	5.3846

<?php

namespace Jade\Compiler;

/**
 * Class Jade\Compiler\SubCodeHandler.
 */
class SubCodeHandler
{
    protected $codeHandler;
    protected $input;
    protected $name;

    public function __construct(CodeHandler $codeHandler, $input, $name)
    {
        $this->codeHandler = $codeHandler;
        $this->input = $input;
        $this->name = $name;
    }

    public function getMiddleString()
    {
        $input = $this->input;

        return function ($start, $end) use ($input) {
            $offset = $start[1] + strlen($start[0]);

            return substr($input, $offset, isset($end) ? $end[1] - $offset : strlen($input));
        };
    }

    public function handleRecursion(&$result)
    {
        $getMiddleString = $this->getMiddleString();
        $codeHandler = $this->codeHandler;

        return function ($arg, $name = '') use (&$result, $codeHandler, $getMiddleString) {
            list($start, $end) = $arg;
            $str = trim($getMiddleString($start, $end));

            if (!strlen($str)) {
                return '';
            }

            $innerCode = $codeHandler->innerCode($str, $name);

            if (count($innerCode) > 1) {
                $result = array_merge($result, array_slice($innerCode, 0, -1));

                return array_pop($innerCode);
            }

            return $innerCode[0];
        };
    }

    protected function handleNestedExpression(&$result)
    {
        $handleRecursion = $this->handleRecursion($result);
        $name = $this->name;

        return function (&$arguments, $start, $end) use ($name, $handleRecursion) {
            if ($end !== false && $start[1] !== $end[1]) {
                array_push(
                    $arguments,
                    $handleRecursion(
                        array($start, $end),
                        $name * 10 + count($arguments)
                    )
                );
            }
        };
    }

    protected function scanSeparators(&$separators, &$result)
    {
        $handleNested = $this->handleNestedExpression($result);

        return function (&$arguments, $open, $close) use (&$separators, $handleNested) {
            $count = 1;

            do {
                $start = current($separators);

                do {
                    $curr = next($separators);

                    if ($curr[0] === $open) {
                        $count++;
                    }
                    if ($curr[0] === $close) {
                        $count--;
                    }
                } while ($curr[0] !== null && $count > 0 && $curr[0] !== ',');

                $handleNested($arguments, $start, current($separators));
            } while ($curr !== false && $count > 0);

            return $count;
        };
    }

    public function handleCodeInbetween(&$separators, &$result)
    {
        $scanSeparators = $this->scanSeparators($separators, $result);
        $input = $this->input;

        return function () use (&$separators, $input, $scanSeparators) {
            $arguments = array();

            $start = current($separators);
            $endPair = array(
                '[' => ']',
                '{' => '}',
                '(' => ')',
                ',' => false,
            );
            $open = $start[0];
            $close = $endPair[$start[0]];

            $count = $scanSeparators($arguments, $open, $close);

            if ($close && $count > 0) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
                throw new \ErrorException($input . "\nMissing closing: " . $close, 14);
            }

            if (($sep = current($separators)) !== false) {
                $end = next($separators);
                if ($end[0] === null && $sep[1] < $end[1]) {
                    $key = count($arguments) - 1;
                    $arguments[$key] .= substr($input, $sep[1] + 1, $end[1] - $sep[1] - 1);
                }
            }

            return $arguments;
        };
    }

    public function getNext($separators)
    {
        return function ($index) use ($separators) {
            if (isset($separators[$index + 1])) {
                return $separators[$index + 1];
            }
        };
    }

    public function addToOutput(&$output, &$key, &$value)
    {
        return function () use (&$output, &$key, &$value) {
            foreach (array('key', 'value') as $var) {
                ${$var} = trim(${$var});
                if (empty(${$var})) {
                    continue;
                }
                if (preg_match('/^\d*[a-zA-Z_]/', ${$var})) {
                    ${$var} = var_export(${$var}, true);
                }
            }
            $output[] = empty($value)
                ? $key
                : $key . ' => ' . $value;
            $key = '';
            $value = null;
        };
    }

    public function consume()
    {
        return function (&$argument, $start) {
            $argument = substr($argument, strlen($start));
        };
    }
}


1			<?php
2
3			namespace Jade\Compiler;
4
5			/**
6			* Class Jade\Compiler\SubCodeHandler.
7			*/
8			class SubCodeHandler
9			{
10			protected $codeHandler;
11			protected $input;
12			protected $name;
13
14			public function __construct(CodeHandler $codeHandler, $input, $name)
15			{
16			$this->codeHandler = $codeHandler;
17			$this->input = $input;
18			$this->name = $name;
19			}
20
21			public function getMiddleString()
22			{
23			$input = $this->input;
24
25			return function ($start, $end) use ($input) {
26			$offset = $start[1] + strlen($start[0]);
27
28			return substr($input, $offset, isset($end) ? $end[1] - $offset : strlen($input));
29			};
30			}
31
32			public function handleRecursion(&$result)
33			{
34			$getMiddleString = $this->getMiddleString();
35			$codeHandler = $this->codeHandler;
36
37			return function ($arg, $name = '') use (&$result, $codeHandler, $getMiddleString) {
38			list($start, $end) = $arg;
39			$str = trim($getMiddleString($start, $end));
40
41			if (!strlen($str)) {
42			return '';
43			}
44
45			$innerCode = $codeHandler->innerCode($str, $name);
46
47			if (count($innerCode) > 1) {
48			$result = array_merge($result, array_slice($innerCode, 0, -1));
49
50			return array_pop($innerCode);
51			}
52
53			return $innerCode[0];
54			};
55			}
56
57			protected function handleNestedExpression(&$result)
58			{
59			$handleRecursion = $this->handleRecursion($result);
60			$name = $this->name;
61
62			return function (&$arguments, $start, $end) use ($name, $handleRecursion) {
63			if ($end !== false && $start[1] !== $end[1]) {
64			array_push(
65			$arguments,
66			$handleRecursion(
67			array($start, $end),
68			$name * 10 + count($arguments)
69			)
70			);
71			}
72			};
73			}
74
75			protected function scanSeparators(&$separators, &$result)
76			{
77			$handleNested = $this->handleNestedExpression($result);
78
79			return function (&$arguments, $open, $close) use (&$separators, $handleNested) {
80			$count = 1;
81
82			do {
83			$start = current($separators);
84
85			do {
86			$curr = next($separators);
87
88			if ($curr[0] === $open) {
89			$count++;
90			}
91			if ($curr[0] === $close) {
92			$count--;
93			}
94			} while ($curr[0] !== null && $count > 0 && $curr[0] !== ',');
95
96			$handleNested($arguments, $start, current($separators));
97			} while ($curr !== false && $count > 0);
98
99			return $count;
100			};
101			}
102
103			public function handleCodeInbetween(&$separators, &$result)
104			{
105			$scanSeparators = $this->scanSeparators($separators, $result);
106			$input = $this->input;
107
108			return function () use (&$separators, $input, $scanSeparators) {
109			$arguments = array();
110
111			$start = current($separators);
112			$endPair = array(
113			'[' => ']',
114			'{' => '}',
115			'(' => ')',
116			',' => false,
117			);
118			$open = $start[0];
119			$close = $endPair[$start[0]];
120
121			$count = $scanSeparators($arguments, $open, $close);
122
123			if ($close && $count > 0) {
			0 ignored issues – show Bug Best Practice introduced 2016-08-26 18:11 UTC by Report Bug Copy Issue Report The expression `$close` of type `string\|false` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== false` 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...
124			throw new \ErrorException($input . "\nMissing closing: " . $close, 14);
125			}
126
127			if (($sep = current($separators)) !== false) {
128			$end = next($separators);
129			if ($end[0] === null && $sep[1] < $end[1]) {
130			$key = count($arguments) - 1;
131			$arguments[$key] .= substr($input, $sep[1] + 1, $end[1] - $sep[1] - 1);
132			}
133			}
134
135			return $arguments;
136			};
137			}
138
139			public function getNext($separators)
140			{
141			return function ($index) use ($separators) {
142			if (isset($separators[$index + 1])) {
143			return $separators[$index + 1];
144			}
145			};
146			}
147
148			public function addToOutput(&$output, &$key, &$value)
149			{
150			return function () use (&$output, &$key, &$value) {
151			foreach (array('key', 'value') as $var) {
152			${$var} = trim(${$var});
153			if (empty(${$var})) {
154			continue;
155			}
156			if (preg_match('/^\d*[a-zA-Z_]/', ${$var})) {
157			${$var} = var_export(${$var}, true);
158			}
159			}
160			$output[] = empty($value)
161			? $key
162			: $key . ' => ' . $value;
163			$key = '';
164			$value = null;
165			};
166			}
167
168			public function consume()
169			{
170			return function (&$argument, $start) {
171			$argument = substr($argument, strlen($start));
172			};
173			}
174			}
175

XaBerr / JATE

Push — master ( 9ad54b...895d73 )

SubCodeHandler::scanSeparators() C

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like