phpmetrics / PhpMetrics

src/Hal/Component/Token/TokenType.php

Spacing +9 added lines, -9 removed lines

@@ -104,7 +104,7 @@ 
      * @var array
      */
     private $operatorsStrings = array(
-        ';', '*', '/', '%', '-','+'
+        ';', '*', '/', '%', '-', '+'
     , '!', '!=', '%', '%=', '&', '&&', '||', '&=', '(', ')'
         /*, '{', '}'*/, '[', ']', '*', '*=', '+', '++', '+=', ','
     , '-', '--', '-=->', '.', '...', '/', '/=', ':', '::'
@@ -120,7 +120,7 @@ 
      */
     private $byPass = array(
         '{', '}' // in PHP, these case are counted with T_IF, '(', ...
-    , ')' , '('   // we count only the first (
+    , ')', '('   // we count only the first (
     , ','
     );
 
@@ -128,7 +128,7 @@ 
      * Constructor
      */
     public function __construct() {
-        if(version_compare(PHP_VERSION, '5.4.0') >= 0) {
+        if (version_compare(PHP_VERSION, '5.4.0') >= 0) {
             $this->operators = array_merge($this->operators, array(
                 T_INSTEADOF
             , T_TRAIT_C
@@ -141,9 +141,9 @@ 
 
         // performance : key will be equals to it value
         $this->operands = array_combine($this->operands, $this->operands);
-        $this->operators= array_combine($this->operators, $this->operators);
+        $this->operators = array_combine($this->operators, $this->operators);
         $this->operatorsStrings = array_combine($this->operatorsStrings, $this->operatorsStrings);
-        $this->byPass= array_combine($this->byPass, $this->byPass);
+        $this->byPass = array_combine($this->byPass, $this->byPass);
     }
 
     /**
@@ -154,8 +154,8 @@ 
      */
     public function isOperand(Token $token)
     {
-        if(T_STRING == $token->getType()) {
-            return !(isset($this->byPass[$token->getValue()]) ||isset($this->operatorsStrings[$token->getValue()]));
+        if (T_STRING == $token->getType()) {
+            return !(isset($this->byPass[$token->getValue()]) || isset($this->operatorsStrings[$token->getValue()]));
         }
 
         return isset($this->operands[$token->getType()]);
@@ -169,8 +169,8 @@ 
      */
     public function isOperator(Token $token)
     {
-        if(T_STRING == $token->getType()) {
-            if(isset($this->byPass[$token->getValue()])) {
+        if (T_STRING == $token->getType()) {
+            if (isset($this->byPass[$token->getValue()])) {
                 return false;
             }
             return isset($this->operatorsStrings[$token->getValue()]);

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

Spacing +3 added lines, -3 removed lines

@@ -56,9 +56,9 @@ 
         $tokens = $this->tokenizer->tokenize($filename);
 
         $ccn = 1; // default path
-        foreach($tokens as $token) {
+        foreach ($tokens as $token) {
 
-            switch($token->getType()) {
+            switch ($token->getType()) {
                 case T_IF:
                 case T_ELSEIF:
                 case T_FOREACH:
@@ -77,7 +77,7 @@ 
                     $ccn++;
                     break;
                 case T_STRING:
-                    if('?' == $token->getValue()) {
+                    if ('?' == $token->getValue()) {
                         $ccn = $ccn + 2;
                     }
                     break;

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

Spacing +2 added lines, -2 removed lines

@@ -63,8 +63,8 @@
             , T_BOOLEAN_OR => T_BOOLEAN_OR
         , T_LOGICAL_OR => T_LOGICAL_OR
         );
-        foreach($tokens as $token) {
-            if(isset($logicalOperators[$token->getType()])) {
+        foreach ($tokens as $token) {
+            if (isset($logicalOperators[$token->getType()])) {
                 $L++;
             }
         }

src/Hal/Metrics/Complexity/Structural/CardAndAgresti/FileSystemComplexity.php

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
         $len = 0;
         $sy = $dc = $sc = array();
         $calculator = new SystemComplexity();
-        foreach($rOOP->getClasses() as $class) {
+        foreach ($rOOP->getClasses() as $class) {
             $r = $calculator->calculate($class);
 
             $len += sizeof($class->getMethods(), COUNT_NORMAL);
@@ -58,7 +58,7 @@ 
             $sc[] = $r->getTotalStructuralComplexity();
         }
 
-        if($len > 0 &&sizeof($dc, COUNT_NORMAL) > 0) {
+        if ($len > 0 && sizeof($dc, COUNT_NORMAL) > 0) {
             $result
                 ->setRelativeStructuralComplexity(round(array_sum($sc) / $len, 2))
                 ->setRelativeDataComplexity(round(array_sum($dc) / $len, 2))

src/Hal/Metrics/Complexity/Structural/CardAndAgresti/SystemComplexity.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
 
         $sy = $dc = $sc = array(); // in system
-        foreach($class->getMethods() as $method) {
+        foreach ($class->getMethods() as $method) {
             $fanout = sizeof($method->getDependencies(), COUNT_NORMAL);
             $v = sizeof($method->getArguments(), COUNT_NORMAL) + sizeof($method->getReturns(), COUNT_NORMAL);
 

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

Spacing +7 added lines, -7 removed lines

@@ -29,8 +29,8 @@ 
         $map = $this->extractCoupling($result);
 
         // instability
-        foreach($map as &$class) {
-            if($class->getAfferentCoupling() + $class->getEfferentCoupling() > 0) {
+        foreach ($map as &$class) {
+            if ($class->getAfferentCoupling() + $class->getEfferentCoupling() > 0) {
                 $class->setInstability($class->getEfferentCoupling() / ($class->getAfferentCoupling() + $class->getEfferentCoupling()));
             }
         }
@@ -49,22 +49,22 @@ 
         $results = $result->all();
 
         $classes = array();
-        foreach($results as $result) {
+        foreach ($results as $result) {
             $classes = array_merge($classes, $result->getClasses());
         }
 
         $map = array();
-        foreach($classes as $class) {
+        foreach ($classes as $class) {
 
-            if(!isset($map[$class->getFullname()])) {
+            if (!isset($map[$class->getFullname()])) {
                 $map[$class->getFullname()] = new Result($class->getFullname());
             }
 
             $dependencies = $class->getDependencies();
             $map[$class->getFullname()]->setEfferentCoupling(sizeof($dependencies, COUNT_NORMAL));
 
-            foreach($dependencies as $dependency) {
-                if(!isset($map[$dependency])) {
+            foreach ($dependencies as $dependency) {
+                if (!isset($map[$dependency])) {
                     $map[$dependency] = new Result($dependency);
                 }
                 $map[$dependency]->setAfferentCoupling($map[$dependency]->getAfferentCoupling() + 1);

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

Spacing +3 added lines, -3 removed lines

@@ -54,21 +54,21 @@
         $result = new Result($filename);
 
         // case of file doesn't contain any class
-        if(null === $rOOP) {
+        if (null === $rOOP) {
             return $result;
         }
 
         $instability = $ce = $ca = 0;
 
         $classes = $rOOP->getClasses();
-        foreach($classes as $declaredClass) {
+        foreach ($classes as $declaredClass) {
             $declaredClassCoupling = $this->couplingMap->get($declaredClass->getFullname());
 
             $ce += $declaredClassCoupling->getEfferentCoupling();
             $ca += $declaredClassCoupling->getAfferentCoupling();
         }
 
-        if($ca + $ce > 0) {
+        if ($ca + $ce > 0) {
             $instability = $ce / ($ca + $ce);
         }
 

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

Spacing +2 added lines, -2 removed lines

@@ -50,9 +50,9 @@
         $n = 0;
         $lcom = new LackOfCohesionOfMethods();
 
-        foreach($rOOP->getClasses() as $class) {
+        foreach ($rOOP->getClasses() as $class) {
 
-            if($class instanceof ReflectedInterface) {
+            if ($class instanceof ReflectedInterface) {
                 continue;
             }
 

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

Spacing +15 added lines, -15 removed lines

@@ -34,13 +34,13 @@ 
         $methodsToInspect = $class->getMethods();
 
         $toSkip = array();
-        foreach($methodsToInspect as $method) {
+        foreach ($methodsToInspect as $method) {
 
-            if(in_array($method->getName(), $toSkip)) {
+            if (in_array($method->getName(), $toSkip)) {
                 continue;
             }
 
-            if($method->isSetter() ||$method->isGetter()) {
+            if ($method->isSetter() || $method->isGetter()) {
                 continue;
             }
 
@@ -78,8 +78,8 @@ 
 
         // foreach directly linked methods, recurs
         $methods = $class->getMethods();
-        foreach($linked as $link) {
-            if(!isset( $methods[$link])) {
+        foreach ($linked as $link) {
+            if (!isset($methods[$link])) {
                 continue;
             }
             $linked = array_merge($linked, $this->getLinkedMethods($class, $methods[$link], $toSkip));
@@ -99,18 +99,18 @@ 
         $linked = array();
 
         // A calls B
-        if(preg_match_all('!\\$this\\->(\w*?)\\(!im', $method->getContent(), $matches)) {
+        if (preg_match_all('!\\$this\\->(\w*?)\\(!im', $method->getContent(), $matches)) {
             list(, $linked) = $matches;
         }
 
         // B calls A
-        foreach($class->getMethods() as $otherMethod) {
+        foreach ($class->getMethods() as $otherMethod) {
             $otherCalls = array();
-            if(preg_match_all('!\\$this\\->(\w*?)\\(!im', $otherMethod->getContent(), $matches)) {
+            if (preg_match_all('!\\$this\\->(\w*?)\\(!im', $otherMethod->getContent(), $matches)) {
                 list(, $otherCalls) = $matches;
             }
 
-            if(in_array($method->getName(), $otherCalls)) {
+            if (in_array($method->getName(), $otherCalls)) {
                 array_push($linked, $otherMethod->getName());
             }
         }
@@ -128,29 +128,29 @@ 
 
         $linked = array();
         $members = array();
-        if(preg_match_all('!\\$this\\->([\w\\(]+)!im', $method->getContent(), $matches)) {
+        if (preg_match_all('!\\$this\\->([\w\\(]+)!im', $method->getContent(), $matches)) {
             list(, $members) = $matches;
         }
 
 
         // search in other methods if they share attribute
-        foreach($class->getMethods() as $otherMethod) {
+        foreach ($class->getMethods() as $otherMethod) {
             $otherMembers = array();
-            if(preg_match_all('!\\$this\\->([\w\\(]+)!im', $otherMethod->getContent(), $matches)) {
+            if (preg_match_all('!\\$this\\->([\w\\(]+)!im', $otherMethod->getContent(), $matches)) {
                 list(, $otherMembers) = $matches;
             }
 
             $intersect = array_intersect($members, $otherMembers);
 
             // remove calls (members and calls are mixed : regex is too complex to be read)
-            foreach($intersect as $k => $name) {
-                if(preg_match('!\($!', $name)) {
+            foreach ($intersect as $k => $name) {
+                if (preg_match('!\($!', $name)) {
                     unset($intersect[$k]);
                 }
             }
 
 
-            if(sizeof($intersect, COUNT_NORMAL) > 0) {
+            if (sizeof($intersect, COUNT_NORMAL) > 0) {
 
                 array_push($linked, $otherMethod->getName());
             }