Route::handle() - Code Metrics - Inspection of "Update package of travis and scrutinizer" - BonfimLabs/PHP-HTTP-Router - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4f941d...095aef )

by Edson

created 2019-08-25 20:20 UTC

Route::handle() A

↳ Parent: Route

Complexity

Conditions	6
Paths	7

Size

Total Lines	36
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	42

Importance

Changes

Metric	Value
cc	6
eloc	19
nc	7
nop	3
dl	0
loc	36
ccs	0
cts	21
cp	0
crap	42
rs	9.0111
c	0
b	0
f	0

<?php

namespace Bonfim\Router;

use ReflectionException;
use ReflectionFunction;

/**
 * Class Route
 * @package Router
 */
/**
 * Class Route
 * @package Router
 */
/**
 * Class Route
 * @package Router
 */
class Route
{
    /**
     * @var Router
     */
    private $router;

    /**
     * @var bool
     */
    private $status = false;


    /**
     * @var
     */
    private static $route;

    /**
     * @var null
     */
    private static $match = null;


    /**
     * Route constructor.
     */
    private function __construct()
    {
        $this->router = new Router();
    }

    /**
     * @return Route
     */
    private static function route(): Route
    {
        if (!isset(self::$route) || is_null(self::$route)) {
            self::$route = new Route;
        }

        return self::$route;
    }

    /**
     * @return null|Client
     */
    private static function dispatch(): ?Client
    {
        if (!isset(self::$match) || is_null(self::$match)) {
            self::$match = self::route()->router->handle();
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
        }

        return self::$match;
    }

    /**
     * @param string $method
     * @param string $uri
     * @param $callback
     * @return void
     * @throws ReflectionException
     */
    private function handle(string $method, string $uri, \Closure $callback): void
    {
        $this->router->add([
            'uri' => $uri,
            'name' => '',
            'method' => $method,
            'callback' => $callback
        ]);

        $match = $this->dispatch();

        $args = [];

        if ($match) {
            $reflect = new ReflectionFunction($callback);

            foreach ($reflect->getParameters() as $i => $param) {
                // Caso o parametro seja tipado
                if ($param->hasType()) {
                    // Adiciona o objecto Request nos parametros da classe
                    if ($param->getType()->getName() == Request::class) {
                        $args[$i] = new Request();
                    }

                    // Adiciona o objeto Response nos parametros da classe
                    if ($param->getType()->getName() == Response::class) {
                        $args[$i] = new Response();
                    }
                } else {
                    //Adiciona o restante dos parametros nao tipados
                    $args[$i] = $match->getArg($param->getName());
                }
            }

            call_user_func_array($match->getCallback(), $args);
            exit;

        }
    }

    /**
     * @param string $uri
     * @param $callback
     */
    public static function get(string $uri, $callback): void
    {
        self::route()->handle('get', $uri, $callback);
    }

    /**
     * @param string $uri
     * @param $callback
     */
    public static function post(string $uri, $callback): void
    {
        self::route()->handle('post', $uri, $callback);
    }

    /**
     * @param string $uri
     * @param $callback
     */
    public static function put(string $uri, $callback): void
    {
        self::route()->handle('put', $uri, $callback);
    }

    /**
     * @param string $uri
     * @param $callback
     */
    public static function patch(string $uri, $callback): void
    {
        self::route()->handle('patch', $uri, $callback);
    }

    /**
     * @param string $uri
     * @param $callback
     */
    public static function delete(string $uri, $callback): void
    {
        self::route()->handle('delete', $uri, $callback);
    }

    /**
     * @param string $uri
     * @param $callback
     */
    public static function options(string $uri, $callback): void
    {
        self::route()->handle('options', $uri, $callback);
    }

    /**
     * @param string $uri
     * @param $callback
     */
    public static function any(string $uri, $callback): void
    {
        $methods = ['get', 'post', 'put', 'patch', 'delete', 'options'];
        self::match($methods, $uri, $callback);
    }

    /**
     * @param array $methods
     * @param string $uri
     * @param $callback
     */
    public static function match(array $methods, string $uri, $callback): void
    {
        foreach ($methods as $method) {
            self::route()->handle($method, $uri, $callback);
        }
    }
}


1			<?php
2
3			namespace Bonfim\Router;
4
5			use ReflectionException;
6			use ReflectionFunction;
7
8			/**
9			* Class Route
10			* @package Router
11			*/
12			/**
13			* Class Route
14			* @package Router
15			*/
16			/**
17			* Class Route
18			* @package Router
19			*/
20			class Route
21			{
22			/**
23			* @var Router
24			*/
25			private $router;
26
27			/**
28			* @var bool
29			*/
30			private $status = false;
			0 ignored issues – show introduced 2019-08-25 20:22 UTC by Report Bug Copy Issue Report The private property `$status` is not used, and could be removed. Loading history...
31
32			/**
33			* @var
34			*/
35			private static $route;
36
37			/**
38			* @var null
39			*/
40			private static $match = null;
41
42
43			/**
44			* Route constructor.
45			*/
46			private function __construct()
47			{
48			$this->router = new Router();
49			}
50
51			/**
52			* @return Route
53			*/
54			private static function route(): Route
55			{
56			if (!isset(self::$route) \|\| is_null(self::$route)) {
57			self::$route = new Route;
58			}
59
60			return self::$route;
61			}
62
63			/**
64			* @return null\|Client
65			*/
66			private static function dispatch(): ?Client
67			{
68			if (!isset(self::$match) \|\| is_null(self::$match)) {
69			self::$match = self::route()->router->handle();
			0 ignored issues – show Documentation Bug introduced 2019-08-25 20:22 UTC by Report Bug Copy Issue Report It seems like `self::route()->router->handle()` can also be of type `Bonfim\Router\Client`. However, the property `$match` is declared as type `null`. Maybe add an additional type check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable `$accountId` that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the `id` property of an instance of the `Account` class. This class holds a proper account, so the id value must no longer be false. Either this assignment is in error or a type check should be added for that assignment. class Id { public $id; public function __construct($id) { $this->id = $id; } } class Account { /** @var Id $id */ public $id; } $account_id = false; if (starsAreRight()) { $account_id = new Id(42); } $account = new Account(); if ($account instanceof Id) { $account->id = $account_id; } Loading history...
70			}
71
72			return self::$match;
73			}
74
75			/**
76			* @param string $method
77			* @param string $uri
78			* @param $callback
79			* @return void
80			* @throws ReflectionException
81			*/
82			private function handle(string $method, string $uri, \Closure $callback): void
83			{
84			$this->router->add([
85			'uri' => $uri,
86			'name' => '',
87			'method' => $method,
88			'callback' => $callback
89			]);
90
91			$match = $this->dispatch();
92
93			$args = [];
94
95			if ($match) {
96			$reflect = new ReflectionFunction($callback);
97
98			foreach ($reflect->getParameters() as $i => $param) {
99			// Caso o parametro seja tipado
100			if ($param->hasType()) {
101			// Adiciona o objecto Request nos parametros da classe
102			if ($param->getType()->getName() == Request::class) {
103			$args[$i] = new Request();
104			}
105
106			// Adiciona o objeto Response nos parametros da classe
107			if ($param->getType()->getName() == Response::class) {
108			$args[$i] = new Response();
109			}
110			} else {
111			//Adiciona o restante dos parametros nao tipados
112			$args[$i] = $match->getArg($param->getName());
113			}
114			}
115
116			call_user_func_array($match->getCallback(), $args);
117			exit;
			0 ignored issues – show Best Practice introduced 2018-04-17 03:16 UTC by Report Bug Copy Issue Report Using `exit` here is not recommended. In general, usage of exit should be done with care and only when running in a scripting context like a CLI script. Loading history...
118			}
119			}
120
121			/**
122			* @param string $uri
123			* @param $callback
124			*/
125			public static function get(string $uri, $callback): void
126			{
127			self::route()->handle('get', $uri, $callback);
128			}
129
130			/**
131			* @param string $uri
132			* @param $callback
133			*/
134			public static function post(string $uri, $callback): void
135			{
136			self::route()->handle('post', $uri, $callback);
137			}
138
139			/**
140			* @param string $uri
141			* @param $callback
142			*/
143			public static function put(string $uri, $callback): void
144			{
145			self::route()->handle('put', $uri, $callback);
146			}
147
148			/**
149			* @param string $uri
150			* @param $callback
151			*/
152			public static function patch(string $uri, $callback): void
153			{
154			self::route()->handle('patch', $uri, $callback);
155			}
156
157			/**
158			* @param string $uri
159			* @param $callback
160			*/
161			public static function delete(string $uri, $callback): void
162			{
163			self::route()->handle('delete', $uri, $callback);
164			}
165
166			/**
167			* @param string $uri
168			* @param $callback
169			*/
170			public static function options(string $uri, $callback): void
171			{
172			self::route()->handle('options', $uri, $callback);
173			}
174
175			/**
176			* @param string $uri
177			* @param $callback
178			*/
179			public static function any(string $uri, $callback): void
180			{
181			$methods = ['get', 'post', 'put', 'patch', 'delete', 'options'];
182			self::match($methods, $uri, $callback);
183			}
184
185			/**
186			* @param array $methods
187			* @param string $uri
188			* @param $callback
189			*/
190			public static function match(array $methods, string $uri, $callback): void
191			{
192			foreach ($methods as $method) {
193			self::route()->handle($method, $uri, $callback);
194			}
195			}
196			}
197

BonfimLabs / PHP-HTTP-Router

Push — master ( 4f941d...095aef )

Route::handle() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like