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-04 08:12 UTC

JsonLoader::__construct() B

↳ Parent: JsonLoader

Complexity

Conditions	5
Paths	3

Size

Total Lines	10
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	10
rs	8.8571
cc	5
eloc	6
nc	3
nop	1

<?php

namespace ComposerRequireChecker;

use ComposerRequireChecker\Exception\InvalidJsonException;
use ComposerRequireChecker\Exception\NotReadableException;

class JsonLoader {

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

    /**
     * @param string $path
     * @throws InvalidJsonException
     * @throws NotReadableException
     */
    public function __construct($path)
    {
        if (!is_readable($path) || ($content = file_get_contents($path)) === false) {
            throw new NotReadableException('unable to read ' . $path);
        }
        $this->data = json_decode($content, true);
        if ($this->data === null && JSON_ERROR_NONE !== json_last_error()) {
            throw new InvalidJsonException('error parsing ' . $path . ': ' . json_last_error_msg());
        }
    }

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

}


1			<?php
2
3			namespace ComposerRequireChecker;
4
5			use ComposerRequireChecker\Exception\InvalidJsonException;
6			use ComposerRequireChecker\Exception\NotReadableException;
7
8			class JsonLoader {
9
10			/**
11			* @var mixed
12			*/
13			private $data;
14
15			/**
16			* @param string $path
17			* @throws InvalidJsonException
18			* @throws NotReadableException
19			*/
20			public function __construct($path)
21			{
22			if (!is_readable($path) \|\| ($content = file_get_contents($path)) === false) {
23			throw new NotReadableException('unable to read ' . $path);
24			}
25			$this->data = json_decode($content, true);
26			if ($this->data === null && JSON_ERROR_NONE !== json_last_error()) {
27			throw new InvalidJsonException('error parsing ' . $path . ': ' . json_last_error_msg());
28			}
29			}
30
31			/**
32			* @return mixed
33			*/
34			public function getData()
35			{
36			return $this->data;
37			}
38
39			}
40

maglnet / ComposerRequireChecker

Pull Request — master (#31)

JsonLoader::__construct() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like