AlgoWeb\xsdTypes\AxillaryClasses\Calender

Complexity

Total Complexity

35

Size/Duplication

Total Lines	104
Duplicated Lines	49.04 %

Coupling/Cohesion

Components	2
Dependencies	0

Importance

Changes	2
Bugs	1	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
B	validateInput()	6	9	7
B	valdidateState()	0	14	10
A	getYearOrHolder()	0	7	2
A	fromDay()	9	9	3
A	FromMonthDay()	9	9	3
A	fromMonth()	9	9	3
A	fromYearMonth()	9	9	3
A	fromYear()	9	9	3

<?php
/**
 * Created by PhpStorm.
 * User: Barnso
 * Date: 14/07/2017
 * Time: 8:28 PM.
 */
namespace AlgoWeb\xsdTypes\AxillaryClasses;

class Calender

Since you have declared the constructor as private, maybe you should also declare the class as final.

{
    const timezoneRegexPattern = '([-+][0-1]\d:[0-6]\d|Z*)';

This class constant is not uppercase (expected TIMEZONEREGEXPATTERN).

    private $year;
    private $month;
    private $day;

    private function __construct($year = null, $month = null, $day = null, $timezone = 'Z')

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

    {
        $this->validateInput($year, $month, $day);
        $this->year = $year;
        $this->month = $month;
        $this->day = $day;
        $this->valdidateState();
    }

    private function validateInput($year = null, $month = null, $day = null)
    {
        if (null === $year && null === $month && null === $day) {

This code seems to be duplicated across your project.

            throw new \InvalidArgumentException('A Caldender class must have at least a day, month, or year');
        }
        if (null !== $year && null === $month && null !== $day) {

This code seems to be duplicated across your project.

            throw new \InvalidArgumentException('a year day value is not valid');
        }
    }

    private function valdidateState()
    {
        if (null !== $this->day && (1 > $this->day || 31 < $this->day)) {
            throw new \InvalidArgumentException('the day must be greater 0 and less then 32');
        }
        if (null !== $this->month && (1 > $this->month || 12 < $this->month)) {
            throw new \InvalidArgumentException('the month must be greater 0 and less then 12');
        }
        if ($this->month !== null && $this->day !== null &&
            cal_days_in_month($this->getYearOrHolder(), $this->month, $this->day) < $this->day
        ) {
            throw new \InvalidArgumentException('there are not that many days in the assigned month');
        }
    }

    private function getYearOrHolder()

The return type could not be reliably inferred; please add a @return annotation.

    {
        if (null == $this->year) {
            return '2016';
        }
        return $this->year;
    }

    /**
     * @param string $day should match regex pattern ----\d\d
     *
     * @return Calender
     */
    public static function fromDay($day)

This method seems to be duplicated in your project.

    {
        $re = '/----(0[1-9]|1[0-9]|2[0-9]|3[0-1]|[1-9])([-+][0-1]\d:[0-6]\d|Z*)/';
        preg_match_all($re, $day, $matches, PREG_SET_ORDER, 0);
        if (count($matches) != 1 && count($matches[0]) != 3) {
            throw new \InvalidArgumentException('Unable to extract day from input string');
        }
        return new self(null, null, $matches[0][1], $matches[0][2]);
    }

    public static function FromMonthDay($monthDay)

This method seems to be duplicated in your project.

This method is not in camel caps format.

    {
        $re = '/--(1[0-2]|0[1-9]|[1-9])-(0[1-9]|1[0-9]|2[0-9]|3[0-1]|[1-9])([-+][0-1]\d:[0-6]\d|Z*)/';
        preg_match_all($re, $monthDay, $matches, PREG_SET_ORDER, 0);
        if (count($matches) != 1 && count($matches[0]) != 4) {
            throw new \InvalidArgumentException('Unable to extract month day from input string');
        }
        return new self(null, $matches[0][1], $matches[0][2], $matches[0][3]);
    }

    public static function fromMonth($month)

This method seems to be duplicated in your project.

    {
        $re = '/--(1[0-2]|0[1-9]|[1-9])([-+][0-1]\d:[0-6]\d|Z*)/';
        preg_match_all($re, $month, $matches, PREG_SET_ORDER, 0);
        if (count($matches) != 1 && count($matches[0]) != 3) {
            throw new \InvalidArgumentException('Unable to extract month from input string');
        }
        return new self(null, $matches[0][1], null, $matches[0][2]);
    }

    public static function fromYearMonth($yearMonth)

This method seems to be duplicated in your project.

    {
        $re = '/(\d{4})-(1[0-2]|0[1-9]|[1-9])([-+][0-1]\d:[0-6]\d|Z*)/';
        preg_match_all($re, $yearMonth, $matches, PREG_SET_ORDER, 0);
        if (count($matches) != 1 && count($matches[0]) != 3) {
            throw new \InvalidArgumentException('Unable to extract month from input string');
        }
        return new self($matches[0][1], null, $matches[0][2], $matches[0][3]);
    }

    public static function fromYear($year)

This method seems to be duplicated in your project.

    {
        $re = '/(\d{4})([-+][0-1]\d:[0-6]\d|Z*)/';
        preg_match_all($re, $year, $matches, PREG_SET_ORDER, 0);
        if (count($matches) != 1 && count($matches[0]) != 3) {
            throw new \InvalidArgumentException('Unable to extract month from input string');
        }
        return new self($matches[0][1], null, null, $matches[0][2]);
    }
}