PHPSA\Command\PrintCFGCommand::execute()

Complexity

Conditions	16
Paths	288

Size

Total Lines

117

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	272

Importance

Changes

0

<?php
    protected function execute(InputInterface $input, OutputInterface $output)

This operation has 3600 execution paths which exceeds the configured maximum of 200.

    {
        $output->writeln('');

        if (extension_loaded('xdebug')) {
            /**
             * This will disable only showing stack traces on error conditions.
             */
            if (function_exists('xdebug_disable')) {
                xdebug_disable();
            }

            $output->writeln('<error>It is highly recommended to disable the XDebug extension before invoking this command.</error>');
        }

        /** @var Application $application */
        $application = $this->getApplication();
        $application->compiler = new Compiler();

        $configFile = $input->getOption('config-file') ?: '.phpsa.yml';
        $configDir = realpath($input->getArgument('path'));
        $application->configuration = $this->loadConfiguration($configFile, $configDir);

        $parser = $this->createParser($application);

$application is of type object<PHPSA\Application>, but the function expects a string.

        $output->writeln('Used config file: ' . $application->configuration->getPath());

        $em = EventManager::getInstance();

Avoid variables with short names like $em. Configured minimum length is 3.

        $context = new Context($output, $application, $em);

        $fileParser = new FileParser(
            $parser,
            $application->compiler
        );

        $path = $input->getArgument('path');
        if (is_dir($path)) {
            $directoryIterator = new RecursiveIteratorIterator(
                new RecursiveDirectoryIterator($path, FilesystemIterator::SKIP_DOTS)
            );
            $output->writeln('Scanning directory <info>' . $path . '</info>');

            $count = 0;

            /** @var SplFileInfo $file */
            foreach ($directoryIterator as $file) {
                if ($file->getExtension() !== 'php') {
                    continue;
                }

                $context->debug($file->getPathname());
                $count++;
            }

            $output->writeln("Found <info>{$count} files</info>");

            if ($count > 100) {
                $output->writeln('<comment>Caution: You are trying to scan a lot of files; this might be slow. For bigger libraries, consider setting up a dedicated platform or using ci.lowl.io.</comment>');
            }

            $output->writeln('');

            /** @var SplFileInfo $file */
            foreach ($directoryIterator as $file) {
                if ($file->getExtension() !== 'php') {
                    continue;
                }

                $fileParser->parserFile($file->getPathname(), $context);
            }
        } elseif (is_file($path)) {
            $fileParser->parserFile($path, $context);

It seems like $path defined by $input->getArgument('path') on line 78 can also be of type array<integer,string> or null; however, PHPSA\Definition\FileParser::parserFile() does only seem to accept string, maybe add an additional type check?

        }


        /**
         * Step 2 Recursive check ...
         */
        $application->compiler->compile($context);

        $traverser = new BlockTraverser();
        //$traverser->addVisitor(new Visitor\DebugTextVisitor());
        $traverser->addVisitor(new Visitor\UnreachableVisitor());

        $printer = new \PHPSA\ControlFlow\Printer\DebugText();

        $functions = $application->compiler->getFunctions();
        foreach ($functions as $function) {
            $output->writeln('Function: ' . $function->getName());

            $cfg = $function->getCFG();
            if ($cfg) {
                $traverser->traverse($cfg);
                $printer->printGraph($cfg->getRoot());
            }
        }

        $classess = $application->compiler->getClasses();
        foreach ($classess as $class) {
            $output->writeln('Class: ' . $class->getName());

            $methods = $class->getMethods();

            foreach ($methods as $method) {
                $output->writeln('Method: ' . $method->getName());

                $cfg = $method->getCFG();
                if ($cfg) {
                    $traverser->traverse($cfg);
                    $printer->printGraph($cfg->getRoot());
                }
            }
        }

        $output->writeln('');
        $output->writeln('Memory usage: ' . $this->getMemoryUsage(false) . ' (peak: ' . $this->getMemoryUsage(true) . ') MB');
    }
}