DH\NavigationBundle\Command\RoutingCommand::execute()

Complexity

Conditions	12
Paths	513

Size

Total Lines	89
Code Lines	53

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        if (\count($input->getOption('waypoint')) < 2) {

It seems like $input->getOption('waypoint') can also be of type boolean and string; however, parameter $var of count() does only seem to accept Countable|array, maybe add an additional type check?

            throw new InvalidArgumentException('A route needs at least two waypoints (start and end locations).');
        }

        if ($input->getOption('provider')) {
            $this->manager->using($input->getOption('provider'));
        }

        $query = $this->manager->createRoutingQuery();

        foreach ($input->getOption('waypoint') as $waypoint) {
            $query->addWaypoint($waypoint);
        }

        if ($input->getOption('departure')) {
            $query->setDepartureTime(new \DateTime($input->getOption('departure')));

It seems like $input->getOption('departure') can also be of type string[]; however, parameter $time of DateTime::__construct() does only seem to accept string, maybe add an additional type check?

        }

        if ($input->getOption('arrival')) {
            $query->setArrivalTime(new \DateTime($input->getOption('arrival')));
        }

        if ($input->getOption('traffic')) {
            $query->setTrafficMode($input->getOption('traffic'));

The method setTrafficMode() does not exist on DH\NavigationBundle\Contract\Routing\RoutingQueryInterface. It seems like you code against a sub-type of DH\NavigationBundle\Contract\Routing\RoutingQueryInterface such as DH\NavigationBundle\Provider\Here\Routing\RoutingQuery.

        }

        if ($input->getOption('language')) {
            $query->setLanguage($input->getOption('language'));
        }

        $response = $query->execute();

        $io = new SymfonyStyle($input, $output);

        $routes = $response->getRoutes();

        $io->section('Summary');

        $data = [];
        foreach ($routes as $index => $route) {
            /**
             * @var Summary $summary
             */
            $summary = $route->getSummary();
            $data[] = [
                $index + 1,
                $summary->getDistance()->getFormattedValue(2),
                $summary->getTravelTime()->getFormattedValue(2),
            ];
        }

        $table = new Table($output);
        $table
            ->setHeaders(['route', 'distance', 'duration'])
            ->setRows($data)
        ;
        $table->render();

        $io->newLine();

        foreach ($routes as $routeIndex => $route) {
            $io->section('Route #'.($routeIndex + 1));

            $legs = $route->getLegs();
            foreach ($legs as $legIndex => $leg) {
                $steps = $leg->getSteps();
                $io->writeln(sprintf('<comment>Leg #%d</comment> - %d steps.', $legIndex + 1, \count($steps)));

                $data = [];
                foreach ($steps as $stepIndex => $step) {
                    $data[] = [
                        $stepIndex + 1,
                        implode(', ', $step->getPosition()),
                        $step->getDistance()->getFormattedValue(2),
                        $step->getDuration()->getFormattedValue(2),
                        $step->getInstruction()
                    ];
                }

                $table = new Table($output);
                $table
                    ->setHeaders(['step', 'position', 'distance', 'duration', 'instruction'])
                    ->setRows($data)
                    ->setColumnMaxWidth(4, 100)
                ;
                $table->render();
                $io->newLine();
            }