phpmetrics / PhpMetrics

src/functions.php

Spacing +7 added lines, -7 removed lines

@@ -89,7 +89,7 @@ 
     }
 
     if ($node instanceof \PhpParser\Node\Name\FullyQualified) {
-        return (string)$node;
+        return (string) $node;
     }
     if ($node instanceof \PhpParser\Node\Expr\New_) {
         return getNameOfNode($node->class);
@@ -100,7 +100,7 @@ 
     }
 
     if ($node instanceof \PhpParser\Node\Name) {
-        return (string)implode($node->parts);
+        return (string) implode($node->parts);
     }
 
     if (isset($node->name) && $node->name instanceof \PhpParser\Node\Expr\Variable) {
@@ -128,11 +128,11 @@ 
     }
 
     if (isset($node->name) && null === $node->name) {
-        return 'anonymous@' . spl_object_hash($node);
+        return 'anonymous@'.spl_object_hash($node);
     }
 
     if (isset($node->name)) {
-        return (string)$node->name;
+        return (string) $node->name;
     }
 
     return null;
@@ -150,10 +150,10 @@ 
     }
     while (false !== ($file = readdir($dir))) {
         if (('.' != $file) && ('..' != $file)) {
-            if (is_dir($src . '/' . $file)) {
-                recurse_copy($src . '/' . $file, $dst . '/' . $file);
+            if (is_dir($src.'/'.$file)) {
+                recurse_copy($src.'/'.$file, $dst.'/'.$file);
             } else {
-                copy($src . '/' . $file, $dst . '/' . $file);
+                copy($src.'/'.$file, $dst.'/'.$file);
             }
         }
     }

src/Hal/Metric/Consolidated.php

Spacing +3 added lines, -3 removed lines

@@ -66,13 +66,13 @@ 
         }
 
         // sums
-        $sum = (object)[
+        $sum = (object) [
             'loc' => 0,
             'cloc' => 0,
             'lloc' => 0,
             'nbMethods' => 0,
         ];
-        $avg = (object)[
+        $avg = (object) [
             'ccn' => [],
             'bugs' => [],
             'kanDefect' => [],
@@ -132,7 +132,7 @@ 
                 $violations[$name]++;
             }
         }
-        $sum->violations = (object)$violations;
+        $sum->violations = (object) $violations;
 
         $this->avg = $avg;
         $this->sum = $sum;

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

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
                     $output = 0;
                     $fanout = [];
 
-                    \iterate_over_node($node, function ($node) use (&$output, &$fanout) {
+                    \iterate_over_node($node, function($node) use (&$output, &$fanout) {
                         switch (true) {
                             case $node instanceof Stmt\Return_:
                                 $output++;

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

Spacing +5 added lines, -5 removed lines

@@ -44,12 +44,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) && 'this' == $node->var->name) {
                             $name = \getNameOfNode($node);
@@ -67,7 +67,7 @@ 
                             if (!$node->var instanceof Node\Expr\New_ && isset($node->var->name) && 'this' === \getNameOfNode($node->var)) {
                                 // 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/Class_/Text/HalsteadVisitor.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
     {
         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($node->name);
@@ -56,7 +56,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

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

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
     {
         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($node->name);
@@ -60,7 +60,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;
                 }

src/Hal/Metric/Class_/Complexity/KanDefectVisitor.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
 
             $select = $while = $if = 0;
 
-            \iterate_over_node($node, function ($node) use (&$while, &$select, &$if) {
+            \iterate_over_node($node, function($node) use (&$while, &$select, &$if) {
                 switch (true) {
                     case $node instanceof Stmt\Do_:
                     case $node instanceof Stmt\Foreach_:
@@ -59,7 +59,7 @@ 
                 }
             });
 
-            $defect = 0.15 + 0.23 *  $while + 0.22 *  $select + 0.07 * $if;
+            $defect = 0.15 + 0.23 * $while + 0.22 * $select + 0.07 * $if;
             $class->set('kanDefect', \round($defect, 2));
         }
     }

src/Hal/Metric/System/UnitTesting/UnitTesting.php

Spacing +5 added lines, -5 removed lines

@@ -80,7 +80,7 @@ 
 
         // JUNIT format
         foreach ($xpath->query('//testsuite[@file]') as $suite) {
-            $testsuites[] = (object)[
+            $testsuites[] = (object) [
                 'file' => $suite->getAttribute('file'),
                 'name' => $suite->getAttribute('name'),
                 'assertions' => $suite->getAttribute('assertions'),
@@ -110,7 +110,7 @@ 
             }
 
             $attributeClass = $case->getAttribute('class');
-            $testsuites[$attributeClass] = (object)[
+            $testsuites[$attributeClass] = (object) [
                 'file' => $case->getAttribute('file'),
                 'name' => $attributeClass,
                 'assertions' => $assertions,
@@ -129,7 +129,7 @@ 
             $traverser->addVisitor(new ExternalsVisitor($metricsOfUnitTest));
 
             if (!\file_exists($suite->file) || !\is_readable($suite->file)) {
-                throw new \LogicException('Cannot find source file referenced in testsuite: ' . $suite->file);
+                throw new \LogicException('Cannot find source file referenced in testsuite: '.$suite->file);
             }
 
             $code = \file_get_contents($suite->file);
@@ -142,10 +142,10 @@ 
 
             // list of externals sources of unit test
             $metric = $metricsOfUnitTest->get($suite->name);
-            $externals = (array)$metric->get('externals');
+            $externals = (array) $metric->get('externals');
 
             // global stats for each test
-            $infoAboutTests[$suite->name] = (object)[
+            $infoAboutTests[$suite->name] = (object) [
                 'nbExternals' => \count(\array_unique($externals)),
                 'externals' => \array_unique($externals),
                 'filename' => $suite->file,

Doc Comments +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
      * GitChanges constructor.
      *
      * @param Config $config
-     * @param array $files
+     * @param string[] $files
      */
     public function __construct(Config $config, array $files)
     {

src/Hal/Metric/System/Packages/Composer/Composer.php

Spacing +4 added lines, -4 removed lines

@@ -46,7 +46,7 @@ 
         foreach ($rawRequirements as $requirement => $version) {
             $package = $packagist->get($requirement);
 
-            $packages[$requirement] = (object)[
+            $packages[$requirement] = (object) [
                 'name' => $requirement,
                 'required' => $version,
                 'installed' => isset($rawInstalled[$requirement]) ? $rawInstalled[$requirement] : null,
@@ -77,13 +77,13 @@ 
             if (!\preg_match('/composer(-dist)?\.json/', $filename)) {
                 continue;
             }
-            $composerJson = (object)\json_decode(\file_get_contents($filename));
+            $composerJson = (object) \json_decode(\file_get_contents($filename));
 
             if (!isset($composerJson->require)) {
                 continue;
             }
 
-            $rawRequirements[] = (array)$composerJson->require;
+            $rawRequirements[] = (array) $composerJson->require;
         }
 
         return \call_user_func_array('\array_merge', $rawRequirements);
@@ -107,7 +107,7 @@ 
             if (false === \strpos($filename, 'composer.lock')) {
                 continue;
             }
-            $composerLockJson = (object)\json_decode(\file_get_contents($filename));
+            $composerLockJson = (object) \json_decode(\file_get_contents($filename));
 
             if (!isset($composerLockJson->packages)) {
                 continue;

Doc Comments +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
      * GitChanges constructor.
      *
      * @param Config $config
-     * @param array $files
+     * @param string[] $files
      */
     public function __construct(Config $config, array $files)
     {