pixelfederation/swoole-bundle

View on GitHub
src/Bridge/Symfony/Container/Proxy/Generation/ContextualAccessForwarderGenerator.php

Summary

Maintainability
A
1 hr
Test Coverage

Method generate has 32 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    public function generate(\ReflectionClass $originalClass, ClassGenerator $classGenerator): void
    {
        CanProxyAssertion::assertClassCanBeProxied($originalClass);

        $interfaces = [

    The class ContextualAccessForwarderGenerator has a coupling between objects value of 17. Consider to reduce the number of dependencies under 13.
    Open

    class ContextualAccessForwarderGenerator implements ProxyGeneratorInterface
    {
        public function __construct(private MethodForwarderBuilder $forwarderBuilder)
        {
        }

    CouplingBetweenObjects

    Since: 1.1.0

    A class with too many dependencies has negative impacts on several quality aspects of a class. This includes quality criteria like stability, maintainability and understandability

    Example

    class Foo {
        /**
         * @var \foo\bar\X
         */
        private $x = null;
    
        /**
         * @var \foo\bar\Y
         */
        private $y = null;
    
        /**
         * @var \foo\bar\Z
         */
        private $z = null;
    
        public function setFoo(\Foo $foo) {}
        public function setBar(\Bar $bar) {}
        public function setBaz(\Baz $baz) {}
    
        /**
         * @return \SplObjectStorage
         * @throws \OutOfRangeException
         * @throws \InvalidArgumentException
         * @throws \ErrorException
         */
        public function process(\Iterator $it) {}
    
        // ...
    }

    Source https://phpmd.org/rules/design.html#couplingbetweenobjects

    There are no issues that match your filters.

    Category
    Status