SearchableListFactory::classNameFromFileContents() - Code Metrics - Inspection of "Merge pull request #136 from ametad/fix-135" - matchish/laravel-scout-elasticsearch - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 74ebd5...131fc3 )

by Serhii

created 2020-12-11 15:33 UTC

SearchableListFactory::classNameFromFileContents() C

↳ Parent: SearchableListFactory

Complexity

Conditions	16
Paths	7

Size

Total Lines	48
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	22
CRAP Score	16.9313

Importance

Changes	3
Bugs	0	Features	0

Metric	Value
cc	16
eloc	25
c	3
b	0
f	0
nc	7
nop	1
dl	0
loc	48
ccs	22
cts	26
cp	0.8462
crap	16.9313
rs	5.5666

How to fix Complexity

<?php

declare(strict_types=1);

namespace Matchish\ScoutElasticSearch\Searchable;

use Illuminate\Support\Collection;
use Illuminate\Support\Str;
use function in_array;
use Laravel\Scout\Searchable;
use Symfony\Component\Finder\Finder;

final class SearchableListFactory
{
    /**
     * @var array
     */
    private static $declaredClasses;
    /**
     * @var string
     */
    private $namespace;
    /**
     * @var string
     */
    private $appPath;

    /**
     * @param string $namespace
     */
    public function __construct(string $namespace, string $appPath)
    {
        $this->namespace = $namespace;
        $this->appPath = $appPath;
    }

    /**
     * Get a list of searchable models.
     *
     * @return string[]
     */
    private function find(): array
    {
        $appNamespace = $this->namespace;

        return array_values(array_filter($this->getProjectClasses(), function (string $class) use ($appNamespace) {
            return Str::startsWith($class, $appNamespace) && $this->isSearchableModel($class);
        }));
    }

    /**
     * @param  string $class
     *
     * @return bool
     */
    private function isSearchableModel($class): bool
    {
        return in_array(Searchable::class, class_uses_recursive($class), true);
    }

    /**
     * @return array
     */
    private function getProjectClasses(): array
    {
        if (self::$declaredClasses === null) {

            self::$declaredClasses = [];

            $configFiles = Finder::create()->files()->name('*.php')->in($this->appPath);

            foreach ($configFiles->files() as $file) {
                if ($className = $this->classNameFromFileContents($file->getPathname())) {
                    self::$declaredClasses[] = $className;
                }
            }
        }

        return self::$declaredClasses;
    }

    /**
     * @param string $path
     * @return string|null
     * @link https://stackoverflow.com/a/7153391/1359273
     */
    private function classNameFromFileContents($path)
    {
        $fp = fopen($path, 'r');

        if (false === $fp) {
            return null;
        }

        $class = $namespace = $buffer = '';

        while (! $class) {
            if (feof($fp)) {
                break;
            }

            $buffer .= fread($fp, 512);
            $tokens = token_get_all($buffer);

            if (strpos($buffer, '{') === false) {
                continue;
            }

            for ($i = 0, $iMax = count($tokens); $i < $iMax; $i++) {
                if ($tokens[$i][0] === T_NAMESPACE) {
                    for ($j = $i + 1, $jMax = count($tokens); $j < $jMax; $j++) {
                        if ($tokens[$j][0] === T_STRING) {
                            $namespace .= $tokens[$j][1];
                        } elseif ($tokens[$j] === '{' || $tokens[$j] === ';') {
                            break;
                        }
                    }
                }

                if ($tokens[$i][0] === T_CLASS) {
                    for ($j = $i + 1, $jMax = count($tokens); $j < $jMax; $j++) {
                        if ($tokens[$j] === '{') {
                            $class = $tokens[$i + 2][1];
                        }
                    }
                }
            }
        }

        if (! $class) {
            return null;
        }

        return $namespace ? "{$namespace}\\{$class}" : $class;
    }

    /**
     * @return Collection
     */
    public function make(): Collection
    {
        return new Collection($this->find());
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Matchish\ScoutElasticSearch\Searchable;
6
7		use Illuminate\Support\Collection;
8		use Illuminate\Support\Str;
9		use function in_array;
10		use Laravel\Scout\Searchable;
11		use Symfony\Component\Finder\Finder;
12
13		final class SearchableListFactory
14		{
15		/**
16		* @var array
17		*/
18		private static $declaredClasses;
19		/**
20		* @var string
21		*/
22		private $namespace;
23		/**
24		* @var string
25		*/
26		private $appPath;
27
28		/**
29		* @param string $namespace
30		*/
31	12	public function __construct(string $namespace, string $appPath)
32		{
33	12	$this->namespace = $namespace;
34	12	$this->appPath = $appPath;
35	12	}
36
37		/**
38		* Get a list of searchable models.
39		*
40		* @return string[]
41		*/
42	12	private function find(): array
43		{
44	12	$appNamespace = $this->namespace;
45
46	12	return array_values(array_filter($this->getProjectClasses(), function (string $class) use ($appNamespace) {
47	12	return Str::startsWith($class, $appNamespace) && $this->isSearchableModel($class);
48	12	}));
49		}
50
51		/**
52		* @param string $class
53		*
54		* @return bool
55		*/
56	12	private function isSearchableModel($class): bool
57		{
58	12	return in_array(Searchable::class, class_uses_recursive($class), true);
59		}
60
61		/**
62		* @return array
63		*/
64	12	private function getProjectClasses(): array
65		{
66	12	if (self::$declaredClasses === null) {
		0 ignored issues – show introduced 2020-03-09 00:55 UTC by Report Bug Copy Issue Report The condition `self::declaredClasses === null` is always `false`. Loading history...
67	1	self::$declaredClasses = [];
68
69	1	$configFiles = Finder::create()->files()->name('*.php')->in($this->appPath);
70
71	1	foreach ($configFiles->files() as $file) {
72	1	if ($className = $this->classNameFromFileContents($file->getPathname())) {
73	1	self::$declaredClasses[] = $className;
74		}
75		}
76		}
77
78	12	return self::$declaredClasses;
79		}
80
81		/**
82		* @param string $path
83		* @return string\|null
84		* @link https://stackoverflow.com/a/7153391/1359273
85		*/
86	1	private function classNameFromFileContents($path)
87		{
88	1	$fp = fopen($path, 'r');
89
90	1	if (false === $fp) {
91		return null;
92		}
93
94	1	$class = $namespace = $buffer = '';
95
96	1	while (! $class) {
97	1	if (feof($fp)) {
98		break;
99		}
100
101	1	$buffer .= fread($fp, 512);
102	1	$tokens = token_get_all($buffer);
103
104	1	if (strpos($buffer, '{') === false) {
105		continue;
106		}
107
108	1	for ($i = 0, $iMax = count($tokens); $i < $iMax; $i++) {
109	1	if ($tokens[$i][0] === T_NAMESPACE) {
110	1	for ($j = $i + 1, $jMax = count($tokens); $j < $jMax; $j++) {
111	1	if ($tokens[$j][0] === T_STRING) {
112	1	$namespace .= $tokens[$j][1];
113	1	} elseif ($tokens[$j] === '{' \|\| $tokens[$j] === ';') {
114	1	break;
115		}
116		}
117		}
118
119	1	if ($tokens[$i][0] === T_CLASS) {
120	1	for ($j = $i + 1, $jMax = count($tokens); $j < $jMax; $j++) {
121	1	if ($tokens[$j] === '{') {
122	1	$class = $tokens[$i + 2][1];
123		}
124		}
125		}
126		}
127		}
128
129	1	if (! $class) {
130		return null;
131		}
132
133	1	return $namespace ? "{$namespace}\\{$class}" : $class;
134		}
135
136		/**
137		* @return Collection
138		*/
139	12	public function make(): Collection
140		{
141	12	return new Collection($this->find());
142		}
143		}
144

matchish / laravel-scout-elasticsearch

Push — master ( 74ebd5...131fc3 )

SearchableListFactory::classNameFromFileContents() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like