NFLScores\Utilities\Printer

Complexity

Total Complexity

8

Size/Duplication

Total Lines	133
Duplicated Lines	0 %

Test Coverage

Coverage

97.96%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	buildScoreBoardRow()	0	10	1
A	printScoreBoardHeader()	0	18	2
A	printScoreBoard()	0	13	1
A	renderGamesList()	0	15	1
A	renderScoreBoard()	0	8	2
A	__construct()	0	3	1

<?php

namespace NFLScores\Utilities;

use LaravelZero\Framework\Commands\Command;
use NFLScores\Models\Game;
use PHPCollections\Collections\GenericList;

/**
 * Utility class for printing
 * elements into the console.
 */
class Printer
{
    /**
     * The command which will be execute.
     *
     * @var \LaravelZero\Framework\Commands\Command
     */
    private $command;

    /**
     * Initialize values.
     *
     * @param \LaravelZero\Framework\Commands\Command
     */
    public function __construct(Command $command)
    {
        $this->command = $command;
    }

    /**
     * Return an array with the name and
     * score of a specific team.
     *
     * @param string $abbr
     * @param array  $score
     *
     * @return array
     */
    private function buildScoreBoardRow(string $abbr, array $score): array
    {
        return [
            $abbr,
            $score['pointQ1'],
            $score['pointQ2'],
            $score['pointQ3'],
            $score['pointQ4'],
            $score['pointOT'],
            $score['pointTotal'],
        ];
    }

    /**
     * Build and print a game' scoreboard
     * into the terminal.
     *
     * @param \NFLScores\Models\Game $game
     *
     * @return void
     */
    private function printScoreBoard(Game $game): void
    {
        $this->printScoreBoardHeader($game);

        $this->command->table(
            ['', '1', '2', '3', '4', 'OT', 'T'],
            [
                $this->buildScoreBoardRow($game->gameSchedule['homeTeamAbbr'], $game->score['homeTeamScore']),

The property score does not exist on NFLScores\Models\Game. Since you implemented __get, consider adding a @property annotation.

The property gameSchedule does not exist on NFLScores\Models\Game. Since you implemented __get, consider adding a @property annotation.

                $this->buildScoreBoardRow($game->gameSchedule['visitorTeamAbbr'], $game->score['visitorTeamScore']),
            ]
        );

        $this->command->line("\n");
    }

    /**
     * Build and print the scoreboard header.
     *
     * @param \NFLScores\Models\Game $game
     *
     * @return void
     */
    public function printScoreBoardHeader(Game $game)
    {
        $this->command->line(
            sprintf(
                '%s VS %s @ %s',
                $game->gameSchedule['homeTeamAbbr'],

The property gameSchedule does not exist on NFLScores\Models\Game. Since you implemented __get, consider adding a @property annotation.

                $game->gameSchedule['visitorTeamAbbr'],
                $game->gameSchedule['site']['siteFullname']
            )
        );

        if (!$game->isFinished()) {
            $this->command->line(sprintf(
                '%s | %s | %s %s',
                $game->getCurrentQuarter(),
                $game->score['time'],

The property score does not exist on NFLScores\Models\Game. Since you implemented __get, consider adding a @property annotation.

                $game->getPossesionTeam(),
                $game->getCurrentDown()
            ));
        }
    }

    /**
     * Print a table with a list of games.
     *
     * @param \PHPCollections\Collections\GenericList $gameCollection
     *
     * @return void
     */
    public function renderGamesList(?GenericList $gameCollection): void
    {
        $data = [];

        $gameCollection->forEach(function ($game, $key) use (&$data) {

The method forEach() does not exist on null.

The parameter $key is not used and could be removed.

            array_push($data, [
                $game->gameSchedule['homeTeamAbbr'],
                $game->gameSchedule['visitorTeamAbbr'],
                $game->gameSchedule['site']['siteFullname'],
                $game->gameSchedule['gameDate'],
                sprintf('%s ET', $game->gameSchedule['gameTimeEastern']),
            ]);
        });

        $this->command->table(['Home', 'Visitor', 'Stadium', 'Date', 'Hour'], $data);
    }

    /**
     * Print a collection of game'
     * scoreboards into the terminal.
     *
     * @param PHPCollections\Collections\GenericList
     *
     * @return void
     */
    public function renderScoreBoard(?GenericList $games): void
    {
        if (is_null($games)) {
            exit($this->command->line('Sorry, there is no games right now.'));

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

Using exit here is not recommended.

        }

        $games->forEach(function ($game, $key) {

The parameter $key is not used and could be removed.

            $this->printScoreBoard($game);
        });
    }
}