Router::match() - Code Metrics - Inspection of "README badges and documentation" - AndrewCarterUK/SimpleRoute - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( f9c388...b2e961 )

by Andrew

created 2016-01-25 23:51 UTC

Router::match() A

↳ Parent: Router

Complexity

Conditions	4
Paths	4

Size

Total Lines	13
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	4.0961

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	9
c	1
b	0
f	0
nc	4
nop	2
dl	0
loc	13
ccs	9
cts	11
cp	0.8182
crap	4.0961
rs	9.2

<?php

namespace SimpleRoute;

use FastRoute;
use FastRoute\Dispatcher;
use FastRoute\RouteCollector;
use SimpleRoute\Exception\MethodNotAllowedException;
use SimpleRoute\Exception\NotFoundException;

final class Router implements RouterInterface
{
    /**
     * @var Dispatcher
     */
    private $dispatcher;

    /**
     * @var RouteInterface
     */
    private $routes;

    /**
     * @param RouteInterface[] A collection of routes
     */
    public function __construct(array $routes)
    {
        $this->routes = array();


        $this->dispatcher = FastRoute\simpleDispatcher(function (RouteCollector $collector) use ($routes) {
            foreach ($routes as $route) {
                $key = spl_object_hash($route);

                $this->routes[$key] = $route;

                if (!$route instanceof RouteInterface) {
                    throw new \InvalidArgumentException('Routes array must contain only RouteInterface objects');
                }

                $collector->addRoute(
                    $route->getMethods(),
                    $route->getPattern(),
                    $key
                );
            }
        });
    }

    /**
     * {@inheritdoc}
     */
    public function match($method, $uri)
    {
        $result = $this->dispatcher->dispatch($method, $uri);

        switch ($result[0]) {
            case Dispatcher::FOUND:
                return new Result($this->routes[$result[1]], $result[2]); 
            case Dispatcher::METHOD_NOT_ALLOWED:
                throw new MethodNotAllowedException($method, $result[1]);
            case Dispatcher::NOT_FOUND:
                throw new NotFoundException($uri);
        }
    }
}


1		<?php
2
3		namespace SimpleRoute;
4
5		use FastRoute;
6		use FastRoute\Dispatcher;
7		use FastRoute\RouteCollector;
8		use SimpleRoute\Exception\MethodNotAllowedException;
9		use SimpleRoute\Exception\NotFoundException;
10
11		final class Router implements RouterInterface
12		{
13		/**
14		* @var Dispatcher
15		*/
16		private $dispatcher;
17
18		/**
19		* @var RouteInterface
20		*/
21		private $routes;
22
23		/**
24		* @param RouteInterface[] A collection of routes
25		*/
26	12	public function __construct(array $routes)
27		{
28	12	$this->routes = array();
		0 ignored issues – show Documentation Bug introduced 2016-01-25 23:40 UTC by Report Bug Copy Issue Report It seems like `array()` of type `array` is incompatible with the declared type `object<SimpleRoute\RouteInterface>` of property `$routes`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
29
30	12	$this->dispatcher = FastRoute\simpleDispatcher(function (RouteCollector $collector) use ($routes) {
31	12	foreach ($routes as $route) {
32	9	$key = spl_object_hash($route);
33
34	9	$this->routes[$key] = $route;
35
36	9	if (!$route instanceof RouteInterface) {
37		throw new \InvalidArgumentException('Routes array must contain only RouteInterface objects');
38		}
39
40	9	$collector->addRoute(
41	9	$route->getMethods(),
42	9	$route->getPattern(),
43		$key
44	9	);
45	12	}
46	12	});
47	12	}
48
49		/**
50		* {@inheritdoc}
51		*/
52	12	public function match($method, $uri)
53		{
54	12	$result = $this->dispatcher->dispatch($method, $uri);
55
56	12	switch ($result[0]) {
57	12	case Dispatcher::FOUND:
58	6	return new Result($this->routes[$result[1]], $result[2]);
59	6	case Dispatcher::METHOD_NOT_ALLOWED:
60	3	throw new MethodNotAllowedException($method, $result[1]);
61	3	case Dispatcher::NOT_FOUND:
62	3	throw new NotFoundException($uri);
63		}
64		}
65		}
66

AndrewCarterUK / SimpleRoute

Push — master ( f9c388...b2e961 )

Router::match() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like