TransportFactory::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

TransportFactory::getFactories() A

↳ Parent: TransportFactory

Complexity

Conditions	4
Paths	4

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

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	0
cts	5
cp	0
crap	20

<?php

namespace BenTools\MercurePHP\Transport;

use React\Promise\PromiseInterface;

final class TransportFactory implements TransportFactoryInterface
{
    /**
     * @var TransportFactoryInterface[]
     */
    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 transport 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(TransportFactoryInterface ...$factories) => $factories)(...$factories);
    }
}


1		<?php
2
3		namespace BenTools\MercurePHP\Transport;
4
5		use React\Promise\PromiseInterface;
6
7		final class TransportFactory implements TransportFactoryInterface
8		{
9		/**
10		* @var TransportFactoryInterface[]
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\Tran...sportFactoryInterface[]` 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	1
19		public function supports(string $dsn): bool
20	1	{
21	1	foreach ($this->getFactories() as $factory) {
22		if ($factory->supports($dsn)) {
23	1	return true;
24		}
25	1	}
26
27		return false;
28		}
29
30		public function create(string $dsn): PromiseInterface
31	1	{
32		foreach ($this->getFactories() as $factory) {
33		if (!$factory->supports($dsn)) {
34		continue;
35		}
36
37		return $factory->create($dsn);
38		}
39
40		throw new \RuntimeException(\sprintf('Invalid transport DSN %s', $dsn));
41		}
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		}
48
49		$factories = [];
50		foreach ($this->factories as $factory) {
51		if ($factory === $this) {
52		continue;
53		}
54		$factories[] = $factory;
55		}
56		return $this->factories = (fn(TransportFactoryInterface ...$factories) => $factories)(...$factories);
57		}
58		}
59

bpolaszek / mercure-php-hub

Pull Request — master (#21)

TransportFactory::getFactories() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like