FindByNamespace::buildData() - Code Metrics - Inspection of "alter findbynamespace to improve separation of con..." - twhiston/twlib - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3b5440...721fdb )

by Thomas

created 2016-06-23 17:59 UTC

FindByNamespace::buildData() D

↳ Parent: FindByNamespace

Complexity

Conditions	10
Paths	8

Size

Total Lines	27
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	27
rs	4.8196
cc	10
eloc	18
nc	8
nop	1

How to fix Complexity

<?php

/**
 * Created by PhpStorm.
 * User: tom
 * Date: 18/06/2016
 * Time: 16:23
 */

namespace twhiston\twLib\Discovery;

use RecursiveDirectoryIterator;
use RecursiveIteratorIterator;
use RegexIterator;

class FindByNamespace
{

    protected $path;

    protected $data;

    /**
     * FindByNamespace constructor.
     * @param $path
     */
    public function __construct($path = null)
    {
        $this->path = ($path === null) ? __DIR__ : $path;
        $this->data = [];
    }

    public function find($needle = null, $rebuild = false)
    {
        $this->buildData($rebuild);
        return $this->filterData($needle);

    }

    protected function buildData($rebuild)
    {
        if ($rebuild === true || empty($this->data)) {
            $allFiles = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($this->path));
            $phpFiles = new RegexIterator($allFiles, '/\.php$/');
            foreach ($phpFiles as $phpFile) {
                $content = file_get_contents($phpFile->getRealPath());
                $tokens = token_get_all($content);
                $namespace = '';
                for ($index = 0; isset($tokens[$index]); $index++) {
                    if (!isset($tokens[$index][0])) {
                        continue;
                    }
                    if (T_NAMESPACE === $tokens[$index][0]) {
                        $index += 2; // Skip namespace keyword and whitespace
                        while (isset($tokens[$index]) && is_array($tokens[$index])) {
                            $namespace .= $tokens[$index++][1];
                        }
                    }
                    if (T_CLASS === $tokens[$index][0]) {
                        $index += 2; // Skip class keyword and whitespace
                        $this->data[] = $namespace . '\\' . $tokens[$index][1];
                    }
                }
            }
        }
    }

    protected function filterData($needle)
    {
        return array_values(array_filter($this->data, function ($var) use ($needle) {
            if (strpos($var, $needle) !== false) {
                return true;
            }
            return false;
        }));
    }

}

1			<?php
2
3			/**
4			* Created by PhpStorm.
5			* User: tom
6			* Date: 18/06/2016
7			* Time: 16:23
8			*/
9
10			namespace twhiston\twLib\Discovery;
11
12			use RecursiveDirectoryIterator;
13			use RecursiveIteratorIterator;
14			use RegexIterator;
15
16			class FindByNamespace
17			{
18
19			protected $path;
20
21			protected $data;
22
23			/**
24			* FindByNamespace constructor.
25			* @param $path
26			*/
27			public function __construct($path = null)
28			{
29			$this->path = ($path === null) ? __DIR__ : $path;
30			$this->data = [];
31			}
32
33			public function find($needle = null, $rebuild = false)
34			{
35			$this->buildData($rebuild);
36			return $this->filterData($needle);
37
38			}
39
40			protected function buildData($rebuild)
41			{
42			if ($rebuild === true \|\| empty($this->data)) {
43			$allFiles = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($this->path));
44			$phpFiles = new RegexIterator($allFiles, '/\.php$/');
45			foreach ($phpFiles as $phpFile) {
46			$content = file_get_contents($phpFile->getRealPath());
47			$tokens = token_get_all($content);
48			$namespace = '';
49			for ($index = 0; isset($tokens[$index]); $index++) {
50			if (!isset($tokens[$index][0])) {
51			continue;
52			}
53			if (T_NAMESPACE === $tokens[$index][0]) {
54			$index += 2; // Skip namespace keyword and whitespace
55			while (isset($tokens[$index]) && is_array($tokens[$index])) {
56			$namespace .= $tokens[$index++][1];
57			}
58			}
59			if (T_CLASS === $tokens[$index][0]) {
60			$index += 2; // Skip class keyword and whitespace
61			$this->data[] = $namespace . '\\' . $tokens[$index][1];
62			}
63			}
64			}
65			}
66			}
67
68			protected function filterData($needle)
69			{
70			return array_values(array_filter($this->data, function ($var) use ($needle) {
71			if (strpos($var, $needle) !== false) {
72			return true;
73			}
74			return false;
75			}));
76			}
77
78			}

twhiston / twlib

Push — master ( 3b5440...721fdb )

FindByNamespace::buildData() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like