Similar blocks of code found in 2 locations. Consider refactoring.

Open

<?php

declare(strict_types=1);

namespace K911\Swoole\Bridge\Symfony\Bundle\Command;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

<?php

declare(strict_types=1);

namespace K911\Swoole\Bridge\Symfony\Bundle\Command;

• Read up
  Read up

HttpServerConfiguration has 34 functions (exceeds 20 allowed). Consider refactoring.

Open

class HttpServerConfiguration
{
    private const SWOOLE_HTTP_SERVER_CONFIG_DAEMONIZE = 'daemonize';
    private const SWOOLE_HTTP_SERVER_CONFIG_SERVE_STATIC = 'serve_static';
    private const SWOOLE_HTTP_SERVER_CONFIG_REACTOR_COUNT = 'reactor_count';

File ContainerModifier.php has 320 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php

declare(strict_types=1);

namespace K911\Swoole\Bridge\Symfony\Container;

File HttpServerConfiguration.php has 306 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php

declare(strict_types=1);

namespace K911\Swoole\Server;

The class HttpServerConfiguration has an overall complexity of 67 which is very high. The configured complexity threshold is 50.

Open

class HttpServerConfiguration
{
    private const SWOOLE_HTTP_SERVER_CONFIG_DAEMONIZE = 'daemonize';
    private const SWOOLE_HTTP_SERVER_CONFIG_SERVE_STATIC = 'serve_static';
    private const SWOOLE_HTTP_SERVER_CONFIG_REACTOR_COUNT = 'reactor_count';

• Exclude checks
  • Disable engine
  • Disable check

Method validateSetting has 59 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function validateSetting(string $key, mixed $value): void
    {
        Assertion::keyExists(self::SWOOLE_HTTP_SERVER_CONFIGURATION, $key, 'There is no configuration mapping for setting "%s".');

        switch ($key) {

Method generateOverriddenDoInExtension has 57 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private static function generateOverriddenDoInExtension(
        Filesystem $fs,
        string $containerDir,
        string $fileToLoad,
        string $class,

Method overrideGeneratedContainer has 49 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private static function overrideGeneratedContainer(ReflectionClass $reflContainer, string $cacheDir, bool $isDebug): void
    {
        $fs = new Filesystem();
        $containerFqcn = $reflContainer->getName();
        $overriddenFqcn = $containerFqcn.'_Overridden';

Function process has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring.

Open

    public function process(ContainerBuilder $container, Proxifier $proxifier): void
    {
        $resetterDefId = 'swoole_bundle.coroutines_support.cache_adapter_resetter';
        $resetterDef = new Definition(SimpleResetter::class);
        $resetterDef->setArguments(['reset']);

• Read up
  Read up

Method newInstance has 44 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function newInstance(
        object $instance,
        array $factoryConfigs,
        int $globalInstancesLimit
    ): object {

Avoid excessively long class names like BlockingProxyFactoryOverridingManagerConfigurator. Keep class name length under 40.

Open

final class BlockingProxyFactoryOverridingManagerConfigurator
{
    private static ?\ReflectionProperty $emProxyFactoryPropRefl = null;

    public function __construct(

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ATooLongClassNameThatHintsAtADesignProblem {

}

interface ATooLongInterfaceNameThatHintsAtADesignProblem {

}

Function validateSetting has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    private function validateSetting(string $key, mixed $value): void
    {
        Assertion::keyExists(self::SWOOLE_HTTP_SERVER_CONFIGURATION, $key, 'There is no configuration mapping for setting "%s".');

        switch ($key) {

• Read up
  Read up

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 = [

Method generateLazyGetter has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private static function generateLazyGetter(string $methodName, array $internals): string
    {
        $sharedCheck = PHP_EOL;

        if (!empty($internals)) {

Avoid using undefined variables such as '$runtimeConfiguration' which will lead to PHP notices.

Open

            $runtimeConfiguration['trustedProxies'] = \array_filter($runtimeConfiguration['trustedProxies'], fn (string $trustedProxy): bool => '*' !== $trustedProxy);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$runtimeConfiguration' which will lead to PHP notices.

Open

            $runtimeConfiguration['trustedProxies'] = \array_filter($runtimeConfiguration['trustedProxies'], fn (string $trustedProxy): bool => '*' !== $trustedProxy);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$runtimeConfiguration' which will lead to PHP notices.

Open

            $runtimeConfiguration['trustAllProxies'] = true;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$runtimeConfiguration' which will lead to PHP notices.

Open

        $runtimeConfiguration['trustedHosts'] = $this->decodeSet($trustedHosts);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$this' which will lead to PHP notices.

Open

    return \Closure::bind(fn &(string $propertyName) => $this->$propertyName, $obj, $scope ?? $obj)($propertyName);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}