JsonLoader::__construct() - Code Metrics - Inspection of "Fix #22: improve the way to check json_decode resu..." - maglnet/ComposerRequireChecker - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#31)

unknown

created 2017-10-03 14:42 UTC

JsonLoader::__construct() B

↳ Parent: JsonLoader

Complexity

Conditions	5
Paths	3

Size

Total Lines	15
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	5

Importance

Changes	1
Bugs	0	Features	0

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

<?php

namespace ComposerRequireChecker;

class JsonLoader {

    const NO_ERROR = 0;
    const ERROR_NO_READABLE = 1;
    const ERROR_INVALID_JSON = 2;

    /**
     * @var string
     */
    private $path;

    /**
     * @var int
     */
    private $errorCode;

    /**
     * @var string
     */
    private $errorMessage;

    /**
     * @var mixed
     */
    private $data;

    /**
     * @param string $path
     */
    public function __construct($path)
    {
        $this->path = $path;
        if (!is_readable($path) || ($content = file_get_contents($path)) === false) {
            $this->errorCode = self::ERROR_NO_READABLE;
            return;
        }
        $this->data = json_decode($content, true);
        if ($this->data === null && JSON_ERROR_NONE !== json_last_error()) {
            $this->errorCode = self::ERROR_INVALID_JSON;
            $this->errorMessage = json_last_error_msg();
            return;
        }
        $this->errorCode = self::NO_ERROR;
    }

    /**
     * @return string
     */
    public function getPath()
    {
        return $this->path;
    }

    /**
     * @return int
     */
    public function getErrorCode()
    {
        return $this->errorCode;
    }

    /**
     * @return string
     */
    public function getErrorMessage()
    {
        return $this->errorMessage;
    }

    /**
     * @return mixed
     */
    public function getData()
    {
        return $this->data;
    }

}


1		<?php
2
3		namespace ComposerRequireChecker;
4
5		class JsonLoader {
6
7		const NO_ERROR = 0;
8		const ERROR_NO_READABLE = 1;
9		const ERROR_INVALID_JSON = 2;
10
11		/**
12		* @var string
13		*/
14		private $path;
15
16		/**
17		* @var int
18		*/
19		private $errorCode;
20
21		/**
22		* @var string
23		*/
24		private $errorMessage;
25
26		/**
27		* @var mixed
28		*/
29		private $data;
30
31		/**
32		* @param string $path
33		*/
34	3	public function __construct($path)
35		{
36	3	$this->path = $path;
37	3	if (!is_readable($path) \|\| ($content = file_get_contents($path)) === false) {
38	1	$this->errorCode = self::ERROR_NO_READABLE;
39	1	return;
40		}
41	2	$this->data = json_decode($content, true);
42	2	if ($this->data === null && JSON_ERROR_NONE !== json_last_error()) {
43	1	$this->errorCode = self::ERROR_INVALID_JSON;
44	1	$this->errorMessage = json_last_error_msg();
45	1	return;
46		}
47	1	$this->errorCode = self::NO_ERROR;
48	1	}
49
50		/**
51		* @return string
52		*/
53	3	public function getPath()
54		{
55	3	return $this->path;
56		}
57
58		/**
59		* @return int
60		*/
61	3	public function getErrorCode()
62		{
63	3	return $this->errorCode;
64		}
65
66		/**
67		* @return string
68		*/
69	2	public function getErrorMessage()
70		{
71	2	return $this->errorMessage;
72		}
73
74		/**
75		* @return mixed
76		*/
77	3	public function getData()
78		{
79	3	return $this->data;
80		}
81
82		}
83

maglnet / ComposerRequireChecker

Pull Request — master (#31)

JsonLoader::__construct() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like