DavisPeixoto\ReportDates\WeekInterval

Complexity

Total Complexity

20

Size/Duplication

Total Lines	157
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

72.34%

Importance

Changes

0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getStartDate()	0	4	1
A	getEndDate()	0	4	1
A	getBusinessDays()	0	4	1
A	getTotalDays()	0	4	1
A	getWeekNumber()	0	4	1
A	getYearWeek()	0	4	1
A	getYearMonth()	0	4	1
A	countBusinessDays()	0	15	3
A	isBusinessDay()	0	10	3
A	makeDayNumber()	0	11	3
A	incrementDay()	0	17	3

<?php

namespace DavisPeixoto\ReportDates;

use DateInterval;
use DateTime;
use Exception;

class WeekInterval
{
    /**
     * @var DateTime $startDate
     */
    private $startDate;

    /**
     * @var DateTime $endDate
     */
    private $endDate;

    /**
     * @var DatesConfig $config
     */
    private $config;

    /**
     * WeekInterval constructor.
     *
     * @param DateTime $startDate
     * @param DateTime $endDate
     * @param DatesConfig $config
     */
    public function __construct(DateTime $startDate, DateTime $endDate, DatesConfig $config)
    {
        $this->startDate = $startDate;
        $this->endDate = $endDate;
        $this->config = $config;
    }

    /**
     * @return DateTime
     */
    public function getStartDate(): DateTime
    {
        return $this->startDate;
    }

    /**
     * @return DateTime
     */
    public function getEndDate(): DateTime
    {
        return $this->endDate;
    }

    /**
     * @return int
     */
    public function getBusinessDays(): int
    {
        return $this->countBusinessDays();
    }

    /**
     * @return int
     */
    public function getTotalDays(): int
    {
        return (int)$this->endDate->diff($this->startDate)->format('%a') + 1;
    }

    /**
     * @return string
     */
    public function getWeekNumber(): string
    {
        return $this->endDate->format('W');
    }

    /**
     * @return string
     */
    public function getYearWeek(): string
    {
        return $this->endDate->format('oW');
    }

    /**
     * @return string
     */
    public function getYearMonth(): string
    {
        return $this->endDate->format('Ym');
    }

    private function countBusinessDays()
    {
        $startDate = clone $this->startDate;
        $count = 0;

        while ($startDate <= $this->endDate) {
            if ($this->isBusinessDay($this->makeDayNumber($startDate))) {

It seems like $this->makeDayNumber($startDate) targeting DavisPeixoto\ReportDates\WeekInterval::makeDayNumber() can also be of type boolean; however, DavisPeixoto\ReportDates\WeekInterval::isBusinessDay() does only seem to accept object<DavisPeixoto\ReportDates\DayNumber>, maybe add an additional type check?

                $count++;
            }

            $startDate = $this->incrementDay($startDate);
        }

        return $count;
    }

    /**
     * @param DayNumber $day
     * @return bool
     */
    private function isBusinessDay(DayNumber $day)
    {
        foreach ($this->config->getBusinessDays() as $businessDay) {
            if ($day->equals($businessDay)) {
                return true;
            }
        }

        return false;
    }

    /**
     * @param DateTime $day
     * @return bool|DayNumber
     */
    private function makeDayNumber(DateTime $day)
    {
        $values = DayNumber::values();
        foreach ($values as $dayNumber) {
            if ($dayNumber->getValue() === $day->format('w')) {
                return $dayNumber;
            }
        }

        return false;
    }

    /**
     * @param DateTime $date
     * @return DateTime
     * @throws Exception
     */
    private function incrementDay(DateTime $date)
    {
        $oneDay = new DateInterval('P1D');
        $oneHour = new DateInterval('PT1H');

        $date->add($oneDay);

        if ($date->format('h') === '01') {
            $date->sub($oneHour);
        }

        if ($date->format('h') === '23') {
            $date->add($oneHour);
        }

        return $date;
    }
}