phpmetrics / PhpMetrics

src/Hal/Violation/Class_/TooComplexClassCode.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 
     public function apply(Metric $metric)
     {
-        if (! $metric instanceof ClassMetric) {
+        if (!$metric instanceof ClassMetric) {
             return;
         }
 

src/Hal/Component/Ast/NodeTraverser.php

Doc Comments +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
 
     /**
      * @param array $nodes
-     * @param array|NodeVisitor[] $visitors
+     * @param NodeVisitor[] $visitors
      * @return array
      */
     public function traverseArray(array $nodes, array $visitors) {

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
 
     public function __construct(Mother $traverser, $stopCondition = null)
     {
-        if(null === $stopCondition) {
+        if (null === $stopCondition) {
             $stopCondition = function($node) {
                 if ($node instanceof Node\Stmt\Class_ || $node instanceof Node\Stmt\Interface_) {
                     return false;
@@ -96,7 +96,7 @@ 
 }
 
 if (PHP_VERSION_ID >= 70000) {
-    class_alias(Php7NodeTraverser::class, __NAMESPACE__ . '\\NodeTraverser');
+    class_alias(Php7NodeTraverser::class, __NAMESPACE__.'\\NodeTraverser');
 } else {
-    class_alias(Php5NodeTraverser::class, __NAMESPACE__ . '\\NodeTraverser');
+    class_alias(Php5NodeTraverser::class, __NAMESPACE__.'\\NodeTraverser');
 }

src/Hal/Metric/Consolidated.php

Spacing +3 added lines, -3 removed lines

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

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

Doc Comments +3 added lines

@@ -136,6 +136,9 @@
         }
     }
 
+    /**
+     * @param string|null $dependency
+     */
     private function pushToDependencies(array &$dependencies, $dependency)
     {
         $lowercase = strtolower($dependency);

Spacing +11 added lines, -11 removed lines

@@ -62,20 +62,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);
                 }
             }
 
@@ -84,7 +84,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);
                         }
                     }
 
@@ -92,13 +92,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
@@ -118,12 +118,12 @@ 
                             foreach ($this->uses as $use) {
                                 if (method_exists($use, 'getAlias')) {
                                     if (((string) $use->getAlias()) === $check) {
-                                        $this->pushToDependencies($dependencies, (string)($use->name));
+                                        $this->pushToDependencies($dependencies, (string) ($use->name));
                                     }
                                     continue;
                                 }
                                 if ($use->alias === $check) {
-                                    $this->pushToDependencies($dependencies, (string)($use->name));
+                                    $this->pushToDependencies($dependencies, (string) ($use->name));
                                 }
                             }
                         }