phpmetrics / PhpMetrics

src/Hal/Metric/Class_/ClassEnumVisitor.php

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
                 $class->set('interface', true);
                 $class->set('abstract', true);
             } else {
-                $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node));
+                $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node));
                 $class = new ClassMetric($name);
                 $class->set('interface', false);
                 $class->set('abstract', $node instanceof Stmt\Trait_ || $node->isAbstract());
@@ -50,7 +50,7 @@ 
             $roleDetector = new RoleOfMethodDetector();
             foreach ($node->stmts as $stmt) {
                 if ($stmt instanceof Stmt\ClassMethod) {
-                    $function = new FunctionMetric((string)$stmt->name);
+                    $function = new FunctionMetric((string) $stmt->name);
 
                     $role = $roleDetector->detects($stmt);
                     $function->set('role', $role);

src/Hal/Metric/Class_/Component/MaintainabilityIndexVisitor.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     public function leaveNode(Node $node)
     {
         if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Trait_) {
-            $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node));
+            $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node));
             $classOrFunction = $this->metrics->get($name);
 
             if (null === $lloc = $classOrFunction->get('lloc')) {

src/Hal/Metric/Class_/Coupling/ExternalsVisitor.php

Spacing +13 added lines, -13 removed lines

@@ -58,20 +58,20 @@ 
             // extends
             if (isset($node->extends)) {
                 if (is_array($node->extends)) {
-                    foreach ((array)$node->extends as $interface) {
-                        $this->pushToDependencies($dependencies, (string)$interface);
-                        array_push($parents, (string)$interface);
+                    foreach ((array) $node->extends as $interface) {
+                        $this->pushToDependencies($dependencies, (string) $interface);
+                        array_push($parents, (string) $interface);
                     }
                 } else {
-                    $this->pushToDependencies($dependencies, (string)$node->extends);
-                    array_push($parents, (string)$node->extends);
+                    $this->pushToDependencies($dependencies, (string) $node->extends);
+                    array_push($parents, (string) $node->extends);
                 }
             }
 
             // implements
             if (isset($node->implements)) {
                 foreach ($node->implements as $interface) {
-                    $this->pushToDependencies($dependencies, (string)$interface);
+                    $this->pushToDependencies($dependencies, (string) $interface);
                 }
             }
 
@@ -80,7 +80,7 @@ 
                     // return
                     if (isset($stmt->returnType)) {
                         if ($stmt->returnType instanceof Node\Name\FullyQualified) {
-                            $this->pushToDependencies($dependencies, (string)$stmt->returnType);
+                            $this->pushToDependencies($dependencies, (string) $stmt->returnType);
                         }
                     }
 
@@ -88,13 +88,13 @@ 
                     foreach ($stmt->params as $param) {
                         if ($param->type) {
                             if ($param->type instanceof Node\Name\FullyQualified) {
-                                $this->pushToDependencies($dependencies, (string)$param->type);
+                                $this->pushToDependencies($dependencies, (string) $param->type);
                             }
                         }
                     }
 
                     // instantiations, static calls
-                    \iterate_over_node($stmt, function ($node) use (&$dependencies) {
+                    \iterate_over_node($stmt, function($node) use (&$dependencies) {
                         switch (true) {
                             case $node instanceof Node\Expr\New_:
                                 // new MyClass
@@ -113,13 +113,13 @@ 
                         foreach ($matches[1] as $check) {
                             foreach ($this->uses as $use) {
                                 if (method_exists($use, 'getAlias')) {
-                                    if (((string)$use->getAlias()) === $check) {
-                                        $this->pushToDependencies($dependencies, (string)($use->name));
+                                    if (((string) $use->getAlias()) === $check) {
+                                        $this->pushToDependencies($dependencies, (string) ($use->name));
                                     }
                                     continue;
                                 }
                                 if ($use->alias === $check) {
-                                    $this->pushToDependencies($dependencies, (string)($use->name));
+                                    $this->pushToDependencies($dependencies, (string) ($use->name));
                                 }
                             }
                         }
@@ -138,6 +138,6 @@ 
         if ('self' === $lowercase || 'parent' === $lowercase) {
             return;
         }
-        array_push($dependencies, (string)$dependency);
+        array_push($dependencies, (string) $dependency);
     }
 }

src/Hal/Metric/Class_/Text/HalsteadVisitor.php

Spacing +3 added lines, -3 removed lines

@@ -44,10 +44,10 @@ 
     {
         if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Function_ || $node instanceof Stmt\Trait_) {
             if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Trait_) {
-                $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node));
+                $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node));
                 $classOrFunction = $this->metrics->get($name);
             } else {
-                $classOrFunction = new FunctionMetric((string)$node->name);
+                $classOrFunction = new FunctionMetric((string) $node->name);
                 $this->metrics->attach($classOrFunction);
             }
 
@@ -55,7 +55,7 @@ 
             $operands = [];
             $operators = [];
 
-            iterate_over_node($node, function ($node) use (&$operators, &$operands) {
+            iterate_over_node($node, function($node) use (&$operators, &$operands) {
                 if ($node instanceof Node\Expr\BinaryOp
                     || $node instanceof Node\Expr\AssignOp
                     || $node instanceof Stmt\If_

src/Hal/Metric/Class_/Text/LengthVisitor.php

Spacing +3 added lines, -3 removed lines

@@ -34,10 +34,10 @@ 
     {
         if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Function_ || $node instanceof Stmt\Trait_) {
             if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Trait_) {
-                $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node));
+                $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node));
                 $classOrFunction = $this->metrics->get($name);
             } else {
-                $classOrFunction = new FunctionMetric((string)$node->name);
+                $classOrFunction = new FunctionMetric((string) $node->name);
                 $this->metrics->attach($classOrFunction);
             }
 
@@ -57,7 +57,7 @@ 
             $code = preg_replace('!/\*.*?\*/!s', '', $code);
 
             // count and remove single line comments
-            $code = preg_replace_callback('!(\'[^\']*\'|"[^"]*")|((?:#|\/\/).*$)!m', function (array $matches) use (&$cloc) {
+            $code = preg_replace_callback('!(\'[^\']*\'|"[^"]*")|((?:#|\/\/).*$)!m', function(array $matches) use (&$cloc) {
                 if (isset($matches[2])) {
                     $cloc += 1;
                 }

src/Hal/Metric/Class_/Structural/LcomVisitor.php

Spacing +5 added lines, -5 removed lines

@@ -42,12 +42,12 @@ 
 
             foreach ($node->stmts as $stmt) {
                 if ($stmt instanceof Stmt\ClassMethod) {
-                    if (!$graph->has($stmt->name . '()')) {
-                        $graph->insert(new TreeNode($stmt->name . '()'));
+                    if (!$graph->has($stmt->name.'()')) {
+                        $graph->insert(new TreeNode($stmt->name.'()'));
                     }
-                    $from = $graph->get($stmt->name . '()');
+                    $from = $graph->get($stmt->name.'()');
 
-                    \iterate_over_node($stmt, function ($node) use ($from, &$graph) {
+                    \iterate_over_node($stmt, function($node) use ($from, &$graph) {
                         if ($node instanceof Node\Expr\PropertyFetch && isset($node->var->name) && $node->var->name == 'this') {
                             $name = getNameOfNode($node);
                             // use of attribute $this->xxx;
@@ -63,7 +63,7 @@ 
                             if (!$node->var instanceof Node\Expr\New_ && isset($node->var->name) && getNameOfNode($node->var) === 'this') {
                                 // use of method call $this->xxx();
                                 // use of attribute $this->xxx;
-                                $name = getNameOfNode($node->name) . '()';
+                                $name = getNameOfNode($node->name).'()';
                                 if (!$graph->has($name)) {
                                     $graph->insert(new TreeNode($name));
                                 }

src/Hal/Metric/Package/PackageCollectingVisitor.php

Spacing +6 added lines, -6 removed lines

@@ -27,7 +27,7 @@ 
     public function enterNode(Node $node)
     {
         if ($node instanceof Namespace_) {
-            $this->namespace = (string)$node->name;
+            $this->namespace = (string) $node->name;
         }
     }
 
@@ -42,17 +42,17 @@ 
                 $package = $matches[1];
             }
             if (preg_match('/^\s*\* @subpackage (.*)/m', $docBlockText, $matches)) {
-                $package = $package . '\\' . $matches[1];
+                $package = $package.'\\'.$matches[1];
             }
 
-            $packageName = $package . '\\';
-            if (! $packageMetric = $this->metrics->get($packageName)) {
+            $packageName = $package.'\\';
+            if (!$packageMetric = $this->metrics->get($packageName)) {
                 $packageMetric = new PackageMetric($packageName);
                 $this->metrics->attach($packageMetric);
             }
             /* @var PackageMetric $packageMetric */
-            $elementName = isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node);
-            $elementName = (string)$elementName;
+            $elementName = isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node);
+            $elementName = (string) $elementName;
             $packageMetric->addClass($elementName);
 
             $this->metrics->get($elementName)->set('package', $packageName);

src/Hal/Metric/PackageMetric.php

Spacing +7 added lines, -7 removed lines

@@ -18,7 +18,7 @@ 
     public function addClass($name)
     {
         $elements = $this->get('classes');
-        $elements[] = (string)$name;
+        $elements[] = (string) $name;
         $this->set('classes', $elements);
     }
 
@@ -26,7 +26,7 @@ 
     public function setAbstraction($abstraction)
     {
         if ($abstraction !== null) {
-            $abstraction = (float)$abstraction;
+            $abstraction = (float) $abstraction;
         }
         $this->set('abstraction', $abstraction);
     }
@@ -41,7 +41,7 @@ 
     public function setInstability($instability)
     {
         if ($instability !== null) {
-            $instability = (float)$instability;
+            $instability = (float) $instability;
         }
         $this->set('instability', $instability);
     }
@@ -63,11 +63,11 @@ 
         }
         $classDependencies = $this->getOutgoingClassDependencies();
         $packageDependencies = $this->getOutgoingPackageDependencies();
-        if (! in_array($className, $classDependencies)) {
+        if (!in_array($className, $classDependencies)) {
             $classDependencies[] = $className;
             $this->set('outgoing_class_dependencies', $classDependencies);
         }
-        if (! in_array($packageName, $packageDependencies)) {
+        if (!in_array($packageName, $packageDependencies)) {
             $packageDependencies[] = $packageName;
             $this->set('outgoing_package_dependencies', $packageDependencies);
         }
@@ -96,11 +96,11 @@ 
         }
         $classDependencies = $this->getIncomingClassDependencies();
         $packageDependencies = $this->getIncomingPackageDependencies();
-        if (! in_array($className, $classDependencies)) {
+        if (!in_array($className, $classDependencies)) {
             $classDependencies[] = $className;
             $this->set('incoming_class_dependencies', $classDependencies);
         }
-        if (! in_array($packageName, $packageDependencies)) {
+        if (!in_array($packageName, $packageDependencies)) {
             $packageDependencies[] = $packageName;
             $this->set('incoming_package_dependencies', $packageDependencies);
         }

src/Hal/Report/Html/Reporter.php

Spacing +30 added lines, -30 removed lines

@@ -41,11 +41,11 @@ 
         $consolidated = new Consolidated($metrics);
 
         // history of builds
-        $today = (object)[
+        $today = (object) [
             'avg' => $consolidated->getAvg(),
             'sum' => $consolidated->getSum()
         ];
-        $files = glob($logDir . '/js/history-*.json');
+        $files = glob($logDir.'/js/history-*.json');
         $next = count($files) + 1;
         $history = [];
         natsort($files);
@@ -54,39 +54,39 @@ 
         }
 
         // copy sources
-        if (!file_exists($logDir . '/js')) {
-            mkdir($logDir . '/js', 0755, true);
+        if (!file_exists($logDir.'/js')) {
+            mkdir($logDir.'/js', 0755, true);
         }
-        if (!file_exists($logDir . '/css')) {
-            mkdir($logDir . '/css', 0755, true);
+        if (!file_exists($logDir.'/css')) {
+            mkdir($logDir.'/css', 0755, true);
         }
-        if (!file_exists($logDir . '/images')) {
-            mkdir($logDir . '/images', 0755, true);
+        if (!file_exists($logDir.'/images')) {
+            mkdir($logDir.'/images', 0755, true);
         }
-        if (!file_exists($logDir . '/fonts')) {
-            mkdir($logDir . '/fonts', 0755, true);
+        if (!file_exists($logDir.'/fonts')) {
+            mkdir($logDir.'/fonts', 0755, true);
         }
-        recurse_copy(__DIR__ . '/template/js', $logDir . '/js');
-        recurse_copy(__DIR__ . '/template/css', $logDir . '/css');
-        recurse_copy(__DIR__ . '/template/images', $logDir . '/images');
-        recurse_copy(__DIR__ . '/template/fonts', $logDir . '/fonts');
+        recurse_copy(__DIR__.'/template/js', $logDir.'/js');
+        recurse_copy(__DIR__.'/template/css', $logDir.'/css');
+        recurse_copy(__DIR__.'/template/images', $logDir.'/images');
+        recurse_copy(__DIR__.'/template/fonts', $logDir.'/fonts');
 
         // render dynamic pages
-        $this->renderPage(__DIR__ . '/template/index.php', $logDir . '/index.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/loc.php', $logDir . '/loc.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/relations.php', $logDir . '/relations.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/coupling.php', $logDir . '/coupling.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/all.php', $logDir . '/all.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/oop.php', $logDir . '/oop.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/complexity.php', $logDir . '/complexity.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/panel.php', $logDir . '/panel.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/violations.php', $logDir . '/violations.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/packages.php', $logDir . '/packages.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/package_relations.php', $logDir . '/package_relations.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/index.php', $logDir.'/index.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/loc.php', $logDir.'/loc.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/relations.php', $logDir.'/relations.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/coupling.php', $logDir.'/coupling.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/all.php', $logDir.'/all.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/oop.php', $logDir.'/oop.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/complexity.php', $logDir.'/complexity.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/panel.php', $logDir.'/panel.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/violations.php', $logDir.'/violations.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/packages.php', $logDir.'/packages.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/package_relations.php', $logDir.'/package_relations.html', $consolidated, $history);
         if ($this->config->has('git')) {
-            $this->renderPage(__DIR__ . '/template/git.php', $logDir . '/git.html', $consolidated, $history);
+            $this->renderPage(__DIR__.'/template/git.php', $logDir.'/git.html', $consolidated, $history);
         }
-        $this->renderPage(__DIR__ . '/template/junit.php', $logDir . '/junit.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/junit.php', $logDir.'/junit.html', $consolidated, $history);
 
         // js data
         file_put_contents(
@@ -100,8 +100,8 @@ 
 
         // json data
         file_put_contents(
-            $logDir . '/js/classes.js',
-            'var classes = ' . json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
+            $logDir.'/js/classes.js',
+            'var classes = '.json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
         );
 
         $this->output->writeln(sprintf('HTML report generated in "%s" directory', $logDir));
@@ -168,7 +168,7 @@ 
 
         $diff = $newValue - $oldValue;
         if ($diff > 0) {
-            $diff = '+' . $diff;
+            $diff = '+'.$diff;
         }
 
         $goodOrBad = 'neutral';