phpmetrics / PhpMetrics

src/Hal/Metric/Class_/Complexity/CyclomaticComplexityVisitor.php

Spacing +3 added lines, -3 removed lines

@@ -96,7 +96,7 @@ 
             // `case ...:` from a `switch`. Ignore `default:`.
             'Stmt_Case' => static fn (Stmt\Case_ $node): int => (null !== $node->cond) ? 1 : 0,
             // `... => ...` from a `match`. Ignore `default =>`.
-            'MatchArm' => static fn (Node\MatchArm $node): int => count((array)$node->conds)
+            'MatchArm' => static fn (Node\MatchArm $node) : int => count((array) $node->conds)
         ];
     }
 
@@ -132,7 +132,7 @@ 
         // Apply the CCN of the method for each method found in the class.
         /** @var array<FunctionMetric> $classMethods */
         $classMethods = $class->get('methods'); // $class->get('methods') is defined in ClassEnumVisitor.
-        array_map(static function (FunctionMetric $method) use ($allMethods): void {
+        array_map(static function(FunctionMetric $method) use ($allMethods): void {
             $methodName = $method->getName();
             $method->set('ccn', $allMethods[$methodName]['ccn']);
             $method->set('isAccessor', $allMethods[$methodName]['isAccessor']);
@@ -179,7 +179,7 @@ 
     private function discoverMethods(array $stmts): array
     {
         $methodsNames = array_map(static fn (Stmt\ClassMethod $stmt): string => $stmt->name->toString(), $stmts);
-        $methodsData = array_map(function (Stmt\ClassMethod $stmt): array {
+        $methodsData = array_map(function(Stmt\ClassMethod $stmt): array {
             $isAccessor = in_array($this->roleOfMethodDetector->detects($stmt), ['getter', 'setter'], true);
             return [
                 // Each method by default is CCN 1 even if it's empty. Ignoring accessors and give them the default CCN.

src/Hal/Metric/Class_/Structural/SystemComplexityVisitor.php

Spacing +2 added lines, -2 removed lines

@@ -69,11 +69,11 @@
         $this->structuralComplexityByMethod = [];
         $this->localComplexityByMethod = [];
 
-        array_map(function (Stmt\ClassMethod $stmt): void {
+        array_map(function(Stmt\ClassMethod $stmt): void {
             $io = count($stmt->getParams());
             $fanOut = 0;
 
-            $getBasicCardAndAgrestiMetrics = static function (Node $node) use (&$io, &$fanOut): void {
+            $getBasicCardAndAgrestiMetrics = static function(Node $node) use (&$io, &$fanOut): void {
                 switch (true) {
                     case $node instanceof Stmt\Return_:
                         $io += (null !== $node->expr);

src/Hal/Metric/Class_/Structural/LcomVisitor.php

Spacing +12 added lines, -12 removed lines

@@ -95,7 +95,7 @@ 
         $class = $this->metrics->get(MetricNameGenerator::getClassName($node));
 
         // We don't want to increase the LCOM for getters and setters.
-        $methods = array_filter($node->getMethods(), function (Stmt\ClassMethod $stmt): bool {
+        $methods = array_filter($node->getMethods(), function(Stmt\ClassMethod $stmt): bool {
             return !in_array($this->roleOfMethodDetector->detects($stmt), ['getter', 'setter'], true);
         });
         array_map($this->buildGraph(...), $methods);
@@ -118,7 +118,7 @@ 
      */
     private function buildGraph(Stmt\ClassMethod $stmt): void
     {
-        $from = $this->graph->gather($stmt->name . '()');
+        $from = $this->graph->gather($stmt->name.'()');
         $this->nodeIterator->iterateOver($stmt, $this->getVisitorCallback($from));
     }
 
@@ -130,14 +130,14 @@ 
      */
     private function getVisitorCallback(TreeNode $from): Closure
     {
-        return function (Node $node) use ($from): void {
+        return function(Node $node) use ($from): void {
             $relations = [
                 ...$this->getPromotedConstructorsTreeNodes($node),
                 ...$this->getPropertyFetchTreeNodes($node),
                 ...$this->getMethodCallTreeNodes($node),
             ];
 
-            array_map(function (TreeNode $treeNode) use ($from): void {
+            array_map(function(TreeNode $treeNode) use ($from): void {
                 $this->graph->addEdge($from, $treeNode);
             }, $relations);
         };
@@ -168,17 +168,17 @@ 
      */
     private function getPromotedConstructorsTreeNodes(Node $node): array
     {
-        if (!$node instanceof Stmt\ClassMethod || '__construct' !== (string)$node->name) {
+        if (!$node instanceof Stmt\ClassMethod || '__construct' !== (string) $node->name) {
             return [];
         }
 
         // Promoted properties in constructors are defined by the presence of "public", "protected" or "private" flag.
         // Since PHP 8.1, the presence of "readonly" flag also promotes a property, with a "public" default visibility.
-        $promotions = array_filter($node->getParams(), static function (Node\Param $param): bool {
-            $mask = (Modifiers::VISIBILITY_MASK | Modifiers::READONLY);
-            return (bool)($param->flags & $mask);
+        $promotions = array_filter($node->getParams(), static function(Node\Param $param): bool {
+            $mask = (Modifiers::VISIBILITY_MASK|Modifiers::READONLY);
+            return (bool) ($param->flags&$mask);
         });
-        return array_map(function (Node\Param $param): TreeNode {
+        return array_map(function(Node\Param $param): TreeNode {
             /** @var Node\Expr\Variable $var Can't be an error as promoted properties are correctly parsed. */
             $var = $param->var;
             /** @var string $name Is mandatory to be a string as promoted properties cannot have expression as name. */
@@ -216,7 +216,7 @@ 
             && !$node->var instanceof Node\Expr\New_ // Prevents `(new X())->methodCall()` issues.
             && null !== ($nodeName = $this->readMemberNodeClearName($node))
         ) {
-            return [$this->graph->gather($nodeName . '()')];
+            return [$this->graph->gather($nodeName.'()')];
         }
         return [];
     }
@@ -246,10 +246,10 @@ 
         if (
             property_exists($node->var, 'name')
             && !($node->var->name instanceof Node\Expr\Variable)
-            && 'this' === (string)$node->var->name
+            && 'this' === (string) $node->var->name
             && ($nodeName instanceof Stringable || is_string($nodeName))
         ) {
-            return (string)$nodeName;
+            return (string) $nodeName;
         }
         return null;
     }

src/Hal/Metric/Class_/ClassEnumVisitor.php

Spacing +3 added lines, -3 removed lines

@@ -68,13 +68,13 @@
             $class->set('final', $node instanceof Stmt\Class_ && $node->isFinal());
         }
 
-        $dataMethods = (object)[
+        $dataMethods = (object) [
             'nbPublic' => 0,
             'nbPrivate' => 0,
         ];
-        $functionMetrics = array_map(function (Stmt\ClassMethod $stmt) use ($dataMethods): FunctionMetric {
+        $functionMetrics = array_map(function(Stmt\ClassMethod $stmt) use ($dataMethods): FunctionMetric {
             /** @var FunctionMetric $function */
-            $function = $this->metrics->get((string)$stmt->name);
+            $function = $this->metrics->get((string) $stmt->name);
             $isPublic = $stmt->isPublic();
 
             $dataMethods->nbPublic += $isPublic;

src/Hal/Metric/Class_/Text/HalsteadVisitor.php

Spacing +2 added lines, -2 removed lines

@@ -87,7 +87,7 @@ 
      */
     private static function getUniqueOperatorsAndOperands(): array
     {
-        $uniq = static function (array $elementsToDedupe): array {
+        $uniq = static function(array $elementsToDedupe): array {
             return array_map(unserialize(...), array_unique(array_map(serialize(...), $elementsToDedupe)));
         };
 
@@ -184,7 +184,7 @@ 
      */
     private function getVisitorCallback(): Closure
     {
-        return static function (Node $node): void {
+        return static function(Node $node): void {
             if ($node instanceof Node\Param && $node->var instanceof Node\Expr\Variable) {
                 return;
             }

src/Hal/Metric/Class_/Text/LengthVisitor.php

Spacing +2 added lines, -2 removed lines

@@ -74,7 +74,7 @@ 
         // Count and remove multi lines comments.
         $cloc = 0;
         preg_match_all('!/\*.*?\*/!s', $code, $matches);
-        array_map(function (string $commentedCode) use (&$cloc): void {
+        array_map(function(string $commentedCode) use (&$cloc): void {
             $cloc += max(1, $this->countSplitLines($commentedCode));
         }, $matches[0]);
         /** @var string $code subject and replacement are string, so the return remains a string. */
@@ -84,7 +84,7 @@ 
         /** @var string $code subject and replacement are string, so the return remains a string. */
         $code = preg_replace_callback(
             '!(\'[^\']*\'|"[^"]*")|((?:#[^\[]|//).*$)!m',
-            static function (array $matches) use (&$cloc): string {
+            static function(array $matches) use (&$cloc): string {
                 [, $logicalCode, $commentedCode] = array_pad($matches, 3, null);
                 $cloc += (null !== $commentedCode);
                 /** @var string */

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

Spacing +9 added lines, -9 removed lines

@@ -114,7 +114,7 @@ 
      */
     private function addDependency(string|Stringable $dependency): void
     {
-        $dependency = (string)$dependency;
+        $dependency = (string) $dependency;
         if (in_array(strtolower($dependency), ['self', 'parent'], true)) {
             return;
         }
@@ -129,7 +129,7 @@ 
      */
     private function addParent(string|Stringable $parent): void
     {
-        $parent = (string)$parent;
+        $parent = (string) $parent;
         $this->parents[] = ltrim($parent, '\\');
         // Parenting is a dependency.
         $this->addDependency($parent);
@@ -141,7 +141,7 @@ 
      */
     private function addImplementation(string|Stringable $implements): void
     {
-        $implements = (string)$implements;
+        $implements = (string) $implements;
         $this->interfaces[] = ltrim($implements, '\\');
         // Implementation is a dependency.
         $this->addDependency($implements);
@@ -158,7 +158,7 @@ 
         $this->addDependenciesFromTypeHint($stmt->returnType);
 
         // Type hint of method's arguments
-        array_map(function (Node\Param $param): void {
+        array_map(function(Node\Param $param): void {
             $this->addDependenciesFromTypeHint($param->type);
         }, $stmt->params);
 
@@ -168,7 +168,7 @@ 
         // - MyClass::$staticProperty;
         // - MyClass::class;
         // - instanceof MyClass;
-        $this->nodeIterator->iterateOver($stmt, function (Node $node): void {
+        $this->nodeIterator->iterateOver($stmt, function(Node $node): void {
             if ($node instanceof Node\Expr && property_exists($node, 'class') && $node->class instanceof Node\Name) {
                 $this->addDependency($node->class);
             }
@@ -211,7 +211,7 @@ 
         $comments = $node->getDocComment();
         /** @var scalar $reformattedTest */
         $reformattedTest = $comments?->getReformattedText();
-        preg_match_all('!\s+\*\s+@([\w\\\\]+)!', (string)$reformattedTest, $matches);
+        preg_match_all('!\s+\*\s+@([\w\\\\]+)!', (string) $reformattedTest, $matches);
         $annotations = $matches[1] ?? [];
         array_map($this->addDependency(...), $annotations);
     }
@@ -232,12 +232,12 @@ 
 
         // Otherwise, check with the defined `use` statements.
         foreach ($this->uses as $use) {
-            $useAlias = (string)$use->getAlias();
+            $useAlias = (string) $use->getAlias();
             if ($useAlias === $classNameToResolve && !str_contains($classNameToResolve, '\\')) {
-                return (string)$use->name;
+                return (string) $use->name;
             }
             if ($useAlias === strstr($classNameToResolve, '\\', true)) {
-                return $use->name . strstr($classNameToResolve, '\\');
+                return $use->name.strstr($classNameToResolve, '\\');
             }
         }
 

src/Hal/Metric/Class_/Component/MaintainabilityIndexVisitor.php

Spacing +3 added lines, -3 removed lines

@@ -77,13 +77,13 @@
         $volume = $class->get('volume');
 
         if (null === $lloc || null === $cloc || null === $loc) {
-            throw new LogicException('Please enable ' . LengthVisitor::class . ' visitor first');
+            throw new LogicException('Please enable '.LengthVisitor::class.' visitor first');
         }
         if (null === $ccn) {
-            throw new LogicException('Please enable ' . CyclomaticComplexityVisitor::class . ' visitor first');
+            throw new LogicException('Please enable '.CyclomaticComplexityVisitor::class.' visitor first');
         }
         if (null === $volume) {
-            throw new LogicException('Please enable ' . HalsteadVisitor::class . ' visitor first');
+            throw new LogicException('Please enable '.HalsteadVisitor::class.' visitor first');
         }
 
         // Maintainability index without comment.

src/Hal/Metric/Group/Group.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     public function reduceMetrics(Metrics $metrics): Metrics
     {
         $matched = new Metrics();
-        array_map(function (Metric $metric) use ($matched): void {
+        array_map(function(Metric $metric) use ($matched): void {
             if (1 !== preg_match($this->getRegex(), $metric->getName())) {
                 return;
             }