drupol\Yaroc\Utilities\ClassFinder

Complexity

Total Complexity

5

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Dependencies

0

Test Coverage

Coverage

95.45%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getClassesInNamespace()	0	11	1
A	getDefinedNamespaces()	0	8	1
A	getNamespaceDirectory()	0	18	3

<?php

namespace drupol\Yaroc\Utilities;

/**
 * Class ClassFinder.
 *
 * @package drupol\Yaroc\Utilities
 */
class ClassFinder  {

Expected 1 space after class name; 2 found

  /**
   *
   */
  const appRoot = __DIR__ . "/../../";

This class constant is not uppercase (expected APPROOT).

  /**
   * @param $namespace
   *
   * @return array
   */
  public static function getClassesInNamespace($namespace) {
    $files = scandir(self::getNamespaceDirectory($namespace));

    $classes = array_map(function($file) use ($namespace){

Expected 1 space after FUNCTION keyword; 0 found

Expected 1 space after closing parenthesis; found 0

      return $namespace . '\\' . str_replace('.php', '', $file);
    }, $files);

    return array_filter($classes, function($possibleClass){

Expected 1 space after FUNCTION keyword; 0 found

Expected 1 space after closing parenthesis; found 0

      return class_exists($possibleClass);
    });
  }

  /**
   * @return array
   */
  private static function getDefinedNamespaces() {
    $composerJsonPath = self::appRoot . 'composer.json';
    $composerConfig = json_decode(file_get_contents($composerJsonPath));

    //Apparently PHP doesn't like hyphens, so we use variable variables instead.
    $psr4 = "psr-4";
    return (array) $composerConfig->autoload->$psr4;
  }

  /**
   * @param $namespace
   *
   * @return bool|string

Consider making the return type a bit more specific; maybe use string|false.

   */
  private static function getNamespaceDirectory($namespace) {
    $composerNamespaces = self::getDefinedNamespaces();

    $namespaceFragments = explode('\\', $namespace);
    $undefinedNamespaceFragments = [];

    while($namespaceFragments) {

The expression $namespaceFragments of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

Expected 1 space after WHILE keyword; 0 found

      $possibleNamespace = implode('\\', $namespaceFragments) . '\\';

      if(array_key_exists($possibleNamespace, $composerNamespaces)){

Expected 1 space after IF keyword; 0 found

        return realpath(self::appRoot . $composerNamespaces[$possibleNamespace] . implode('/', array_reverse($undefinedNamespaceFragments)));
      }

      $undefinedNamespaceFragments[] = array_pop($namespaceFragments);
    }

    return false;
  }

}