Monitor::create() - Code Metrics - Inspection of "Merge branch 'dev' into master" - ZsgsDesign/NOJ - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 49f1ef...b2146d )

by John

created 2019-08-01 18:26 UTC

Monitor::create() A

↳ Parent: Monitor

Complexity

Conditions	5
Paths	14

Size

Total Lines	15
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	15
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)) $monitor->check();
			0 ignored issues – show Comprehensibility Best Practice introduced 2019-08-01 18:31 UTC by Report Bug Copy Issue Report The variable `$crawler` seems to never exist and therefore `isset` should always be `false`. Loading history...
21			}
22
23			public static function create($conf) {
24			$name=$conf["name"];
25			$monitorProvider="Monitor";
26			try {
27			$BabelConfig=json_decode(file_get_contents(babel_path("Extension/$name/babel.json")), true);
28			$monitorProvider=$BabelConfig["provider"]["monitor"];
29			} 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...
30			} 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...
31			} 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...
32			}
33			$className = "App\\Babel\\Extension\\$name\\$monitorProvider";
34			if(class_exists($className)) {
35			return new $className();
36			} else {
37			return null;
38			}
39			}
40			}
41

ZsgsDesign / NOJ

Push — master ( 49f1ef...b2146d )

Monitor::create() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like