phpmetrics / PhpMetrics

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

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,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_:
@@ -58,7 +58,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));
         }
     }

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace Hal\Metric\Class_\Complexity;
 
-use Hal\Component\Reflected\Method;
 use Hal\Metric\Metrics;
 use PhpParser\Node;
 use PhpParser\Node\Stmt;

src/Hal/Metric/System/Changes/GitChanges.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
         }
 
         $r = shell_exec(sprintf('%s --version', $bin));
-        if(!preg_match('!git version!', $r)) {
+        if (!preg_match('!git version!', $r)) {
             throw new ConfigException(sprintf('Git binary (%s) incorrect', $bin));
         }
 

src/Hal/Metric/Helper/RoleOfMethodDetector.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
         // build a fingerprint of the given method
         $fingerprintOfMethod = [];
-        iterate_over_node($node, function ($node) use (&$fingerprintOfMethod) {
+        iterate_over_node($node, function($node) use (&$fingerprintOfMethod) {
 
             // avoid cast
             if ($node instanceof Cast) {

artifacts/phar/build.php

Spacing +5 added lines, -5 removed lines

@@ -1,8 +1,8 @@ 
 <?php
-chdir(__DIR__ . '/../../');
+chdir(__DIR__.'/../../');
 
 if (!file_exists('vendor/autoload.php')) {
-    echo '[ERROR] It\'s required to run "composer install" before building PhpMetrics!' . PHP_EOL;
+    echo '[ERROR] It\'s required to run "composer install" before building PhpMetrics!'.PHP_EOL;
     exit(1);
 }
 
@@ -21,7 +21,7 @@ 
     if (preg_match($exclude, $file)) {
         continue;
     }
-    $path = str_replace(__DIR__ . '/', '', $file);
+    $path = str_replace(__DIR__.'/', '', $file);
     $phar->addFromString($path, file_get_contents($file));
 }
 
@@ -52,8 +52,8 @@ 
 
 function rglob($pattern = '*', $flags = 0, $path = '')
 {
-    $paths = glob($path . '*', GLOB_MARK | GLOB_ONLYDIR | GLOB_NOSORT);
-    $files = glob($path . $pattern, $flags);
+    $paths = glob($path.'*', GLOB_MARK|GLOB_ONLYDIR|GLOB_NOSORT);
+    $files = glob($path.$pattern, $flags);
     foreach ($paths as $path) {
         $files = array_merge($files, rglob($pattern, $flags, $path));
     }

src/functions.php

Doc Comments +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
 
 /**
  * @param $node
- * @param $callback
+ * @param Closure $callback
  */
 function iterate_over_node($node, $callback)
 {
@@ -136,8 +136,8 @@ 
 }
 
 /**
- * @param $src
- * @param $dst
+ * @param string $src
+ * @param string $dst
  */
 function recurse_copy($src, $dst)
 {

Spacing +7 added lines, -7 removed lines

@@ -90,7 +90,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);
@@ -101,7 +101,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) {
@@ -129,11 +129,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;
@@ -151,10 +151,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

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

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

Spacing +8 added lines, -8 removed lines

@@ -60,18 +60,18 @@ 
             // extends
             if (isset($node->extends)) {
                 if (is_array($node->extends)) {
-                    foreach ((array)$node->extends as $interface) {
-                        array_push($dependencies, (string)$interface);
+                    foreach ((array) $node->extends as $interface) {
+                        array_push($dependencies, (string) $interface);
                     }
                 } else {
-                    array_push($dependencies, (string)$node->extends);
+                    array_push($dependencies, (string) $node->extends);
                 }
             }
 
             // implements
             if (isset($node->implements)) {
                 foreach ($node->implements as $interface) {
-                    array_push($dependencies, (string)$interface);
+                    array_push($dependencies, (string) $interface);
                 }
             }
 
@@ -80,7 +80,7 @@ 
                     // return
                     if (isset($stmt->returnType)) {
                         if ($stmt->returnType instanceof Node\Name\FullyQualified) {
-                            array_push($dependencies, (string)$stmt->returnType);
+                            array_push($dependencies, (string) $stmt->returnType);
                         }
                     }
 
@@ -88,13 +88,13 @@ 
                     foreach ($stmt->params as $param) {
                         if ($param->type) {
                             if ($param->type instanceof Node\Name\FullyQualified) {
-                                array_push($dependencies, (string)$param->type);
+                                array_push($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,7 +113,7 @@ 
                         foreach ($matches[1] as $check) {
                             foreach ($this->uses as $use) {
                                 if ($use->alias === $check) {
-                                    array_push($dependencies, (string)($use->name));
+                                    array_push($dependencies, (string) ($use->name));
                                 }
                             }
                         }

src/Hal/Report/Html/template/loc.php

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php require __DIR__ . '/_header.php'; ?>
+<?php require __DIR__.'/_header.php'; ?>
 
 
 <?php
@@ -19,10 +19,10 @@ 
 
 // 2. percentile map
 $json = [];
-if(sizeof($array) > 1) {
+if (sizeof($array) > 1) {
     $range = range(0.5, 1, .05);
     foreach ($range as $percentile) {
-        $json[] = (object)[
+        $json[] = (object) [
             'lloc' => procentile($array, $percentile),
             'percentile' => $percentile * 100,
         ];
@@ -74,7 +74,7 @@ 
 </div>
 
 
-<?php require __DIR__ . '/_footer.php'; ?>
+<?php require __DIR__.'/_footer.php'; ?>
 
 
 <script>

src/Hal/Application/Analyze.php

Doc Comments +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 
     /**
      * Analyze constructor.
-     * @param OutputInterface $output
+     * @param Output $output
      */
     public function __construct(Config $config, Output $output, Issuer $issuer)
     {