Wowstack\Dbc\Command\ViewCommand::execute()

Complexity

Conditions	7
Paths	24

Size

Total Lines	59
Code Lines	40

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $DBC = new DBC($input->getArgument('file'), Mapping::fromYAML($input->getArgument('map')));

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

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);
        $table = new Table($output);

        $io->title('DBC Viewer');
        $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

                ),
                '',
            ]);
        }
        $io->newLine();

        $rows = $input->getOption('rows');
        $table->setHeaders($DBC->getMap()->getFieldNames());

        foreach ($DBC as $index => $record) {
            $table->addRow($record->read());

            if (false !== $rows) {
                --$rows;
                if (0 === $rows) {
                    break;
                }
            }
        }

        $table->render();
        $io->newLine();

        $errors = $DBC->getErrors();
        if (count($errors) > 0) {
            $io->section('Errors');
            foreach ($DBC->getErrors() as $error) {
                $io->text([
                    '#'.$error['record'].' ('.$error['type'].'/'.$error['field'].'): '.$error['hint'],

Concat operator must be surrounded by a single space

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