TrelloBurndown\Model\StoryPointBurndown

Complexity

Total Complexity

20

Size/Duplication

Total Lines	163
Duplicated Lines	1.84 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

88.89%

Importance

Changes	2
Bugs	1	Features	1

13 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	getAverageSP()	0	4	1
A	setAverageSP()	0	4	1
A	getDoneSP()	0	4	1
A	setDoneSP()	0	4	1
A	getTotalSP()	0	4	1
A	setTotalSP()	0	4	1
A	getSprint()	0	4	1
A	setSprint()	0	4	1
A	formatDate()	0	4	1
A	getRealBurndown()	0	12	2
C	getTheoreticalBurndown()	3	23	7
A	generate()	0	7	1

<?php

namespace TrelloBurndown\Model;

/**
 * Class StoryPointBurndown.
 */
class StoryPointBurndown
{
    /**
     * @var float
     */
    private $averageSP;
    /**
     * @var array
     */
    private $doneSP;
    /**
     * @var float
     */
    private $totalSP;
    /**
     * @var Sprint
     */
    private $sprint;

    /**
     * @var string
     */
    private static $dateFormat = 'Y-m-d';

    /**
     * StoryPointBurndown constructor.
     *
     * @param Sprint $sprint
     * @param float  $totalSP
     * @param array  $doneSP
     * @param float  $averageSP
     */
    public function __construct(Sprint $sprint, float $totalSP, array $doneSP, float $averageSP)
    {
        $this->sprint = $sprint;
        $this->totalSP = $totalSP;
        $this->doneSP = $doneSP;
        $this->averageSP = $averageSP;
    }

    /**
     * @return float
     */
    public function getAverageSP()
    {
        return $this->averageSP;
    }

    /**
     * @param float $averageSP
     */
    public function setAverageSP($averageSP)
    {
        $this->averageSP = $averageSP;
    }

    /**
     * @return float

Should the return type not be array?

     */
    public function getDoneSP()
    {
        return $this->doneSP;
    }

    /**
     * @param float $doneSP
     */
    public function setDoneSP($doneSP)
    {
        $this->doneSP = $doneSP;

It seems like $doneSP of type double is incompatible with the declared type array of property $doneSP.

    }

    /**
     * @return float
     */
    public function getTotalSP()
    {
        return $this->totalSP;
    }

    /**
     * @param float $totalSP
     */
    public function setTotalSP($totalSP)
    {
        $this->totalSP = $totalSP;
    }

    /**
     * @return Sprint
     */
    public function getSprint()
    {
        return $this->sprint;
    }

    /**
     * @param Sprint $sprint
     */
    public function setSprint($sprint)
    {
        $this->sprint = $sprint;
    }

    public function formatDate(\DateTime $date)
    {
        return $date->format(self::$dateFormat);
    }

    /**
     * @return array
     */
    public function getRealBurndown()
    {
        $realBurndown = [];
        $total = $this->totalSP;
        $realBurndown[$this->formatDate($this->sprint->getStart())] = $total;
        foreach ($this->doneSP as $sp) {
            $total = $total - $sp['count'];
            $realBurndown[$this->formatDate($sp['date'])] = $total;
        }

        return $realBurndown;
    }

    /**
     * @return array|null
     */
    public function getTheoreticalBurndown()
    {
        $theoreticalBurndown = [];
        $theoreticalBurndown[$this->sprint->getStart()->format('Y-m-d')] = $this->totalSP;

        $sprintDays = $this->sprint->getSprintDays();
        if (!$sprintDays instanceof \DatePeriod) {
            return;
        }

        foreach ($sprintDays as $day) {
            if ($day instanceof \DateTime && ($day->format('N') == 6 || $day->format('N') == 7)) {

This code seems to be duplicated across your project.

                continue;
            }

            $rest = end($theoreticalBurndown) != false ? end($theoreticalBurndown) : $this->totalSP;
            $formatedDate = $this->formatDate($day);
            $doneSP = $rest - $this->averageSP;
            $theoreticalBurndown[$formatedDate] = round($doneSP, 2);
        }

        return $theoreticalBurndown;
    }

    /**
     * @return array
     */
    public function generate()
    {
        $real = $this->getRealBurndown();
        $ideal = $this->getTheoreticalBurndown();

        return ['real' => $real, 'theorical' => $ideal];
    }
}