Expander::expendReferences() - Code Metrics - imanghafoori1/laravel-microscope - Measure and Improve Code Quality continuously with Scrutinizer

Expander::expendReferences() C
last analyzed 2021-06-25 21:33 UTC

↳ Parent: Expander

Complexity

Conditions	14
Paths	25

Size

Total Lines

Duplication

Lines	6
Ratio	10.34 %

Importance

Changes

Metric	Value
cc	14
nc	25
nop	3
dl	6
loc	58
rs	6.2666
c	0
b	0
f	0

How to fix Long Method Complexity

<?php

namespace Imanghafoori\LaravelMicroscope\Analyzers;

use Illuminate\Support\Str;

class Expander
{
    /**
     * @param  string  $type
     *
     * @return bool
     */
    public static function isKeywordType($type)
    {
        return \in_array(strtolower($type), [
            'self',
            'static',
            'parent',
        ], true);
    }

    public static function expendReferences($classes, $imports, $namespace)
    {
        // Here we implode the tokens to form the full namespaced class path
        $results = [];
        $c = 0;
        foreach ($classes as $importeds) {
            $results[$c]['class'] = '';

            // attach the current namespace if it does not begin with '\'
            if ($importeds[0][1][0] != '\\') {
                $results[$c]['class'] = $namespace ? $namespace.'\\' : '';
            }

            foreach ($importeds as $row) {
                if (self::isKeywordType($row[1])) {
                    unset($results[$c]);
                    $c--;
                    continue;
                }

                // if starts with "\" or is not imported by the "use"
                if ($importeds[0][1][0] != '\\' && Str::contains($importeds[0][1], '\\')) {
                    // for php 8.x
                    $tmp = explode('\\', $importeds[0][1]);
                } else {
                    $tmp = [$importeds[0][1]];
                }

                if ($importeds[0][1] == '\\' || ! isset(array_values($imports)[0][$tmp[0]][0])) {
                    $results[$c]['class'] .= $row[1];
                    $results[$c]['line'] = $row[2];
                    continue;
                }

                // reads the import from the top
                $results[$c]['class'] = array_values($imports)[0][$tmp[0]][0];

                // for half imported references
                if ($importeds[0][1][0] != '\\' && Str::contains($importeds[0][1], '\\')) {
                    // for php 8.x
                    $tmp = explode('\\', $importeds[0][1]);
                    for ($j = 1; $j < count($tmp); $j++) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
                        $results[$c]['class'] .= '\\'.$tmp[$j];
                    }
                } else {
                    // for php 7.x
                    for ($j = 1; $j < count($importeds); $j++) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
                        $results[$c]['class'] .= $importeds[$j][1];
                    }
                }

                $results[$c]['line'] = $row[2];
            }
            $c++;
        }

        return [$results, $namespace];
    }
}


1		<?php
2
3		namespace Imanghafoori\LaravelMicroscope\Analyzers;
4
5		use Illuminate\Support\Str;
6
7		class Expander
8		{
9		/**
10		* @param string $type
11		*
12		* @return bool
13		*/
14		public static function isKeywordType($type)
15		{
16		return \in_array(strtolower($type), [
17		'self',
18		'static',
19		'parent',
20		], true);
21		}
22
23		public static function expendReferences($classes, $imports, $namespace)
24		{
25		// Here we implode the tokens to form the full namespaced class path
26		$results = [];
27		$c = 0;
28		foreach ($classes as $importeds) {
29		$results[$c]['class'] = '';
30
31		// attach the current namespace if it does not begin with '\'
32		if ($importeds[0][1][0] != '\\') {
33		$results[$c]['class'] = $namespace ? $namespace.'\\' : '';
34		}
35
36		foreach ($importeds as $row) {
37		if (self::isKeywordType($row[1])) {
38		unset($results[$c]);
39		$c--;
40		continue;
41		}
42
43		// if starts with "\" or is not imported by the "use"
44		if ($importeds[0][1][0] != '\\' && Str::contains($importeds[0][1], '\\')) {
45		// for php 8.x
46		$tmp = explode('\\', $importeds[0][1]);
47		} else {
48		$tmp = [$importeds[0][1]];
49		}
50
51		if ($importeds[0][1] == '\\' \|\| ! isset(array_values($imports)[0][$tmp[0]][0])) {
52		$results[$c]['class'] .= $row[1];
53		$results[$c]['line'] = $row[2];
54		continue;
55		}
56
57		// reads the import from the top
58		$results[$c]['class'] = array_values($imports)[0][$tmp[0]][0];
59
60		// for half imported references
61		if ($importeds[0][1][0] != '\\' && Str::contains($importeds[0][1], '\\')) {
62		// for php 8.x
63		$tmp = explode('\\', $importeds[0][1]);
64	View Code Duplication	for ($j = 1; $j < count($tmp); $j++) {
		0 ignored issues – show Performance Best Practice introduced 2020-09-15 07:18 UTC by Report Bug Copy Issue Report It seems like you are calling the size function `count()` as part of the test condition. You might want to compute the size beforehand, and not on each iteration. If the size of the collection does not change during the iteration, it is generally a good practice to compute it beforehand, and not on each iteration: for ($i=0; $i<count($array); $i++) { // calls count() on each iteration } // Better for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once } Loading history... Duplication introduced 2021-06-05 20:05 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
65		$results[$c]['class'] .= '\\'.$tmp[$j];
66		}
67		} else {
68		// for php 7.x
69	View Code Duplication	for ($j = 1; $j < count($importeds); $j++) {
		0 ignored issues – show Performance Best Practice introduced 2021-06-05 20:05 UTC by Report Bug Copy Issue Report It seems like you are calling the size function `count()` as part of the test condition. You might want to compute the size beforehand, and not on each iteration. If the size of the collection does not change during the iteration, it is generally a good practice to compute it beforehand, and not on each iteration: for ($i=0; $i<count($array); $i++) { // calls count() on each iteration } // Better for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once } Loading history... Duplication introduced 2021-06-05 20:05 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
70		$results[$c]['class'] .= $importeds[$j][1];
71		}
72		}
73
74		$results[$c]['line'] = $row[2];
75		}
76		$c++;
77		}
78
79		return [$results, $namespace];
80		}
81		}
82

imanghafoori1 / laravel-microscope

Expander::expendReferences() C last analyzed 2021-06-25 21:33 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

Expander::expendReferences() C
last analyzed 2021-06-25 21:33 UTC