SliceElementMatcher::match() - Code Metrics - Inspection of "Index slice matcher implemented" - remorhaz/php-json-path - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 37a545...2ac5c4 )

by Edward

created 2019-07-28 21:57 UTC

SliceElementMatcher::match() A

↳ Parent: SliceElementMatcher

Complexity

Conditions	4
Paths	3

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	4
eloc	5
nc	3
nop	1
dl	0
loc	9
ccs	0
cts	6
cp	0
crap	20
rs	10
c	1
b	0
f	1

<?php
declare(strict_types=1);

namespace Remorhaz\JSON\Path\Runtime\Matcher;

use function is_int;
use function max;

final class SliceElementMatcher implements ChildMatcherInterface
{

    private $count;

    private $start;

    private $end;

    private $step;

    private $isReverse;

    public function __construct(int $count, ?int $start, ?int $end, ?int $step)
    {
        $this->count = $count;
        $this->step = $step ?? 1;
        $this->isReverse = $step < 0;

        $this->start = $start;
        $this->end = $end;
    }

    public function match($address): bool
    {
        if (0 == $this->step || !is_int($address)) {
            return false;
        }
        $start = $this->detectStart($this->count);
        $end = $this->detectEnd($this->count);

        return $this->isInRange($address, $start, $end) && $this->isOnStep($address, $start);
    }

    private function detectStart(int $count): int
    {
        $start = $this->start;
        if (!isset($start)) {
            $start = $this->isReverse ? -1 : 0;
        }
        if ($start < 0) {
            $start = max($start + $count, 0);
        }

        return $start;
    }

    private function detectEnd(int $count): int
    {
        $end = $this->end;
        if (!isset($end)) {
            $end = $this->isReverse ? -$count - 1 : $count;
        }
        if ($end > $count) {
            return $count;
        }

        return $end < 0
            ? max($end + $count, $this->isReverse ? -1 : 0)
            : $end;
    }

    private function isInRange(int $address, int $start, int $end): bool
    {
        return $this->isReverse
            ? $address <= $start && $address > $end
            : $address >= $start && $address < $end;
    }

    private function isOnStep(int $address, int $start): bool
    {
        return 0 == ($this->isReverse ? $start - $address : $address - $start) % $this->step;
    }
}


1			<?php
2			declare(strict_types=1);
3
4			namespace Remorhaz\JSON\Path\Runtime\Matcher;
5
6			use function is_int;
7			use function max;
8
9			final class SliceElementMatcher implements ChildMatcherInterface
10			{
11
12			private $count;
13
14			private $start;
15
16			private $end;
17
18			private $step;
19
20			private $isReverse;
21
22			public function __construct(int $count, ?int $start, ?int $end, ?int $step)
23			{
24			$this->count = $count;
25			$this->step = $step ?? 1;
26			$this->isReverse = $step < 0;
27
28			$this->start = $start;
29			$this->end = $end;
30			}
31
32			public function match($address): bool
33			{
34			if (0 == $this->step \|\| !is_int($address)) {
35			return false;
36			}
37			$start = $this->detectStart($this->count);
38			$end = $this->detectEnd($this->count);
39
40			return $this->isInRange($address, $start, $end) && $this->isOnStep($address, $start);
41			}
42
43			private function detectStart(int $count): int
44			{
45			$start = $this->start;
46			if (!isset($start)) {
47			$start = $this->isReverse ? -1 : 0;
48			}
49			if ($start < 0) {
50			$start = max($start + $count, 0);
51			}
52
53			return $start;
54			}
55
56			private function detectEnd(int $count): int
57			{
58			$end = $this->end;
59			if (!isset($end)) {
60			$end = $this->isReverse ? -$count - 1 : $count;
61			}
62			if ($end > $count) {
63			return $count;
64			}
65
66			return $end < 0
67			? max($end + $count, $this->isReverse ? -1 : 0)
68			: $end;
69			}
70
71			private function isInRange(int $address, int $start, int $end): bool
72			{
73			return $this->isReverse
74			? $address <= $start && $address > $end
75			: $address >= $start && $address < $end;
76			}
77
78			private function isOnStep(int $address, int $start): bool
79			{
80			return 0 == ($this->isReverse ? $start - $address : $address - $start) % $this->step;
81			}
82			}
83

remorhaz / php-json-path

Push — master ( 37a545...2ac5c4 )

SliceElementMatcher::match() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like