AlgoWeb\xsdTypes\AxillaryClasses\XMLDateInterval

Complexity

Total Complexity

14

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	6
Bugs	2	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	2
A	__toString()	0	9	2
A	handleSign()	0	6	2
A	handleChar()	0	11	3
A	handleN()	0	5	3
A	handleT()	0	5	1
A	HandleOther()	0	4	1

<?php

namespace AlgoWeb\xsdTypes\AxillaryClasses;

class XMLDateInterval extends \DateInterval
{
    /**
     * formating string like ISO 8601 (PnYnMnDTnHnMnS).
     */
    const INTERVAL_ISO8601 = 'P%yY%mM%dDT%hH%iM%sS';
    private $pattern;
    private $patternLen;

    public function __construct($intervalSpec, $pattern = 'PnYnMnDTnHnMnS')
    {
        if ($intervalSpec[0] == '-') {
            $intervalSpec = substr($intervalSpec, 1);

Consider using a different name than the parameter $intervalSpec. This often makes code more readable.

            $this->invert = true;

The property $invert was declared of type integer, but true is of type boolean. Maybe add a type cast?

        }
        parent::__construct($intervalSpec);
        $this->pattern = trim($pattern);

        $this->patternLen = strlen($this->pattern);
    }

    /**
     * formating the interval like ISO 8601 (PnYnMnDTnHnMnS).
     *
     * @return string

Should the return type not be string|null?

     */
    public function __toString()
    {
        $sReturn = $this->handleSign();
        $tSeen = false;
        for ($i = 0; $i < $this->patternLen; $i++) {
            $sReturn .= $this->handleChar($i, $tSeen);
        }
        return $sReturn;
    }

    private function handleSign()
    {
        if ($this->invert === 1) {
            return '-';
        }
    }

    private function handleChar($i, &$tSeen)

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

    {
        switch ($this->pattern[$i]) {
            case 'n':
                return $this->handleN($i, $tSeen);
            case 'T':
                return $this->handleT($tSeen);
            default:
                return $this->HandleOther($i);
        }
    }

    private function handleN($i, $tSeen)

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

    {
        $v = ($this->pattern[$i + 1] == 'M' && $tSeen) ? 'i' : strtolower($this->pattern[$i + 1]);
        return $this->$v;
    }

    private function handleT(&$tSeen)
    {
        $tSeen = true;
        return 'T';
    }

    private function HandleOther($i)

This method is not in camel caps format.

    {
        return $this->pattern[$i];
    }
}