Jeancsil\FlightSpy\Api\DataTransfer\Period

Complexity

Total Complexity

10

Size/Duplication

Total Lines	113
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

55.56%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	generateDateCombinations()	0	21	3
A	getDurationInDays()	0	4	1
A	getDateFrom()	0	4	1
A	getDateTo()	0	4	1
A	__toString()	0	18	2
A	getOneDayMore()	0	3	1

<?php
/**
 * @author Jean Silva <me@jeancsil.com>
 * @license MIT
 */
namespace Jeancsil\FlightSpy\Api\DataTransfer;

use Doctrine\Common\Collections\ArrayCollection;

class Period
{
    /**
     * @var int
     */
    private $durationInDays;

    /**
     * @var \Datetime
     */
    private $dateFrom;

    /**
     * @var \Datetime
     */
    private $dateTo;

    /**
     * @var ArrayCollection
     */
    private $combinations;

    /**
     * @param int $durationInDays
     * @param \Datetime $dateFrom
     * @param \Datetime $dateTo
     */
    public function __construct($durationInDays, \Datetime $dateFrom, \Datetime $dateTo)
    {
        $this->durationInDays = $durationInDays;
        $this->dateFrom = $dateFrom;

Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space

        $this->dateTo = $dateTo;

Equals sign not aligned with surrounding assignments; expected 9 spaces but found 1 space

        $this->combinations = new ArrayCollection();

Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space

    }

    /**
     * @return ArrayCollection

Should the return type not be ArrayCollection|null?

     */
    public function generateDateCombinations()
    {
        $possibleDays = $this->dateFrom->diff($this->dateTo)->days;

        $initialDate = clone $this->dateFrom;
        $endDate = clone $this->dateFrom->add(new \DateInterval(sprintf('P%dD', $this->durationInDays)));

Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space

        for ($i = 0; $i < $possibleDays; $i++) {
            $this->combinations->add([
                'outboundDate' => $initialDate->format('Y-m-d'),
                'inboundDate' => $endDate->format('Y-m-d')
            ]);

            if ($endDate->diff($this->dateTo)->days == 0) {
                return $this->combinations;
            }

            $initialDate->add($this->getOneDayMore());
            $endDate->add($this->getOneDayMore());
        }
    }

    /**
     * @return int
     */
    public function getDurationInDays()
    {
        return $this->durationInDays;
    }

    /**
     * @return \Datetime
     */
    public function getDateFrom()
    {
        return $this->dateFrom;
    }

    /**
     * @return \Datetime
     */
    public function getDateTo()
    {
        return $this->dateTo;
    }

    /**
     * @return string
     */
    public function __toString()
    {
        $string = sprintf(
            "Duration: %d days\nBetween %s and %s (inclusive)",
            $this->durationInDays,
            $this->dateFrom->format('d/m/Y'),
            $this->dateTo->format('d/m/Y')
        );

        if (!$this->combinations->isEmpty()) {
            $string .= sprintf(
                "\nResulted in these combinations:\n %s",
                var_export($this->combinations->toArray(), true)
            );
        }

        return $string;
    }

    /**
     * @return \DateInterval
     */
    private function getOneDayMore() {
        return new \DateInterval('P1D');
    }
}