NFLScores\Commands\LiveCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Test Coverage

Coverage

70%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	16	4

<?php

namespace NFLScores\Commands;

use ErrorException;
use NFLScores\Utilities\Printer;

class LiveCommand extends AbstractCommand
{
    /**
     * The signature of the command.
     *
     * @var string
     */
    protected $signature = 'live {team?}';

    /**
     * The description of the command.
     *
     * @var string
     */
    protected $description = 'Show the current live games';

    /**
     * Execute the console command.
     *
     * @return void
     */
    public function handle(): void
    {
        try {
            $games = (!is_null($this->argument('team')))
                ? $this->NFL->getLiveGameByTeam($this->argument('team'))

It seems like $this->argument('team') can also be of type null and string[]; however, parameter $team of NFLScores\Models\NFL::getLiveGameByTeam() does only seem to accept string, maybe add an additional type check?

                : $this->NFL->getLiveGames();

            if (is_null($games)) {
                exit($this->line('Sorry, there is no live games at this moment.'));

Are you sure the usage of $this->line('Sorry, there is no live games at this moment.') targeting Illuminate\Console\Command::line() seems to always return null.

Using exit here is not recommended.

            }

            $printer = new Printer($this);

            $printer->renderScoreBoard($games);
        } catch (ErrorException $e) {
            exit($this->line('Sorry, there was a problem fetching the remote data.'));

Are you sure the usage of $this->line('Sorry, there was a problem fetching the remote data.') targeting Illuminate\Console\Command::line() seems to always return null.

Using exit here is not recommended.

        }
    }
}