phpmetrics / PhpMetrics

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

Spacing +3 added lines, -3 removed lines

@@ -45,10 +45,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 {
-                $name = (string)$node->name;
+                $name = (string) $node->name;
                 $classOrFunction = new FunctionMetric($name);
                 $this->metrics->attach($classOrFunction);
             }
@@ -61,7 +61,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_

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

Spacing +6 added lines, -6 removed lines

@@ -49,17 +49,17 @@ 
 
             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.'()');
                     if ($from === null) {
                         throw new ShouldNotHappenException('Graph $from is null');
                     }
 
-                    \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 = (string)getNameOfNode($node);
+                            $name = (string) getNameOfNode($node);
                             // use of attribute $this->xxx;
                             if (!$graph->has($name)) {
                                 $graph->insert(new TreeNode($name));
@@ -76,7 +76,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));
                                 }

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

Spacing +15 added lines, -15 removed lines

@@ -64,20 +64,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);
                 }
             }
 
@@ -86,7 +86,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);
                         }
                     }
 
@@ -94,21 +94,21 @@ 
                     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
-                                $this->pushToDependencies($dependencies, (string)getNameOfNode($node));
+                                $this->pushToDependencies($dependencies, (string) getNameOfNode($node));
                                 break;
                             case $node instanceof Node\Expr\StaticCall:
                                 // MyClass::Call
-                                $this->pushToDependencies($dependencies, (string)getNameOfNode($node));
+                                $this->pushToDependencies($dependencies, (string) getNameOfNode($node));
                                 break;
                         }
                     });
@@ -119,13 +119,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));
                                 }
                             }
                         }
@@ -152,6 +152,6 @@ 
         if ('self' === $lowercase || 'parent' === $lowercase) {
             return;
         }
-        array_push($dependencies, (string)$dependency);
+        array_push($dependencies, (string) $dependency);
     }
 }

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

Spacing +2 added lines, -2 removed lines

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

phpmetrics/src/Hal/Metric/Package/PackageDependencies.php

Spacing +3 added lines, -3 removed lines

@@ -13,7 +13,7 @@ 
     /** @return void */
     public function calculate(Metrics $metrics)
     {
-        $classes = array_filter($metrics->all(), function (Metric $metric) {
+        $classes = array_filter($metrics->all(), function(Metric $metric) {
             return $metric instanceof ClassMetric || $metric instanceof InterfaceMetric;
         });
 
@@ -30,7 +30,7 @@ 
      */
     private function increaseDependencies(Metric $class, Metrics $metrics)
     {
-        if (! $class->has('package') || ! $class->has('externals')) {
+        if (!$class->has('package') || !$class->has('externals')) {
             return;
         }
         /** @var PackageMetric $incomingPackage */
@@ -66,6 +66,6 @@ 
         }
         $parts = explode('\\', $className);
         array_pop($parts);
-        return implode('\\', $parts) . '\\';
+        return implode('\\', $parts).'\\';
     }
 }

phpmetrics/src/Hal/Metric/Package/PackageAbstraction.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     public function calculate(Metrics $metrics)
     {
         foreach ($metrics->all() as $eachPackage) {
-            if (! $eachPackage instanceof PackageMetric) {
+            if (!$eachPackage instanceof PackageMetric) {
                 continue;
             }
             $abstractClassCount = 0;

phpmetrics/src/Hal/Metric/Package/PackageCollectingVisitor.php

Spacing +6 added lines, -6 removed lines

@@ -28,7 +28,7 @@ 
     public function enterNode(Node $node)
     {
         if ($node instanceof Namespace_) {
-            $this->namespace = (string)$node->name;
+            $this->namespace = (string) $node->name;
         }
 
         return null;
@@ -45,17 +45,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);
 
             $metric = $this->metrics->get($elementName);

phpmetrics/src/Hal/Metric/Package/PackageInstability.php

Spacing +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
     public function calculate(Metrics $metrics)
     {
         /* @var $packages PackageMetric[] */
-        $packages = array_filter($metrics->all(), function ($metric) {
+        $packages = array_filter($metrics->all(), function($metric) {
             return $metric instanceof PackageMetric;
         });
 
@@ -30,7 +30,7 @@ 
         }
         // Set depending instabilities
         foreach ($packages as $eachPackage) {
-            $dependentInstabilities = array_map(function ($packageName) use ($instabilitiesByPackage) {
+            $dependentInstabilities = array_map(function($packageName) use ($instabilitiesByPackage) {
                 return isset($instabilitiesByPackage[$packageName]) ? $instabilitiesByPackage[$packageName] : null;
             }, $eachPackage->getOutgoingPackageDependencies());
 

phpmetrics/src/Hal/Component/File/Finder.php

Spacing +4 added lines, -4 removed lines

@@ -57,7 +57,7 @@ 
         $this->excludedDirs = $excludedDirs;
         $this->flags = $flags;
         if ($flags === -1) {
-            $this->flags = FilesystemIterator::KEY_AS_PATHNAME | FilesystemIterator::CURRENT_AS_FILEINFO;
+            $this->flags = FilesystemIterator::KEY_AS_PATHNAME|FilesystemIterator::CURRENT_AS_FILEINFO;
         }
     }
 
@@ -72,15 +72,15 @@ 
         $files = [];
         foreach ($paths as $path) {
             if (is_dir($path)) {
-                $path = rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR;
+                $path = rtrim($path, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR;
                 $directory = new RecursiveDirectoryIterator($path, $this->flags);
                 $iterator = new RecursiveIteratorIterator($directory);
 
                 $filterRegex = sprintf(
                     '`^%s%s%s$`',
                     preg_quote($path, '`'),
-                    !empty($this->excludedDirs) ? '((?!' . implode('|', array_map('preg_quote', $this->excludedDirs)) . ').)+' : '.+',
-                    '\.(' . implode('|', $this->extensions) . ')'
+                    !empty($this->excludedDirs) ? '((?!'.implode('|', array_map('preg_quote', $this->excludedDirs)).').)+' : '.+',
+                    '\.('.implode('|', $this->extensions).')'
                 );
 
                 $filteredIterator = new RegexIterator(