Monitor::create() - Code Metrics - Inspection of "0.3.1 Citoniensis" - ZsgsDesign/NOJ - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#235)

by John

created 2019-08-05 03:37 UTC

Monitor::create() A

↳ Parent: Monitor

Complexity

Conditions	5
Paths	14

Size

Total Lines	16
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	5
eloc	13
c	1
b	0
f	1
nc	14
nop	1
dl	0
loc	16
rs	9.5222

<?php

namespace App\Babel\Monit;

use ErrorException;
use Exception;
use Throwable;

class Monitor
{

    /**
     * Initial
     *
     * @return Response

     */
    public function __construct($conf)
    {
        $monitor=self::create($conf);
        if (!is_null($monitor) && isset($crawler)) {

            $monitor->check();
        }
    }

    public static function create($conf)
    {
        $name=$conf["name"];
        $monitorProvider="Monitor";
        try {
            $BabelConfig=json_decode(file_get_contents(babel_path("Extension/$name/babel.json")), true);
            $monitorProvider=$BabelConfig["provider"]["monitor"];
        } catch (Throwable $e) {

        } catch (ErrorException $e) {

        } catch (Exception $e) {

        }
        $className = "App\\Babel\\Extension\\$name\\$monitorProvider";
        if (class_exists($className)) {
            return new $className();
        } else {
            return null;
        }
    }
}


1			<?php
2
3			namespace App\Babel\Monit;
4
5			use ErrorException;
6			use Exception;
7			use Throwable;
8
9			class Monitor
10			{
11
12			/**
13			* Initial
14			*
15			* @return Response
			0 ignored issues – show Bug introduced 2019-08-01 18:31 UTC by Report Bug Copy Issue Report The type `App\Babel\Monit\Response` was not found. Did you mean `Response`? If so, make sure to prefix the type with `\`. Loading history...
16			*/
17			public function __construct($conf)
18			{
19			$monitor=self::create($conf);
20			if (!is_null($monitor) && isset($crawler)) {
			0 ignored issues – show Comprehensibility Best Practice introduced 2019-08-04 17:48 UTC by Report Bug Copy Issue Report The variable `$crawler` seems to never exist and therefore `isset` should always be `false`. Loading history...
21			$monitor->check();
22			}
23			}
24
25			public static function create($conf)
26			{
27			$name=$conf["name"];
28			$monitorProvider="Monitor";
29			try {
30			$BabelConfig=json_decode(file_get_contents(babel_path("Extension/$name/babel.json")), true);
31			$monitorProvider=$BabelConfig["provider"]["monitor"];
32			} catch (Throwable $e) {
			0 ignored issues – show Coding Style Comprehensibility introduced 2019-08-01 18:31 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
33			} catch (ErrorException $e) {
			0 ignored issues – show Coding Style Comprehensibility introduced 2019-08-01 18:31 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
34			} catch (Exception $e) {
			0 ignored issues – show Coding Style Comprehensibility introduced 2019-08-01 18:31 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
35			}
36			$className = "App\\Babel\\Extension\\$name\\$monitorProvider";
37			if (class_exists($className)) {
38			return new $className();
39			} else {
40			return null;
41			}
42			}
43			}
44

ZsgsDesign / NOJ

Pull Request — master (#235)

Monitor::create() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like