Core\Autoloader

Complexity

Total Complexity

14

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Test Coverage

Coverage

40.54%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	autoloader()	0	15	7
A	reset()	0	2	1
A	register()	0	18	4
A	_constructorStatic()	0	4	2

<?php
/**
 * Created by Gorlum 18.06.2017 13:24
 */

namespace Core;

/**
 * Class Autoloader
 *
 * One of core class to supply autoload facilities to the engine
 *
 * @package Core
 */
class Autoloader {
  const P_FOLDER = 'P_FOLDER';
  const P_PREFIX = 'P_PREFIX';

  /**
   * @var string[][] $folders - [[P_FOLDER => (str)$absoluteFolder, P_PREFIX => (str)$prefixToIgnore]]
   */
  protected static $folders = [];

  protected static $autoloaderRegistered = false;

  protected static function _constructorStatic() {
    if(!static::$autoloaderRegistered) {
      spl_autoload_register(array(__CLASS__, 'autoloader'));
      static::$autoloaderRegistered = true;
    }
  }

  /**
   * @param string $class - Fully-qualified path with namespaces
   */
  public static function autoloader($class) {

Please use __construct() instead of a PHP4-style constructor that is named after the class.

    static::_constructorStatic();

    foreach(static::$folders as $data) {
      $theClassFile = $class;

      if($data[static::P_PREFIX] && strrpos($class, $data[static::P_PREFIX]) !== false) {

It seems like $class can also be of type object; however, parameter $haystack of strrpos() does only seem to accept string, maybe add an additional type check?

        $theClassFile = substr($class, strlen($data[static::P_PREFIX]));

It seems like $class can also be of type object; however, parameter $string of substr() does only seem to accept string, maybe add an additional type check?

      }

      $classFullFileName = str_replace('\\', '/', $data[static::P_FOLDER] . $theClassFile) . DOT_PHP_EX;

Are you sure $theClassFile of type object|string can be used in concatenation?

      if(file_exists($classFullFileName) && is_file($classFullFileName)) {
        require_once($classFullFileName);
        if(method_exists($class, '_constructorStatic')) {
          $class::_constructorStatic();
        }
      }
    }
  }

  /**
   * @param string $absoluteClassRoot - absolute path to root class folder
   * @param string $classPrefix - PHP class prefix to ignore. Can be whole namespace or part of it
   */
  public static function register($absoluteClassRoot, $classPrefix = '') {
    static::_constructorStatic();

    $absoluteClassRoot = str_replace('\\', '/', SN_ROOT_PHYSICAL . $absoluteClassRoot);

    if(!($absoluteClassRoot = realpath($absoluteClassRoot))) {
      // TODO - throw new \Exception("There is some error when installing autoloader for '{$absoluteClassRoot}' class prefix '{$classPrefix}'");

43% of this comment could be valid code. Did you maybe forget this after debugging?

      return;
    }
    $absoluteClassRoot = str_replace('\\', '/', $absoluteClassRoot) . '/';

    if($classPrefix && strrpos($classPrefix, 1) != '\\') {
      $classPrefix .= '\\';
    }

    static::$folders[] = [
      static::P_FOLDER => $absoluteClassRoot,
      static::P_PREFIX => $classPrefix,
    ];
  }

//  /**
//   * @param string $relativeClassRoot - relative path to root class folder from game root (where index.php lies)
//   * @param string $classPrefix - PHP class prefix to ignore. Can be whole namespace or part of it
//   */
//  public static function registerRelative($relativeClassRoot, $classPrefix = '') {
//    static::register(SN_ROOT_PHYSICAL . $relativeClassRoot, $classPrefix);
//  }

  public static function reset() {
    static::$folders = [];
  }

}