Router::__construct() - Code Metrics - Inspection of "phpspec code coverage extension added, phpunit cov..." - venta/framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (ae28f9)

by Alexey

created 2016-12-01 10:27 UTC

Router::__construct() A

↳ Parent: Router

Complexity

Conditions	1
Paths	1

Size

Total Lines	13
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	1.0982

Importance

Changes

Metric	Value
dl	0
loc	13
ccs	7
cts	13
cp	0.5385
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	11
nc	1
nop	5
crap	1.0982

<?php declare(strict_types = 1);

namespace Venta\Routing;

use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;
use Venta\Contracts\Routing\MiddlewarePipelineFactory as MiddlewarePipelineFactoryContract;
use Venta\Contracts\Routing\RequestRouteCollectionFactory;
use Venta\Contracts\Routing\RouteCollection as RouteCollectionContract;
use Venta\Contracts\Routing\RouteDispatcherFactory as RouteDispatcherFactoryContract;
use Venta\Contracts\Routing\RouteMatcher as RouteMatcherContract;
use Venta\Contracts\Routing\Router as RouterContract;

/**
 * Class Router
 *
 * @package Venta\Routing
 */
final class Router implements RouterContract
{
    /**
     * @var RouteDispatcherFactoryContract
     */
    private $dispatcherFactory;

    /**
     * @var RouteMatcherContract
     */
    private $matcher;

    /**
     * @var MiddlewarePipelineFactoryContract
     */
    private $pipelineFactory;

    /**
     * @var RequestRouteCollectionFactory
     */
    private $routeCollectionFactory;

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

    /**
     * Router constructor.
     *
     * @param RouteDispatcherFactoryContract $dispatcherFactory
     * @param RouteMatcherContract $matcher
     * @param MiddlewarePipelineFactoryContract $pipelineFactory
     * @param RouteCollectionContract $routes
     * @param RequestRouteCollectionFactory $routeCollectionFactory
     */
    public function __construct(
        RouteDispatcherFactoryContract $dispatcherFactory,
        RouteMatcherContract $matcher,
        MiddlewarePipelineFactoryContract $pipelineFactory,
        RouteCollectionContract $routes,
        RequestRouteCollectionFactory $routeCollectionFactory
    ) {
        $this->dispatcherFactory = $dispatcherFactory;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->matcher = $matcher;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->pipelineFactory = $pipelineFactory;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->routes = $routes;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->routeCollectionFactory = $routeCollectionFactory;
    }

    /**
     * @inheritDoc
     */
    public function next(ServerRequestInterface $request): ResponseInterface
    {
        $requestRouteCollection = $this->routeCollectionFactory->create($this->routes, $request);
        // Find matching route against provided request.
        $route = $this->matcher->match($request, $requestRouteCollection);

        // Create route middleware pipeline.
        $pipeline = $this->pipelineFactory->create($route->getMiddlewares());
        // Create the last delegate, which calls route handler.
        $routeDispatcher = $this->dispatcherFactory->create($route);

        return $pipeline->process($request, $routeDispatcher);
    }

}

1		<?php declare(strict_types = 1);
2
3		namespace Venta\Routing;
4
5		use Psr\Http\Message\ResponseInterface;
6		use Psr\Http\Message\ServerRequestInterface;
7		use Venta\Contracts\Routing\MiddlewarePipelineFactory as MiddlewarePipelineFactoryContract;
8		use Venta\Contracts\Routing\RequestRouteCollectionFactory;
9		use Venta\Contracts\Routing\RouteCollection as RouteCollectionContract;
10		use Venta\Contracts\Routing\RouteDispatcherFactory as RouteDispatcherFactoryContract;
11		use Venta\Contracts\Routing\RouteMatcher as RouteMatcherContract;
12		use Venta\Contracts\Routing\Router as RouterContract;
13
14		/**
15		* Class Router
16		*
17		* @package Venta\Routing
18		*/
19		final class Router implements RouterContract
20		{
21		/**
22		* @var RouteDispatcherFactoryContract
23		*/
24		private $dispatcherFactory;
25
26		/**
27		* @var RouteMatcherContract
28		*/
29		private $matcher;
30
31		/**
32		* @var MiddlewarePipelineFactoryContract
33		*/
34		private $pipelineFactory;
35
36		/**
37		* @var RequestRouteCollectionFactory
38		*/
39		private $routeCollectionFactory;
40
41		/**
42		* @var RouteCollectionContract
43		*/
44		private $routes;
45
46		/**
47		* Router constructor.
48		*
49		* @param RouteDispatcherFactoryContract $dispatcherFactory
50		* @param RouteMatcherContract $matcher
51		* @param MiddlewarePipelineFactoryContract $pipelineFactory
52		* @param RouteCollectionContract $routes
53		* @param RequestRouteCollectionFactory $routeCollectionFactory
54		*/
55	2	public function __construct(
56		RouteDispatcherFactoryContract $dispatcherFactory,
57		RouteMatcherContract $matcher,
58		MiddlewarePipelineFactoryContract $pipelineFactory,
59		RouteCollectionContract $routes,
60		RequestRouteCollectionFactory $routeCollectionFactory
61		) {
62	2	$this->dispatcherFactory = $dispatcherFactory;
		1 ignored issue – show Coding Style introduced 2016-12-01 09:47 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 6 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...
63	2	$this->matcher = $matcher;
		1 ignored issue – show Coding Style introduced 2016-12-01 09:47 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 16 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...
64	2	$this->pipelineFactory = $pipelineFactory;
		1 ignored issue – show Coding Style introduced 2016-12-01 09:47 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 8 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...
65	2	$this->routes = $routes;
		1 ignored issue – show Coding Style introduced 2016-12-01 09:47 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 17 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...
66	2	$this->routeCollectionFactory = $routeCollectionFactory;
67	2	}
68
69		/**
70		* @inheritDoc
71		*/
72	1	public function next(ServerRequestInterface $request): ResponseInterface
73		{
74	1	$requestRouteCollection = $this->routeCollectionFactory->create($this->routes, $request);
75		// Find matching route against provided request.
76	1	$route = $this->matcher->match($request, $requestRouteCollection);
77
78		// Create route middleware pipeline.
79	1	$pipeline = $this->pipelineFactory->create($route->getMiddlewares());
80		// Create the last delegate, which calls route handler.
81	1	$routeDispatcher = $this->dispatcherFactory->create($route);
82
83	1	return $pipeline->process($request, $routeDispatcher);
84		}
85
86		}

venta / framework

Branch — master (ae28f9)

Router::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like