MetricsHandlerFactory::getFactories() - Code Metrics - Inspection of "refactor: leverage Symfony DI when possible" - bpolaszek/mercure-php-hub - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#21)

by BENOIT

created 2021-01-03 15:28 UTC

MetricsHandlerFactory::getFactories() A

↳ Parent: MetricsHandlerFactory

Complexity

Conditions	4
Paths	4

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	4.128

Importance

Changes

Metric	Value
cc	4
eloc	8
c	0
b	0
f	0
nc	4
nop	0
dl	0
loc	14
rs	10
ccs	4
cts	5
cp	0.8
crap	4.128

<?php

namespace BenTools\MercurePHP\Metrics;

use React\Promise\PromiseInterface;

final class MetricsHandlerFactory implements MetricsHandlerFactoryInterface
{
    /**
     * @var MetricsHandlerFactoryInterface[]
     */
    private iterable $factories;

    public function __construct(iterable $factories)
    {
        $this->factories = $factories;

    }

    public function supports(string $dsn): bool
    {
        foreach ($this->getFactories() as $factory) {
            if ($factory->supports($dsn)) {
                return true;
            }
        }

        return false;
    }

    public function create(string $dsn): PromiseInterface
    {
        foreach ($this->getFactories() as $factory) {
            if (!$factory->supports($dsn)) {
                continue;
            }

            return $factory->create($dsn);
        }

        throw new \RuntimeException(\sprintf('Invalid metrics handler DSN %s', $dsn));
    }

    private function getFactories(): array
    {
        if (\is_array($this->factories)) {
            return $this->factories;

        }

        $factories = [];
        foreach ($this->factories as $factory) {
            if ($factory === $this) {
                continue;
            }
            $factories[] = $factory;
        }
        return $this->factories = (fn(MetricsHandlerFactoryInterface ...$factories) => $factories)(...$factories);
    }
}


1		<?php
2
3		namespace BenTools\MercurePHP\Metrics;
4
5		use React\Promise\PromiseInterface;
6
7		final class MetricsHandlerFactory implements MetricsHandlerFactoryInterface
8		{
9		/**
10		* @var MetricsHandlerFactoryInterface[]
11		*/
12		private iterable $factories;
13
14		public function __construct(iterable $factories)
15		{
16		$this->factories = $factories;
		0 ignored issues – show Documentation Bug introduced 2021-01-03 15:30 UTC by Report Bug Copy Issue Report It seems like `$factories` of type `iterable` is incompatible with the declared type `BenTools\MercurePHP\Metr...ndlerFactoryInterface[]` of property `$factories`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
17		}
18	2
19		public function supports(string $dsn): bool
20	2	{
21	2	foreach ($this->getFactories() as $factory) {
22		if ($factory->supports($dsn)) {
23	1	return true;
24		}
25	1	}
26	1
27	1	return false;
28		}
29
30		public function create(string $dsn): PromiseInterface
31		{
32		foreach ($this->getFactories() as $factory) {
33		if (!$factory->supports($dsn)) {
34	1	continue;
35		}
36	1
37	1	return $factory->create($dsn);
38		}
39
40		throw new \RuntimeException(\sprintf('Invalid metrics handler DSN %s', $dsn));
41	1	}
42
43		private function getFactories(): array
44		{
45		if (\is_array($this->factories)) {
46		return $this->factories;
		0 ignored issues – show Bug Best Practice introduced 2021-01-03 15:30 UTC by Report Bug Copy Issue Report The expression `return $this->factories` returns the type `iterable` which is incompatible with the type-hinted return `array`. Loading history...
47	2	}
48
49	2	$factories = [];
50		foreach ($this->factories as $factory) {
51	2	if ($factory === $this) {
52	2	continue;
53		}
54		$factories[] = $factory;
55		}
56		return $this->factories = (fn(MetricsHandlerFactoryInterface ...$factories) => $factories)(...$factories);
57		}
58		}
59

bpolaszek / mercure-php-hub

Pull Request — master (#21)

MetricsHandlerFactory::getFactories() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like