phpmetrics / PhpMetrics

src/Hal/Component/OOP/Extractor/MethodExtractor.php

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 use Hal\Component\OOP\Reflected\MethodUsage;
 use Hal\Component\OOP\Reflected\ReflectedArgument;
 use Hal\Component\OOP\Reflected\ReflectedClass;
-use Hal\Component\OOP\Reflected\ReflectedClass\ReflectedAnonymousClass;
 use Hal\Component\OOP\Reflected\ReflectedMethod;
 use Hal\Component\OOP\Reflected\ReflectedReturn;
 use Hal\Component\OOP\Resolver\TypeResolver;

Spacing +32 added lines, -32 removed lines

@@ -55,7 +55,7 @@ 
         $start = $n;
 
         $declaration = $this->searcher->getUnder(array(')'), $n, $tokens);
-        if(!preg_match('!function\s+(.*)\(\s*(.*)!is', $declaration, $matches)) {
+        if (!preg_match('!function\s+(.*)\(\s*(.*)!is', $declaration, $matches)) {
             throw new \Exception(sprintf("Closure detected instead of method\nDetails:\n%s", $declaration));
         }
         list(, $name, $args) = $matches;
@@ -68,21 +68,21 @@ 
         $this->extractState($method, $p = $start, $tokens); // please keep "p = start"
 
         $arguments = preg_split('!\s*,\s*!m', $args);
-        foreach($arguments as $argDecl) {
+        foreach ($arguments as $argDecl) {
 
-            if(0 == strlen($argDecl)) {
+            if (0 == strlen($argDecl)) {
                 continue;
             }
 
             $elems = preg_split('!([\s=]+)!', $argDecl);
             $isRequired = 2 == sizeof($elems, COUNT_NORMAL);
 
-            if(sizeof($elems, COUNT_NORMAL) == 1) {
+            if (sizeof($elems, COUNT_NORMAL) == 1) {
                 list($name, $type) = array_pad($elems, 2, null);
             } else {
-                if('$' == $elems[0][0]) {
+                if ('$' == $elems[0][0]) {
                     $name = $elems[0];
-                    $type  = null;
+                    $type = null;
                     $isRequired = false;
                 } else {
                     list($type, $name) = array_pad($elems, 2, null);
@@ -96,9 +96,9 @@ 
 
 
         // does method has body ? (example: interface ; abstract classes)
-        $p = $n  + 1;
+        $p = $n + 1;
         $underComma = trim($this->searcher->getUnder(array(';'), $p, $tokens));
-        if(strlen($underComma) > 0) {
+        if (strlen($underComma) > 0) {
             //
             // Body
             $this->extractContent($method, $n, $tokens);
@@ -108,7 +108,7 @@ 
 
             // Tokens
             $end = $this->searcher->getPositionOfClosingBrace($n, $tokens);
-            if($end > 0) {
+            if ($end > 0) {
                 $method->setTokens($tokens->extract($n, $end));
             }
         } else {
@@ -138,7 +138,7 @@ 
      * @return $this
      */
     public function extractVisibility(ReflectedMethod $method, $n, TokenCollection $tokens) {
-        switch(true) {
+        switch (true) {
             case $this->searcher->isPrecededBy(T_PRIVATE, $n, $tokens, 4):
                 $visibility = ReflectedMethod::VISIBILITY_PRIVATE;
                 break;
@@ -163,7 +163,7 @@ 
      * @return $this
      */
     public function extractState(ReflectedMethod $method, $n, TokenCollection $tokens) {
-        if($this->searcher->isPrecededBy(T_STATIC, $n, $tokens, 4)) {
+        if ($this->searcher->isPrecededBy(T_STATIC, $n, $tokens, 4)) {
             $method->setState(ReflectedMethod::STATE_STATIC);
         }
         return $this;
@@ -179,7 +179,7 @@ 
      */
     private function extractContent(ReflectedMethod $method, $n, TokenCollection $tokens) {
         $end = $this->searcher->getPositionOfClosingBrace($n, $tokens);
-        if($end > 0) {
+        if ($end > 0) {
             $collection = $tokens->extract($n, $end);
             $method->setContent($collection->asString());
         }
@@ -202,13 +202,13 @@ 
         $extractor = new CallExtractor($this->searcher, $currentClass);
         $start = $n;
         $len = sizeof($tokens, COUNT_NORMAL);
-        for($i = $start; $i < $len; $i++) {
+        for ($i = $start; $i < $len; $i++) {
             $token = $tokens[$i];
-            switch($token->getType()) {
+            switch ($token->getType()) {
                 case T_PAAMAYIM_NEKUDOTAYIM:
                 case T_NEW:
                     $call = $extractor->extract($i, $tokens, $currentClass);
-                    if($call !== 'class') { // anonymous class
+                    if ($call !== 'class') { // anonymous class
                         $method->pushDependency($call);
                         $method->pushInstanciedClass($call);
                     }
@@ -219,9 +219,9 @@ 
         //
         // Parameters in Method API
         $resolver = new TypeResolver();
-        foreach($method->getArguments() as $argument) {
+        foreach ($method->getArguments() as $argument) {
             $name = $argument->getType();
-            if(strlen($name) > 0 && !$resolver->isNative($name)) {
+            if (strlen($name) > 0 && !$resolver->isNative($name)) {
                 $method->pushDependency($name);
             }
         }
@@ -240,10 +240,10 @@ 
     private function extractCalls(ReflectedMethod $method, $n, TokenCollection $tokens) {
 
         // $this->foo(), $c->foo()
-        if(preg_match_all('!(\$[\w]*)\-\>(\w*?)\(!', $method->getContent(), $matches, PREG_SET_ORDER)) {
-            foreach($matches as $m) {
+        if (preg_match_all('!(\$[\w]*)\-\>(\w*?)\(!', $method->getContent(), $matches, PREG_SET_ORDER)) {
+            foreach ($matches as $m) {
                 $function = $m[2];
-                if('$this' == $m[1]) {
+                if ('$this' == $m[1]) {
                     $method->pushInternalCall($function);
                 } else {
                     $method->pushExternalCall($m[1], $function);
@@ -251,8 +251,8 @@ 
             }
         }
         // (new X)->foo()
-        if(preg_match_all('!\(new (\w+?).*?\)\->(\w+)\(!', $method->getContent(), $matches, PREG_SET_ORDER)) {
-            foreach($matches as $m) {
+        if (preg_match_all('!\(new (\w+?).*?\)\->(\w+)\(!', $method->getContent(), $matches, PREG_SET_ORDER)) {
+            foreach ($matches as $m) {
                 $method->pushExternalCall($m[1], $m[2]);
             }
         }
@@ -272,9 +272,9 @@ 
         // we cannot use specific token. The ":" delimiter is a T_STRING token
         // in regex we match ":" followed by any character except another ":"
         $following = $this->searcher->getUnder(array('{', ';'), $n, $tokens);
-        if(preg_match('@(?<!:):\s*(\w+)@', $following, $matches)) {
+        if (preg_match('@(?<!:):\s*(\w+)@', $following, $matches)) {
             $type = trim($matches[1]);
-            if(empty($type)) {
+            if (empty($type)) {
                 return $this;
             }
             $return = new ReflectedReturn($type, ReflectedReturn::VALUE_UNKNOW, ReflectedReturn::STRICT_TYPE_HINT);
@@ -283,8 +283,8 @@ 
         }
 
         // array of available values based on code
-        if(preg_match_all('!([\s;]return\s|^return\s+)(.*?);!', $method->getContent(), $matches)) {
-            foreach($matches[2] as $m) {
+        if (preg_match_all('!([\s;]return\s|^return\s+)(.*?);!', $method->getContent(), $matches)) {
+            foreach ($matches[2] as $m) {
                 $value = trim($m, ";\t\n\r\0\x0B");
                 $return = new ReflectedReturn($resolver->resolve($m), $value, ReflectedReturn::ESTIMATED_TYPE_HINT);
                 $method->pushReturn($return);
@@ -302,21 +302,21 @@ 
     private function extractUsage(ReflectedMethod $method) {
         $tokens = $method->getTokens();
         $codes = $values = array();
-        foreach($tokens as $token) {
-            if(in_array($token->getType(), array(T_WHITESPACE, T_BOOL_CAST, T_INT_CAST, T_STRING_CAST, T_DOUBLE_CAST, T_OBJECT_CAST))) {
+        foreach ($tokens as $token) {
+            if (in_array($token->getType(), array(T_WHITESPACE, T_BOOL_CAST, T_INT_CAST, T_STRING_CAST, T_DOUBLE_CAST, T_OBJECT_CAST))) {
                 continue;
             }
             array_push($codes, $token->getType());
             array_push($values, $token->getValue());
         }
-        switch(true) {
-            case preg_match('!^(get)|(is)|(has).*!',$method->getName()) && $codes == array(T_RETURN, T_VARIABLE, T_OBJECT_OPERATOR, T_STRING, T_STRING):
+        switch (true) {
+            case preg_match('!^(get)|(is)|(has).*!', $method->getName()) && $codes == array(T_RETURN, T_VARIABLE, T_OBJECT_OPERATOR, T_STRING, T_STRING):
                 $method->setUsage(MethodUsage::USAGE_GETTER);
                 break;
             // basic setter
-            case preg_match('!^set.*!',$method->getName()) && $codes == array(T_VARIABLE, T_OBJECT_OPERATOR,T_STRING,T_STRING, T_VARIABLE, T_STRING) && $values[3] == '=':
+            case preg_match('!^set.*!', $method->getName()) && $codes == array(T_VARIABLE, T_OBJECT_OPERATOR, T_STRING, T_STRING, T_VARIABLE, T_STRING) && $values[3] == '=':
             // fluent setter
-            case preg_match('!^set.*!',$method->getName()) && $codes == array(T_VARIABLE, T_OBJECT_OPERATOR,T_STRING,T_STRING, T_VARIABLE, T_STRING, T_RETURN, T_VARIABLE, T_STRING)
+            case preg_match('!^set.*!', $method->getName()) && $codes == array(T_VARIABLE, T_OBJECT_OPERATOR, T_STRING, T_STRING, T_VARIABLE, T_STRING, T_RETURN, T_VARIABLE, T_STRING)
                 && $values[3] == '=' && $values[7] == '$this':
                 $method->setUsage(MethodUsage::USAGE_SETTER);
                 break;

compatibility.php

Spacing +2 added lines, -2 removed lines

@@ -1,4 +1,4 @@
 <?php
 // PHP 7
-defined('T_SPACESHIP') ||define('T_SPACESHIP', '<=>'); // 289
-defined('T_COALESCE') ||define('T_COALESCE', '??'); // 282
\ No newline at end of file
+defined('T_SPACESHIP') || define('T_SPACESHIP', '<=>'); // 289
+defined('T_COALESCE') || define('T_COALESCE', '??'); // 282
\ No newline at end of file

src/Hal/Application/Command/Job/Analyze/FileAnalyzer.php

Spacing +1 added lines, -1 removed lines

@@ -134,7 +134,7 @@
             ->setHalstead($rHalstead)
             ->setMaintainabilityIndex($rMaintainability);
 
-        if($this->withOOP) {
+        if ($this->withOOP) {
             $rOOP = $this->extractor->extract($filename);
             $this->classMap->push($filename, $rOOP);
             $resultSet->setOOP($rOOP);

src/Hal/Application/Command/Job/Analyze/LcomAnalyzer.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
     public function execute(array $files) {
         $fileCoupling = new FileLackOfCohesionOfMethods($this->classMap);
-        foreach($files as $filename) {
+        foreach ($files as $filename) {
             $result = $fileCoupling->calculate($filename);
             $this->collection->get($filename)->setLcom($result);
         }

src/Hal/Application/Command/Job/DoAggregatedAnalyze.php

Spacing +4 added lines, -4 removed lines

@@ -46,7 +46,7 @@ 
      */
     public function __construct(OutputInterface $output, Aggregator $aggregator)
     {
-        $this->aggregator= $aggregator;
+        $this->aggregator = $aggregator;
         $this->output = $output;
     }
 
@@ -66,12 +66,12 @@ 
         $bounds = new Bounds();
         $instability = new Instability();
 
-        foreach($groupedResults as $namespace => $results) {
+        foreach ($groupedResults as $namespace => $results) {
 
             // we filter aggregates to conserve only direct results
             $childs = new ResultCollection();
-            foreach($results as $r) {
-                if($namespace === dirname($r->getName())) {
+            foreach ($results as $r) {
+                if ($namespace === dirname($r->getName())) {
                     $childs->push($r);
                 }
             }

src/Hal/Application/Command/Job/DoAnalyze.php

Spacing +5 added lines, -5 removed lines

@@ -92,7 +92,7 @@ 
 
         $files = $this->finder->find($this->path);
 
-        if(0 == sizeof($files, COUNT_NORMAL)) {
+        if (0 == sizeof($files, COUNT_NORMAL)) {
             throw new \LogicException('No file found');
         }
 
@@ -116,12 +116,12 @@ 
             , $classMap
         );
 
-        foreach($files as $k => $filename) {
+        foreach ($files as $k => $filename) {
 
             $progress->advance();
 
             // Integrity
-            if(!$this->ignoreErrors && !$syntaxChecker->isCorrect($filename)) {
+            if (!$this->ignoreErrors && !$syntaxChecker->isCorrect($filename)) {
                 $this->output->writeln(sprintf('<error>file %s is not valid and has been skipped</error>', $filename));
                 unset($files[$k]);
                 continue;
@@ -130,7 +130,7 @@ 
             // Analyze
             try {
                 $resultSet = $fileAnalyzer->execute($filename);
-            } catch(NoTokenizableException $e) {
+            } catch (NoTokenizableException $e) {
                 $this->output->writeln(sprintf("<error>file %s has been skipped: \n%s</error>", $filename, $e->getMessage()));
                 unset($files[$k]);
                 continue;
@@ -151,7 +151,7 @@ 
         $progress->finish();
 
 
-        if($this->withOOP) {
+        if ($this->withOOP) {
             // COUPLING (should be done after parsing files)
             $this->output->write(str_pad("\x0DAnalyzing coupling. This will take few minutes...", 80, "\x20"));
             $couplingAnalyzer = new CouplingAnalyzer($classMap, $collection);

src/Hal/Application/Command/Job/Queue.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
      * @inheritdoc
      */
     public function execute(ResultCollection $collection, ResultCollection $aggregatedResults) {
-        foreach($this->jobs as $job) {
+        foreach ($this->jobs as $job) {
             $job->execute($collection, $aggregatedResults);
         }
     }

src/Hal/Application/Command/Job/ReportRenderer.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
      * @inheritdoc
      */
     public function execute(ResultCollection $collection, ResultCollection $aggregatedResults) {
-        if(!$this->enabled) {
+        if (!$this->enabled) {
             return;
         }
 

src/Hal/Application/Command/Job/ReportWriter.php

Spacing +2 added lines, -2 removed lines

@@ -61,12 +61,12 @@
      * @inheritdoc
      */
     public function execute(ResultCollection $collection, ResultCollection $aggregatedResults) {
-        if(!$this->destination) {
+        if (!$this->destination) {
             return;
         }
 
         $dir = dirname($this->destination);
-        if(!file_exists($dir)) {
+        if (!file_exists($dir)) {
             mkdir($dir, 0777, true);
         }
         $this->output->writeln(sprintf('Generating %s Report...', $this->formater->getName()));

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 use Hal\Application\Formater\FormaterInterface;
 use Hal\Component\Result\ResultCollection;
 use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Output\StreamOutput;
 
 
 /**