phpmetrics / PhpMetrics

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

Spacing +1 added lines, -1 removed lines

@@ -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

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

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,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

@@ -45,12 +45,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);
@@ -68,7 +68,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/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/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)
     {