Matcher::matchRule() - Code Metrics - Inspection of "up php version for static analyz code" - alexpts/php-routing - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d3b9b9...fcc9a6 )

by Alexpts

created 2018-03-13 17:27 UTC

Matcher::matchRule() A

↳ Parent: Matcher

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	2

Importance

Changes

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

<?php
declare(strict_types = 1);

namespace PTS\Routing;

class Matcher
{
    /** @var RouteService */
    protected $routeService;
    /** @var Route */
    protected $routeNotFound;

    public function __construct(RouteService $routeService)
    {
        $this->routeService = $routeService;
        $this->routeNotFound = new Route('', function () {
            return null;
        });
    }

    public function setNotFoundHandler(Route $route): self
    {
        $this->routeNotFound = $route;
        return $this;
    }

    public function match(CollectionRoute $routes, string $path): \Generator
    {
        foreach ($routes->getRoutes() as $route) {
            $activeRoute = $this->matchRule($route, $path);
            if ($activeRoute !== null) {
                yield $activeRoute;
            }
        }

        return yield $this->routeNotFound;
    }

    protected function matchRule(Route $route, string $pathUrl): ?Route
    {
        $activeRoute = null;

        $regexp = $this->routeService->makeRegExp($route);

        if (preg_match('~^'.$regexp.'$~Uiu', $pathUrl, $values)) {
            $filterValues = array_filter(array_keys($values), '\is_string');
            $matches = array_intersect_key($values, array_flip($filterValues));
            $activeRoute = $route->setMatches($matches);
        }

        return $activeRoute;
    }
}


1		<?php
2		declare(strict_types = 1);
3
4		namespace PTS\Routing;
5
6		class Matcher
7		{
8		/** @var RouteService */
9		protected $routeService;
10		/** @var Route */
11		protected $routeNotFound;
12
13	6	public function __construct(RouteService $routeService)
14		{
15	6	$this->routeService = $routeService;
16	6	$this->routeNotFound = new Route('', function () {
17	1	return null;
18	6	});
19	6	}
20
21	1	public function setNotFoundHandler(Route $route): self
22		{
23	1	$this->routeNotFound = $route;
24	1	return $this;
25		}
26
27	5	public function match(CollectionRoute $routes, string $path): \Generator
28		{
29	5	foreach ($routes->getRoutes() as $route) {
30	4	$activeRoute = $this->matchRule($route, $path);
31	4	if ($activeRoute !== null) {
32	4	yield $activeRoute;
33		}
34		}
35
36	2	return yield $this->routeNotFound;
37		}
38
39	4	protected function matchRule(Route $route, string $pathUrl): ?Route
40		{
41	4	$activeRoute = null;
42
43	4	$regexp = $this->routeService->makeRegExp($route);
44
45	4	if (preg_match('~^'.$regexp.'$~Uiu', $pathUrl, $values)) {
46	3	$filterValues = array_filter(array_keys($values), '\is_string');
47	3	$matches = array_intersect_key($values, array_flip($filterValues));
48	3	$activeRoute = $route->setMatches($matches);
49		}
50
51	4	return $activeRoute;
52		}
53		}
54

alexpts / php-routing

Push — master ( d3b9b9...fcc9a6 )

Matcher::matchRule() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like