phpmetrics / PhpMetrics

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

Spacing +9 added lines, -9 removed lines

@@ -31,7 +31,7 @@ 
     public function asArray() {
 
         $nom = 0;
-        foreach($this->getClasses() as $class) {
+        foreach ($this->getClasses() as $class) {
             $nom += sizeof($class->getMethods(), COUNT_NORMAL);
         }
 
@@ -71,8 +71,8 @@ 
      */
     public function getAbstractClasses() {
         $result = array();
-        foreach($this->getClasses() as $class) {
-            if($class->isAbstract() ||$class instanceof ReflectedInterface) {
+        foreach ($this->getClasses() as $class) {
+            if ($class->isAbstract() || $class instanceof ReflectedInterface) {
                 array_push($result, $class);
             }
         }
@@ -86,8 +86,8 @@ 
      */
     public function getAnonymousClasses() {
         $result = array();
-        foreach($this->getClasses() as $class) {
-            if($class instanceof ReflectedClass\ReflectedAnonymousClass) {
+        foreach ($this->getClasses() as $class) {
+            if ($class instanceof ReflectedClass\ReflectedAnonymousClass) {
                 array_push($result, $class);
             }
         }
@@ -101,8 +101,8 @@ 
      */
     public function getInterfaces() {
         $result = array();
-        foreach($this->getClasses() as $class) {
-            if($class instanceof ReflectedInterface) {
+        foreach ($this->getClasses() as $class) {
+            if ($class instanceof ReflectedInterface) {
                 array_push($result, $class);
             }
         }
@@ -116,8 +116,8 @@ 
      */
     public function getConcreteClasses() {
         $result = array();
-        foreach($this->getClasses() as $class) {
-            if(!$class->isAbstract() &&!$class instanceof ReflectedInterface  && !$class instanceof ReflectedClass\ReflectedAnonymousClass) {
+        foreach ($this->getClasses() as $class) {
+            if (!$class->isAbstract() && !$class instanceof ReflectedInterface && !$class instanceof ReflectedClass\ReflectedAnonymousClass) {
                 array_push($result, $class);
             }
         }

src/Hal/Component/OOP/Reflected/ReflectedClass.php

Spacing +5 added lines, -5 removed lines

@@ -136,11 +136,11 @@ 
     public function getDependencies()
     {
         $dependencies = array();
-        foreach($this->getMethods() as $method) {
+        foreach ($this->getMethods() as $method) {
             $dependencies = array_merge($dependencies, $method->getDependencies());
         }
-        foreach($dependencies as &$name) {
-            if(in_array($name, array('self', 'static'))) {
+        foreach ($dependencies as &$name) {
+            if (in_array($name, array('self', 'static'))) {
                 $name = $this->getFullname();
             }
             $name = $this->nameResolver->resolve($name, $this->namespace);
@@ -207,7 +207,7 @@ 
     public function getInterfaces()
     {
         $resolvedInterfaces = array();
-        foreach($this->interfaces as $interface) {
+        foreach ($this->interfaces as $interface) {
             array_push($resolvedInterfaces, $this->nameResolver->resolve($interface, $this->namespace));
         }
         return $resolvedInterfaces;
@@ -230,7 +230,7 @@ 
      */
     public function getAnonymousClasses() {
         $result = array();
-        foreach($this->methods as $method) {
+        foreach ($this->methods as $method) {
             $result = array_merge($result, $method->getAnonymousClasses());
         }
         return $result;

src/Hal/Component/OOP/Reflected/ReflectedMethod.php

Spacing +7 added lines, -7 removed lines

@@ -189,10 +189,10 @@ 
         // on read : compare with aliases. We cannot make it in pushDependency() => aliases aren't yet known
         $result = array();
         $resolver = new TypeResolver();
-        foreach($this->returns as $return) {
+        foreach ($this->returns as $return) {
             $type = $this->nameResolver->resolve($return->getType(), null);
 
-            if("\\" !== $type[0] &&!$resolver->isNative($type)) {
+            if ("\\" !== $type[0] && !$resolver->isNative($type)) {
                 $type = $this->namespace.'\\'.$type;
             }
 
@@ -233,21 +233,21 @@ 
     {
         // on read : compare with aliases. We cannot make it in pushDependency() => aliases aren't yet known
         $dependencies = array();
-        foreach($this->dependencies as $name) {
+        foreach ($this->dependencies as $name) {
             array_push($dependencies, $this->nameResolver->resolve($name, null));
         }
 
         // returned values
         $resolver = new TypeResolver();
-        foreach($this->returns as $return) {
+        foreach ($this->returns as $return) {
             $name = $return->getType();
-            if(!$resolver->isNative($name)) {
+            if (!$resolver->isNative($name)) {
                 array_push($dependencies, $this->nameResolver->resolve($name, null));
             }
         }
 
         // anonymous classes in method (inner class)
-        foreach($this->anonymousClasses as $c) {
+        foreach ($this->anonymousClasses as $c) {
             array_push($dependencies, $c->getParent());
             $dependencies = array_merge($dependencies, $c->getDependencies());
         }
@@ -379,7 +379,7 @@ 
     public function getInstanciedClasses()
     {
         $classes = array();
-        foreach($this->instanciedClasses as $name) {
+        foreach ($this->instanciedClasses as $name) {
             array_push($classes, $this->nameResolver->resolve($name, null));
         }
         return $classes;

Doc Comments +4 added lines, -4 removed lines

@@ -163,7 +163,7 @@ 
     }
 
     /**
-     * @return TokenCollection
+     * @return \Hal\Component\Token\TokenCollection
      */
     public function getTokens()
     {
@@ -365,7 +365,7 @@ 
     }
 
     /**
-     * @param $class
+     * @param string|null $class
      * @return $this
      */
     public function pushInstanciedClass($class) {
@@ -386,7 +386,7 @@ 
     }
 
     /**
-     * @param $call
+     * @param string $call
      * @return $this
      */
     public function pushInternalCall($call)
@@ -404,7 +404,7 @@ 
     }
 
     /**
-     * @param $call
+     * @param string $call
      * @return $this
      */
     public function pushExternalCall($call)

src/Hal/Component/OOP/Resolver/NameResolver.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
         }
 
         // anonymous class
-        if('class@anonymous' == $name ) {
+        if ('class@anonymous' == $name) {
             return $name;
         }
 

src/Hal/Component/OOP/Resolver/TypeResolver.php

Spacing +11 added lines, -11 removed lines

@@ -41,47 +41,47 @@
         $string = $cased = trim($string, ";\t\n\r\0\x0B");
         $string = strtolower($string);
 
-        if(strlen($string) == 0) {
+        if (strlen($string) == 0) {
             return self::TYPE_VOID;
         }
 
-        if(preg_match('!^\d+$!', $string)) {
+        if (preg_match('!^\d+$!', $string)) {
             return self::TYPE_INTEGER;
         }
 
-        if(preg_match('!^\d+\.\d+$!', $string)) {
+        if (preg_match('!^\d+\.\d+$!', $string)) {
             return self::TYPE_FLOAT;
         }
 
-        if('null' == $string) {
+        if ('null' == $string) {
             return self::TYPE_NULL;
         }
 
-        if(preg_match('!(^\[|^array\()!', $string)) {
+        if (preg_match('!(^\[|^array\()!', $string)) {
             return self::TYPE_ARRAY;
         }
 
-        if(preg_match('!^new\s+class\s+!', $string, $matches)) {
+        if (preg_match('!^new\s+class\s+!', $string, $matches)) {
             return self::TYPE_ANONYMOUS_CLASS;
         }
 
-        if(preg_match('!^(new\s+)(.*?)(\s*[\(;].*|$)!', $cased, $matches)) {
+        if (preg_match('!^(new\s+)(.*?)(\s*[\(;].*|$)!', $cased, $matches)) {
             return $matches[2];
         }
 
-        if(preg_match('!^\$this$!', $string, $matches)) {
+        if (preg_match('!^\$this$!', $string, $matches)) {
             return self::TYPE_FLUENT_INTERFACE;
         }
 
-        if(preg_match('!^(true|false)!', $string, $matches)) {
+        if (preg_match('!^(true|false)!', $string, $matches)) {
             return self::TYPE_BOOL;
         }
 
-        if(preg_match('!^function!', $string, $matches)) {
+        if (preg_match('!^function!', $string, $matches)) {
             return self::TYPE_CALLABLE;
         }
 
-        if(preg_match('!^["\']!', $string, $matches)) {
+        if (preg_match('!^["\']!', $string, $matches)) {
             return self::TYPE_STRING;
         }
 

src/Hal/Component/Phar/Updater.php

Spacing +10 added lines, -10 removed lines

@@ -46,7 +46,7 @@ 
      */
     public function updates($version = self::LATEST) {
         $tag = $version;
-        if(self::LATEST == $version) {
+        if (self::LATEST == $version) {
             $tag = 'master';
         }
 
@@ -59,11 +59,11 @@ 
         }
 
         $currentPharLocation = \Phar::running(false);
-        if(!file_exists($currentPharLocation) ||strlen($currentPharLocation) == 0) {
+        if (!file_exists($currentPharLocation) || strlen($currentPharLocation) == 0) {
             throw new \LogicException("You're not currently using Phar. If you have installed PhpMetrics with Composer, please updates it using Composer.");
         }
 
-        if(!is_writable($currentPharLocation)) {
+        if (!is_writable($currentPharLocation)) {
             throw new \RuntimeException(sprintf('%s is not writable', $currentPharLocation));
         }
 
@@ -74,7 +74,7 @@ 
         $content = file_get_contents($url, false, $ctx);
 
         // replacing file
-        if(!$content) {
+        if (!$content) {
             throw new \RuntimeException('Download failed');
         }
 
@@ -82,13 +82,13 @@ 
         $tmpfile = tempnam(sys_get_temp_dir(), 'phar');
         file_put_contents($tmpfile, $content);
         $output = shell_exec(sprintf('"%s" "%s" --version', PHP_BINARY, $tmpfile));
-        if(!preg_match('!(v\d+\.\d+\.\d+)!', $output, $matches)) {
+        if (!preg_match('!(v\d+\.\d+\.\d+)!', $output, $matches)) {
             throw new \RuntimeException('Phar is corrupted. Please retry');
         }
 
         // compare versions
         $downloadedVersion = $matches[1];
-        if(self::LATEST !==$version &&$downloadedVersion !== $version) {
+        if (self::LATEST !== $version && $downloadedVersion !== $version) {
             throw new \RuntimeException('Incorrect version. Please retry');
         }
 
@@ -112,12 +112,12 @@ 
         static $filesize = null;
         static $progress = null;
 
-        switch($notification_code) {
+        switch ($notification_code) {
             case STREAM_NOTIFY_AUTH_REQUIRED:
             case STREAM_NOTIFY_AUTH_RESULT:
                 break;
             case STREAM_NOTIFY_CONNECT:
-                if(!$progress) {
+                if (!$progress) {
                     $this->output->writeln(sprintf("<info>Downloading</info>"));
                 }
                 break;
@@ -131,7 +131,7 @@ 
             case STREAM_NOTIFY_PROGRESS:
                 if ($bytes_transferred > 0) {
                     if (!isset($filesize)) {
-                        $this->output->writeln(sprintf("<info>Unknown file size.. %2d kb done..</info>", $bytes_transferred/1024));
+                        $this->output->writeln(sprintf("<info>Unknown file size.. %2d kb done..</info>", $bytes_transferred / 1024));
                     } else {
                         $progress->setProgress($bytes_transferred / 100);
                     }
@@ -140,7 +140,7 @@ 
 
             case STREAM_NOTIFY_COMPLETED:
             case STREAM_NOTIFY_FAILURE:
-                if($progress) {
+                if ($progress) {
                     $progress->clear();
                     $progress->finish();
                 }

src/Hal/Component/Result/ResultCollection.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
      */
     public function asArray() {
         $array = array();
-        foreach($this->results as $result) {
+        foreach ($this->results as $result) {
             array_push($array, $result->asArray());
         }
         return $array;

src/Hal/Component/Score/ScoringInterface.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
  *
  * @author Jean-François Lépine <https://twitter.com/Halleck45>
  */
-interface ScoringInterface  {
+interface ScoringInterface {
 
     /**
      * Calculates score

src/Hal/Component/Token/Token.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
      */
     public function __construct($data)
     {
-        if(!is_array($data)) {
+        if (!is_array($data)) {
             $this->type = T_STRING;
             $this->value = $data;
         } else {
@@ -44,7 +44,7 @@ 
         }
 
         // reduce multiple spaces to one
-        if(T_WHITESPACE  === $this->type && preg_match('/^\s*$/', $this->value)) {
+        if (T_WHITESPACE === $this->type && preg_match('/^\s*$/', $this->value)) {
             $this->value = ' ';
         }
     }