@@ -27,7 +27,7 @@ |
||
| 27 | 27 | */ |
| 28 | 28 | public function addEdge(Node $from, Node $to) |
| 29 | 29 | { |
| 30 | - $key = $from->getUniqueId() . '->' . $to->getUniqueId(); |
|
| 30 | + $key = $from->getUniqueId().'->'.$to->getUniqueId(); |
|
| 31 | 31 | |
| 32 | 32 | if (isset($this->edgesMap[$key])) { |
| 33 | 33 | return $this; |
@@ -66,13 +66,13 @@ discard block |
||
| 66 | 66 | } |
| 67 | 67 | |
| 68 | 68 | // sums |
| 69 | - $sum = (object)[ |
|
| 69 | + $sum = (object) [ |
|
| 70 | 70 | 'loc' => 0, |
| 71 | 71 | 'cloc' => 0, |
| 72 | 72 | 'lloc' => 0, |
| 73 | 73 | 'nbMethods' => 0, |
| 74 | 74 | ]; |
| 75 | - $avg = (object)[ |
|
| 75 | + $avg = (object) [ |
|
| 76 | 76 | 'wmc' => [], |
| 77 | 77 | 'ccn' => [], |
| 78 | 78 | 'bugs' => [], |
@@ -164,7 +164,7 @@ discard block |
||
| 164 | 164 | $violations[$name]++; |
| 165 | 165 | } |
| 166 | 166 | } |
| 167 | - $sum->violations = (object)$violations; |
|
| 167 | + $sum->violations = (object) $violations; |
|
| 168 | 168 | |
| 169 | 169 | $this->avg = $avg; |
| 170 | 170 | $this->sum = $sum; |
@@ -37,7 +37,7 @@ discard block |
||
| 37 | 37 | $class->set('interface', true); |
| 38 | 38 | $class->set('abstract', true); |
| 39 | 39 | } else { |
| 40 | - $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node)); |
|
| 40 | + $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node)); |
|
| 41 | 41 | $class = new ClassMetric($name); |
| 42 | 42 | $class->set('interface', false); |
| 43 | 43 | $class->set('abstract', $node instanceof Stmt\Trait_ || $node->isAbstract()); |
@@ -50,7 +50,7 @@ discard block |
||
| 50 | 50 | $roleDetector = new RoleOfMethodDetector(); |
| 51 | 51 | foreach ($node->stmts as $stmt) { |
| 52 | 52 | if ($stmt instanceof Stmt\ClassMethod) { |
| 53 | - $function = new FunctionMetric((string)$stmt->name); |
|
| 53 | + $function = new FunctionMetric((string) $stmt->name); |
|
| 54 | 54 | |
| 55 | 55 | $role = $roleDetector->detects($stmt); |
| 56 | 56 | $function->set('role', $role); |
@@ -47,7 +47,7 @@ |
||
| 47 | 47 | public function leaveNode(Node $node) |
| 48 | 48 | { |
| 49 | 49 | if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Trait_) { |
| 50 | - $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node)); |
|
| 50 | + $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node)); |
|
| 51 | 51 | $classOrFunction = $this->metrics->get($name); |
| 52 | 52 | |
| 53 | 53 | if (null === $lloc = $classOrFunction->get('lloc')) { |
@@ -58,20 +58,20 @@ discard block |
||
| 58 | 58 | // extends |
| 59 | 59 | if (isset($node->extends)) { |
| 60 | 60 | if (is_array($node->extends)) { |
| 61 | - foreach ((array)$node->extends as $interface) { |
|
| 62 | - $this->pushToDependencies($dependencies, (string)$interface); |
|
| 63 | - array_push($parents, (string)$interface); |
|
| 61 | + foreach ((array) $node->extends as $interface) { |
|
| 62 | + $this->pushToDependencies($dependencies, (string) $interface); |
|
| 63 | + array_push($parents, (string) $interface); |
|
| 64 | 64 | } |
| 65 | 65 | } else { |
| 66 | - $this->pushToDependencies($dependencies, (string)$node->extends); |
|
| 67 | - array_push($parents, (string)$node->extends); |
|
| 66 | + $this->pushToDependencies($dependencies, (string) $node->extends); |
|
| 67 | + array_push($parents, (string) $node->extends); |
|
| 68 | 68 | } |
| 69 | 69 | } |
| 70 | 70 | |
| 71 | 71 | // implements |
| 72 | 72 | if (isset($node->implements)) { |
| 73 | 73 | foreach ($node->implements as $interface) { |
| 74 | - $this->pushToDependencies($dependencies, (string)$interface); |
|
| 74 | + $this->pushToDependencies($dependencies, (string) $interface); |
|
| 75 | 75 | } |
| 76 | 76 | } |
| 77 | 77 | |
@@ -80,7 +80,7 @@ discard block |
||
| 80 | 80 | // return |
| 81 | 81 | if (isset($stmt->returnType)) { |
| 82 | 82 | if ($stmt->returnType instanceof Node\Name\FullyQualified) { |
| 83 | - $this->pushToDependencies($dependencies, (string)$stmt->returnType); |
|
| 83 | + $this->pushToDependencies($dependencies, (string) $stmt->returnType); |
|
| 84 | 84 | } |
| 85 | 85 | } |
| 86 | 86 | |
@@ -88,13 +88,13 @@ discard block |
||
| 88 | 88 | foreach ($stmt->params as $param) { |
| 89 | 89 | if ($param->type) { |
| 90 | 90 | if ($param->type instanceof Node\Name\FullyQualified) { |
| 91 | - $this->pushToDependencies($dependencies, (string)$param->type); |
|
| 91 | + $this->pushToDependencies($dependencies, (string) $param->type); |
|
| 92 | 92 | } |
| 93 | 93 | } |
| 94 | 94 | } |
| 95 | 95 | |
| 96 | 96 | // instantiations, static calls |
| 97 | - \iterate_over_node($stmt, function ($node) use (&$dependencies) { |
|
| 97 | + \iterate_over_node($stmt, function($node) use (&$dependencies) { |
|
| 98 | 98 | switch (true) { |
| 99 | 99 | case $node instanceof Node\Expr\New_: |
| 100 | 100 | // new MyClass |
@@ -113,13 +113,13 @@ discard block |
||
| 113 | 113 | foreach ($matches[1] as $check) { |
| 114 | 114 | foreach ($this->uses as $use) { |
| 115 | 115 | if (method_exists($use, 'getAlias')) { |
| 116 | - if (((string)$use->getAlias()) === $check) { |
|
| 117 | - $this->pushToDependencies($dependencies, (string)($use->name)); |
|
| 116 | + if (((string) $use->getAlias()) === $check) { |
|
| 117 | + $this->pushToDependencies($dependencies, (string) ($use->name)); |
|
| 118 | 118 | } |
| 119 | 119 | continue; |
| 120 | 120 | } |
| 121 | 121 | if ($use->alias === $check) { |
| 122 | - $this->pushToDependencies($dependencies, (string)($use->name)); |
|
| 122 | + $this->pushToDependencies($dependencies, (string) ($use->name)); |
|
| 123 | 123 | } |
| 124 | 124 | } |
| 125 | 125 | } |
@@ -138,6 +138,6 @@ discard block |
||
| 138 | 138 | if ('self' === $lowercase || 'parent' === $lowercase) { |
| 139 | 139 | return; |
| 140 | 140 | } |
| 141 | - array_push($dependencies, (string)$dependency); |
|
| 141 | + array_push($dependencies, (string) $dependency); |
|
| 142 | 142 | } |
| 143 | 143 | } |
@@ -44,10 +44,10 @@ discard block |
||
| 44 | 44 | { |
| 45 | 45 | if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Function_ || $node instanceof Stmt\Trait_) { |
| 46 | 46 | if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Trait_) { |
| 47 | - $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node)); |
|
| 47 | + $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node)); |
|
| 48 | 48 | $classOrFunction = $this->metrics->get($name); |
| 49 | 49 | } else { |
| 50 | - $classOrFunction = new FunctionMetric((string)$node->name); |
|
| 50 | + $classOrFunction = new FunctionMetric((string) $node->name); |
|
| 51 | 51 | $this->metrics->attach($classOrFunction); |
| 52 | 52 | } |
| 53 | 53 | |
@@ -55,7 +55,7 @@ discard block |
||
| 55 | 55 | $operands = []; |
| 56 | 56 | $operators = []; |
| 57 | 57 | |
| 58 | - iterate_over_node($node, function ($node) use (&$operators, &$operands) { |
|
| 58 | + iterate_over_node($node, function($node) use (&$operators, &$operands) { |
|
| 59 | 59 | if ($node instanceof Node\Expr\BinaryOp |
| 60 | 60 | || $node instanceof Node\Expr\AssignOp |
| 61 | 61 | || $node instanceof Stmt\If_ |
@@ -34,10 +34,10 @@ discard block |
||
| 34 | 34 | { |
| 35 | 35 | if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Function_ || $node instanceof Stmt\Trait_) { |
| 36 | 36 | if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Trait_) { |
| 37 | - $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node)); |
|
| 37 | + $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node)); |
|
| 38 | 38 | $classOrFunction = $this->metrics->get($name); |
| 39 | 39 | } else { |
| 40 | - $classOrFunction = new FunctionMetric((string)$node->name); |
|
| 40 | + $classOrFunction = new FunctionMetric((string) $node->name); |
|
| 41 | 41 | $this->metrics->attach($classOrFunction); |
| 42 | 42 | } |
| 43 | 43 | |
@@ -57,7 +57,7 @@ discard block |
||
| 57 | 57 | $code = preg_replace('!/\*.*?\*/!s', '', $code); |
| 58 | 58 | |
| 59 | 59 | // count and remove single line comments |
| 60 | - $code = preg_replace_callback('!(\'[^\']*\'|"[^"]*")|((?:#|\/\/).*$)!m', function (array $matches) use (&$cloc) { |
|
| 60 | + $code = preg_replace_callback('!(\'[^\']*\'|"[^"]*")|((?:#|\/\/).*$)!m', function(array $matches) use (&$cloc) { |
|
| 61 | 61 | if (isset($matches[2])) { |
| 62 | 62 | $cloc += 1; |
| 63 | 63 | } |
@@ -42,12 +42,12 @@ discard block |
||
| 42 | 42 | |
| 43 | 43 | foreach ($node->stmts as $stmt) { |
| 44 | 44 | if ($stmt instanceof Stmt\ClassMethod) { |
| 45 | - if (!$graph->has($stmt->name . '()')) { |
|
| 46 | - $graph->insert(new TreeNode($stmt->name . '()')); |
|
| 45 | + if (!$graph->has($stmt->name.'()')) { |
|
| 46 | + $graph->insert(new TreeNode($stmt->name.'()')); |
|
| 47 | 47 | } |
| 48 | - $from = $graph->get($stmt->name . '()'); |
|
| 48 | + $from = $graph->get($stmt->name.'()'); |
|
| 49 | 49 | |
| 50 | - \iterate_over_node($stmt, function ($node) use ($from, &$graph) { |
|
| 50 | + \iterate_over_node($stmt, function($node) use ($from, &$graph) { |
|
| 51 | 51 | if ($node instanceof Node\Expr\PropertyFetch && isset($node->var->name) && $node->var->name == 'this') { |
| 52 | 52 | $name = getNameOfNode($node); |
| 53 | 53 | // use of attribute $this->xxx; |
@@ -63,7 +63,7 @@ discard block |
||
| 63 | 63 | if (!$node->var instanceof Node\Expr\New_ && isset($node->var->name) && getNameOfNode($node->var) === 'this') { |
| 64 | 64 | // use of method call $this->xxx(); |
| 65 | 65 | // use of attribute $this->xxx; |
| 66 | - $name = getNameOfNode($node->name) . '()'; |
|
| 66 | + $name = getNameOfNode($node->name).'()'; |
|
| 67 | 67 | if (!$graph->has($name)) { |
| 68 | 68 | $graph->insert(new TreeNode($name)); |
| 69 | 69 | } |
@@ -27,7 +27,7 @@ discard block |
||
| 27 | 27 | public function enterNode(Node $node) |
| 28 | 28 | { |
| 29 | 29 | if ($node instanceof Namespace_) { |
| 30 | - $this->namespace = (string)$node->name; |
|
| 30 | + $this->namespace = (string) $node->name; |
|
| 31 | 31 | } |
| 32 | 32 | } |
| 33 | 33 | |
@@ -42,17 +42,17 @@ discard block |
||
| 42 | 42 | $package = $matches[1]; |
| 43 | 43 | } |
| 44 | 44 | if (preg_match('/^\s*\* @subpackage (.*)/m', $docBlockText, $matches)) { |
| 45 | - $package = $package . '\\' . $matches[1]; |
|
| 45 | + $package = $package.'\\'.$matches[1]; |
|
| 46 | 46 | } |
| 47 | 47 | |
| 48 | - $packageName = $package . '\\'; |
|
| 49 | - if (! $packageMetric = $this->metrics->get($packageName)) { |
|
| 48 | + $packageName = $package.'\\'; |
|
| 49 | + if (!$packageMetric = $this->metrics->get($packageName)) { |
|
| 50 | 50 | $packageMetric = new PackageMetric($packageName); |
| 51 | 51 | $this->metrics->attach($packageMetric); |
| 52 | 52 | } |
| 53 | 53 | /* @var PackageMetric $packageMetric */ |
| 54 | - $elementName = isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node); |
|
| 55 | - $elementName = (string)$elementName; |
|
| 54 | + $elementName = isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node); |
|
| 55 | + $elementName = (string) $elementName; |
|
| 56 | 56 | $packageMetric->addClass($elementName); |
| 57 | 57 | |
| 58 | 58 | $this->metrics->get($elementName)->set('package', $packageName); |