AutoLoader::autoload() - Code Metrics - Inspection of "remove dead code" - suricate-php/framework - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 740422...05a763 )

by Mathieu

created 2016-11-11 17:21 UTC

AutoLoader::autoload() A

↳ Parent: AutoLoader

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2

Importance

Changes

Metric	Value
cc	2
eloc	6
nc	2
nop	1
dl	0
loc	12
ccs	7
cts	7
cp	1
crap	2
rs	9.4285
c	0
b	0
f	0

<?php
namespace Suricate;

class AutoLoader
{
    public static function autoload($class)
    {
        $class      = str_replace('\\', DIRECTORY_SEPARATOR, $class);

        $filename   = dirname(__DIR__) . DIRECTORY_SEPARATOR . $class . '.php';

        if (is_file($filename)) {
            include $filename;
        } else {
if (rand(1, 6) > 3) {
print "Check failed";
} else {
    //print "Check succeeded";
}
            //throw new \Exception("Object '$class' not found", 2);

        }
    }

    public static function loadClass($class)
    {
        if (!class_exists($class, false) && !interface_exists($class, false)) {
            self::autoload($class);
        }
    }

    public static function register()
    {
        ini_set('unserialize_callback_func', 'spl_autoload_call');
        spl_autoload_register(__NAMESPACE__ .'\Autoloader::loadClass');
    }

    public static function unRegister()
    {
        spl_autoload_unregister(__NAMESPACE__ .'\Autoloader::loadClass');
    }
}


1		<?php
2		namespace Suricate;
3
4		class AutoLoader
5		{
6	3	public static function autoload($class)
7		{
8	3	$class = str_replace('\\', DIRECTORY_SEPARATOR, $class);
9
10	3	$filename = dirname(__DIR__) . DIRECTORY_SEPARATOR . $class . '.php';
11
12	3	if (is_file($filename)) {
13	3	include $filename;
14	3	} else {
		0 ignored issues – show Unused Code introduced 2016-08-28 17:07 UTC by Report Bug Copy Issue Report This `else` statement is empty and can be removed. This check looks for the `else` branches of `if` statements that have no statements or where all statements have been commented out. This may be the result of changes for debugging or the code may simply be obsolete. These `else` branches can be removed. if (rand(1, 6) > 3) { print "Check failed"; } else { //print "Check succeeded"; } could be turned into if (rand(1, 6) > 3) { print "Check failed"; } This is much more concise to read. Loading history...
15		//throw new \Exception("Object '$class' not found", 2);
		0 ignored issues – show Unused Code Comprehensibility introduced 2016-08-28 17:07 UTC by Report Bug Copy Issue Report `67%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
16		}
17	3	}
18
19	3	public static function loadClass($class)
20		{
21	3	if (!class_exists($class, false) && !interface_exists($class, false)) {
22	3	self::autoload($class);
23	3	}
24	3	}
25
26		public static function register()
27		{
28		ini_set('unserialize_callback_func', 'spl_autoload_call');
29		spl_autoload_register(__NAMESPACE__ .'\Autoloader::loadClass');
30		}
31
32		public static function unRegister()
33		{
34		spl_autoload_unregister(__NAMESPACE__ .'\Autoloader::loadClass');
35		}
36		}
37

suricate-php / framework

Push — master ( 740422...05a763 )

AutoLoader::autoload() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like