Wowstack\Dbc\Command\InspectCommand

Complexity

Total Complexity

5

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	execute()	0	41	4
A	configure()	0	19	1

<?php

declare(strict_types=1);

namespace Wowstack\Dbc\Command;

use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle;
use Wowstack\Dbc\DBC;

/**
 * @codeCoverageIgnore
 */
class InspectCommand extends Command
{
    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        $this
            ->setName('dbc:inspect')
            ->setDescription('Inspect a DBC file.')
            ->setHelp('This command allows you to inspect any DBC file and receive basic information about it.')
            ;

        $this
            ->addArgument('file', InputArgument::REQUIRED, 'Path to the DBC file')
            ;

        $this
            ->addOption(
                'string-samples',
                null,
                InputOption::VALUE_REQUIRED,
                'Print selected number of string samples',
                10
            );
    }

    /**
     * {@inheritdoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $DBC = new DBC($input->getArgument('file'));

It seems like $input->getArgument('file') can also be of type null and string[]; however, parameter $path of Wowstack\Dbc\DBC::__construct() does only seem to accept string, maybe add an additional type check?

        $io = new SymfonyStyle($input, $output);
        $io->title('DBC Inspect');
        $io->section('Stats');
        $io->text([
            sprintf(
                'The %s file contains %u rows at %u bytes per column, split into %u fields.',
                $DBC->getName(),
                $DBC->getRecordCount(),
                $DBC->getRecordSize(),
                $DBC->getFieldCount()
            ),
            '',
        ]);

        if ($DBC->hasStrings()) {
            $string_block = $DBC->getStringBlock();

Variable "string_block" is not in valid camel caps format

            $io->section('Strings');
            $io->text([
                sprintf(
                    'The %s file contains %u strings.',
                    $DBC->getName(),
                    count($string_block)

Variable "string_block" is not in valid camel caps format

                ),
                '',
            ]);

            $string_samples = $input->getOption('string-samples');

Variable "string_samples" is not in valid camel caps format

            foreach ($string_block as $index => $string) {

Variable "string_block" is not in valid camel caps format

                $io->text([$index.': '.$string]);

Concat operator must be surrounded by a single space

                --$string_samples;

Variable "string_samples" is not in valid camel caps format

                if (0 === $string_samples) {

Variable "string_samples" is not in valid camel caps format

                    break;
                }
            }
        }
        $io->newLine();
    }
}