phpmetrics / PhpMetrics

src/Hal/Component/Reflected/Klass.php

Spacing +3 added lines, -3 removed lines

@@ -67,11 +67,11 @@ 
     public function getDependencies($unique = true)
     {
         $dependencies = array();
-        foreach($this->methods as $method) {
+        foreach ($this->methods as $method) {
             $dependencies = array_merge($dependencies, $method->getDependencies());
         }
 
-        if($unique) {
+        if ($unique) {
             $dependencies = array_unique($dependencies);
         }
 
@@ -169,7 +169,7 @@ 
      */
     public function setIsAbstract($isAbstract)
     {
-        $this->isAbstract = (bool)$isAbstract;
+        $this->isAbstract = (bool) $isAbstract;
         return $this;
     }
 

src/Hal/Component/Reflected/Method.php

Spacing +6 added lines, -6 removed lines

@@ -63,19 +63,19 @@ 
         $typeResolver = new TypeResolver();
 
         $dependencies = array();
-        foreach($this->returns as $return) {
-            if($typeResolver->isObject($return->getType())) {
+        foreach ($this->returns as $return) {
+            if ($typeResolver->isObject($return->getType())) {
                 array_push($dependencies, $return->getType());
             }
         }
-        foreach($this->arguments as $argument) {
+        foreach ($this->arguments as $argument) {
             array_push($dependencies, $argument->getType());
         }
-        foreach($this->calls as $call) {
+        foreach ($this->calls as $call) {
             array_push($dependencies, $call->getType());
         }
 
-        if($unique) {
+        if ($unique) {
             $dependencies = array_unique($dependencies);
         }
 
@@ -165,7 +165,7 @@ 
      */
     public function setIsStatic($isStatic)
     {
-        $this->isStatic = (bool)$isStatic;
+        $this->isStatic = (bool) $isStatic;
         return $this;
     }
 

src/Hal/Metrics/Mood/Abstractness/Abstractness.php

Spacing +3 added lines, -3 removed lines

@@ -29,9 +29,9 @@ 
 
         $ac = $cc = $abstractness = 0;
 
-        foreach($graph->all() as $node) {
+        foreach ($graph->all() as $node) {
             $class = $node->getData();
-            if (($class->isAbstract() ||$class->isInterface())) {
+            if (($class->isAbstract() || $class->isInterface())) {
                 $ac++;
             } else {
                 $cc++;
@@ -39,7 +39,7 @@ 
         }
 
         $result = new Result;
-        if($ac + $cc > 0) {
+        if ($ac + $cc > 0) {
             $abstractness = round($ac / ($ac + $cc), 2);
         }
         $result->setAbstractness($abstractness);

src/Hal/Metrics/Mood/Instability/Instability.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
         $ca = $ce = $i = 0;
 
         $coupling = new Coupling();
-        foreach($graph->all() as $node) {
+        foreach ($graph->all() as $node) {
 
             $r = $coupling->calculate($node);
             $ce += $r->getEfferentCoupling();
@@ -39,7 +39,7 @@ 
         }
 
         $result = new Result;
-        if($ca + $ce > 0) {
+        if ($ca + $ce > 0) {
             $i = round($ce / ($ca + $ce), 2);
         }
         $result->setInstability($i);

src/Hal/Metrics/Complexity/Structural/HenryAndKafura/Coupling.php

Spacing +4 added lines, -4 removed lines

@@ -28,14 +28,14 @@ 
     public function calculate(Node $node)
     {
         $efferent = $afferent = 0;
-        foreach($node->getEdges() as $edge) {
+        foreach ($node->getEdges() as $edge) {
 
-            if($edge->getFrom()->getKey() == $node->getKey()) {
+            if ($edge->getFrom()->getKey() == $node->getKey()) {
                 // affects
                 $afferent++;
             }
 
-            if($edge->getTo()->getKey() == $node->getKey()) {
+            if ($edge->getTo()->getKey() == $node->getKey()) {
                 // receive effects
                 $efferent++;
             }
@@ -46,7 +46,7 @@ 
             ->setAfferentCoupling($afferent)
             ->setEfferentCoupling($efferent);
 
-        if($efferent + $afferent > 0) {
+        if ($efferent + $afferent > 0) {
             $result->setInstability($efferent / ($afferent + $efferent));
         }
         return $result;

src/Hal/Metrics/Complexity/Structural/LCOM/LackOfCohesionOfMethods.php

Spacing +13 added lines, -13 removed lines

@@ -30,27 +30,27 @@ 
         $graph = new Graph();
 
         // attributes in graph are prefixed with '_attr_' string
-        foreach($class->getMethods() as $method) {
+        foreach ($class->getMethods() as $method) {
 
             // avoid getters and setters
-            if($method->isGetter() ||$method->isSetter()) {
+            if ($method->isGetter() || $method->isSetter()) {
                 continue;
             }
 
-            if(null === ($from = $graph->get($method->getName()))) {
+            if (null === ($from = $graph->get($method->getName()))) {
                 $from = new Node($method->getName());
                 $graph->insert($from);
             }
 
             // calls
-            foreach($method->getCalls() as $call) {
+            foreach ($method->getCalls() as $call) {
 
-                if(!$call->isItself()) {
+                if (!$call->isItself()) {
                     continue;
 
                 }
 
-                if(null === ($to = $graph->get($call->getMethodName()))) {
+                if (null === ($to = $graph->get($call->getMethodName()))) {
                     $to = new Node($call->getMethodName());
                     $graph->insert($to);
                 }
@@ -58,12 +58,12 @@ 
             }
 
             // attributes
-            foreach($method->getTokens() as $token) {
-                if(preg_match('!\$this\->(\w+)$!', $token, $matches)) {
+            foreach ($method->getTokens() as $token) {
+                if (preg_match('!\$this\->(\w+)$!', $token, $matches)) {
                     list(, $attribute) = $matches;
 
-                    if(null === ($to = $graph->get('_attr_' . $attribute))) {
-                        $to = new Node('_attr_' . $attribute);
+                    if (null === ($to = $graph->get('_attr_'.$attribute))) {
+                        $to = new Node('_attr_'.$attribute);
                         $graph->insert($to);
                     }
                     $graph->addEdge($from, $to);
@@ -74,7 +74,7 @@ 
 
         // iterate over nodes, and count paths
         $paths = 0;
-        foreach($graph->all() as $node) {
+        foreach ($graph->all() as $node) {
             $paths += $this->traverse($node);
         }
 
@@ -93,12 +93,12 @@ 
      */
     private function traverse(Node $node)
     {
-        if($node->visited) {
+        if ($node->visited) {
             return 0;
         }
         $node->visited = true;
 
-        foreach($node->getAdjacents() as $adjacent) {
+        foreach ($node->getAdjacents() as $adjacent) {
             $this->traverse($adjacent);
         }
 

src/Hal/Metrics/Complexity/Text/Length/Loc.php

Spacing +4 added lines, -4 removed lines

@@ -31,9 +31,9 @@ 
 
         // count and remove multi lines comments
         $cloc = 0;
-        if(preg_match_all('!/\*.*?\*/!s', $code, $matches)) {
-            foreach($matches[0] as $match) {
-                $cloc +=  max(1, sizeof(preg_split('/\r\n|\r|\n/', $match)));
+        if (preg_match_all('!/\*.*?\*/!s', $code, $matches)) {
+            foreach ($matches[0] as $match) {
+                $cloc += max(1, sizeof(preg_split('/\r\n|\r|\n/', $match)));
             }
         }
         $code = preg_replace('!/\*.*?\*/!s', '', $code);
@@ -44,7 +44,7 @@ 
 
         // count and remove empty lines
         $code = trim(preg_replace('!(^\s*[\r\n])!sm', '', $code));
-        $lloc = sizeof(preg_split('/\r\n|\r|\n/', $code)) ;
+        $lloc = sizeof(preg_split('/\r\n|\r|\n/', $code));
 
         $result = new Result;
         $result

src/Hal/Metrics/Complexity/Text/Halstead/Halstead.php

Spacing +12 added lines, -12 removed lines

@@ -66,11 +66,11 @@ 
     {
         $this->operators = $this->operands = array();
 
-        foreach($tokens as $token) {
-            if($this->tokenType->isOperator($token)) {
+        foreach ($tokens as $token) {
+            if ($this->tokenType->isOperator($token)) {
                 $this->operators[] = $token;
             }
-            else if($this->tokenType->isOperand($token)) {
+            else if ($this->tokenType->isOperand($token)) {
                 $this->operands[] = $token;
             }
         }
@@ -93,23 +93,23 @@ 
         $this->inventory($class->getTokens());
         $result = new Result;
 
-        $uniqueOperators = array_map( 'unserialize', array_unique( array_map( 'serialize', $this->operators ) ) );
-        $uniqueOperands = array_map( 'unserialize', array_unique( array_map( 'serialize', $this->operands ) ) );
+        $uniqueOperators = array_map('unserialize', array_unique(array_map('serialize', $this->operators)));
+        $uniqueOperands = array_map('unserialize', array_unique(array_map('serialize', $this->operands)));
 
         $n1 = sizeof($uniqueOperators, COUNT_NORMAL);
         $n2 = sizeof($uniqueOperands, COUNT_NORMAL);
         $N1 = sizeof($this->operators, COUNT_NORMAL);
         $N2 = sizeof($this->operands, COUNT_NORMAL);
 
-        if(($n2 == 0)||($N2 == 0)) {
+        if (($n2 == 0) || ($N2 == 0)) {
             // files without operators
             $V = $n1 = $n2 = $N1 = $N2 = $E = $D = $B = $T = $I = $L = 0;
         } else {
             $devAbility = 3000;
             $N = $N1 + $N2;
             $n = $n1 + $n2;
-            $V = $N * log($n ,2);
-            $L = (2 / max(1,$n1)) * ($n2 / $N2);
+            $V = $N * log($n, 2);
+            $L = (2 / max(1, $n1)) * ($n2 / $N2);
             $D = ($n1 / 2) * ($N2 / $n2);
             $E = $V * $D;
             $B = $V / $devAbility;
@@ -120,13 +120,13 @@ 
         $result
             ->setLength($N1 + $N2)
             ->setVocabulary($n1 + $n2)
-            ->setVolume(round($V,2))
-            ->setDifficulty(round($D,2))
-            ->setEffort(round($E,2))
+            ->setVolume(round($V, 2))
+            ->setDifficulty(round($D, 2))
+            ->setEffort(round($E, 2))
             ->setLevel(round($L, 2))
             ->setBugs(round($B, 2))
             ->setTime(round($T))
-            ->setIntelligentContent(round($I,2))
+            ->setIntelligentContent(round($I, 2))
             ->setNumberOfOperators($N1)
             ->setNumberOfOperands($N2)
             ->setNumberOfUniqueOperators($n1)

src/Hal/Metrics/Complexity/Component/McCabe/McCabe.php

Spacing +2 added lines, -2 removed lines

@@ -40,9 +40,9 @@
         $info = new Result;
 
         $ccn = 1; // default path
-        foreach($class->getTokens() as $token) {
+        foreach ($class->getTokens() as $token) {
 
-            switch($token) {
+            switch ($token) {
                 case Token::T_IF:
                 case Token::T_ELSEIF:
                 case Token::T_FOREACH: