Request::__construct() - Code Metrics - Inspection of "tidy up" - mvc5/mvc5 - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 62f043...679ba5 )

by devosc

created 2018-06-07 18:28 UTC

Request::__construct() B

↳ Parent: Request

Complexity

Conditions	5
Paths	12

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	6.6

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	3
cts	5
cp	0.6
c	0
b	0
f	0
rs	8.8571
cc	5
eloc	7
nc	12
nop	1
crap	6.6

<?php
/**
 *
 */

namespace Mvc5\Http\Config;

use Mvc5\Arg;
use Mvc5\Http;

trait Request
{
    /**
     * @var array
     */
    protected $config = [];

    /**
     * @param array $config
     */
    function __construct($config = [])
    {
        !isset($config[Arg::HEADERS]) &&
            $config[Arg::HEADERS] = new Http\HttpHeaders;

        is_array($config[Arg::HEADERS]) &&
            $config[Arg::HEADERS] = new Http\HttpHeaders($config[Arg::HEADERS]);

        isset($config[Arg::URI]) && !($config[Arg::URI] instanceof Http\Uri) &&
            $config[Arg::URI] = new Http\HttpUri($config[Arg::URI]);

        $this->config = $config;
    }

    /**
     * @return mixed
     */
    function body()

    {
        return $this[Arg::BODY];
    }

    /**
     * @return Http\Headers
     */
    function headers() : Http\Headers

    {
        return $this[Arg::HEADERS];
    }

    /**
     * @return string|null
     */
    function method() : ?string

    {
        return $this[Arg::METHOD];
    }

    /**
     * @return Http\Uri|null
     */
    function uri() : ?Http\Uri

    {
        return $this[Arg::URI];
    }

    /**
     * @return string|null
     */
    function version() : ?string

    {
        return $this[Arg::VERSION];
    }
}


1		<?php
2		/**
3		*
4		*/
5
6		namespace Mvc5\Http\Config;
7
8		use Mvc5\Arg;
9		use Mvc5\Http;
10
11		trait Request
12		{
13		/**
14		* @var array
15		*/
16	1	protected $config = [];
17
18	1	/**
19		* @param array $config
20		*/
21		function __construct($config = [])
22		{
23		!isset($config[Arg::HEADERS]) &&
24	3	$config[Arg::HEADERS] = new Http\HttpHeaders;
25
26	3	is_array($config[Arg::HEADERS]) &&
27		$config[Arg::HEADERS] = new Http\HttpHeaders($config[Arg::HEADERS]);
28
29		isset($config[Arg::URI]) && !($config[Arg::URI] instanceof Http\Uri) &&
30		$config[Arg::URI] = new Http\HttpUri($config[Arg::URI]);
31
32	9	$this->config = $config;
33		}
34	9
35		/**
36		* @return mixed
37		*/
38		function body()
		1 ignored issue – show Best Practice introduced 2018-04-22 13:30 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
39		{
40	1	return $this[Arg::BODY];
41		}
42	1
43		/**
44		* @return Http\Headers
45		*/
46		function headers() : Http\Headers
		1 ignored issue – show Best Practice introduced 2018-06-07 18:31 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
47		{
48	4	return $this[Arg::HEADERS];
49		}
50	4
51		/**
52		* @return string\|null
53		*/
54		function method() : ?string
		1 ignored issue – show Best Practice introduced 2018-04-22 13:30 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
55		{
56		return $this[Arg::METHOD];
57		}
58
59		/**
60		* @return Http\Uri\|null
61		*/
62		function uri() : ?Http\Uri
		1 ignored issue – show Best Practice introduced 2018-06-07 18:31 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
63		{
64		return $this[Arg::URI];
65		}
66
67		/**
68		* @return string\|null
69		*/
70		function version() : ?string
		1 ignored issue – show Best Practice introduced 2018-04-22 13:30 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
71		{
72		return $this[Arg::VERSION];
73		}
74		}
75

mvc5 / mvc5

Push — master ( 62f043...679ba5 )

Request::__construct() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like