AbstractSpecialDate::getEndDate() - Code Metrics - Inspection of "Lock on PHP7" - stefanius/special-dates - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#7)

by steef

created 2016-11-29 19:34 UTC

AbstractSpecialDate::getEndDate() A

↳ Parent: AbstractSpecialDate

Complexity

Conditions	3
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	8
rs	9.4285
c	0
b	0
f	0
ccs	4
cts	4
cp	1
cc	3
eloc	4
nc	2
nop	0
crap	3

<?php

namespace Stefanius\SpecialDates\SDK;

abstract class AbstractSpecialDate implements SpecialDateInterface
{
    /**
     * @var \DateTime
     */
    protected $startDate;

    /**
     * @var \DateTime
     */
    protected $endDate;

    /**
     * @var \DateTime
     */
    protected $zeroDate;

    /**
     * @var string
     */
    protected $description;

    /**
     * @var int
     */
    protected $totalLength = 0;

    /**
     * @var bool
     */
    protected $valid = true;

    /**
     * @var bool
     */
    protected $bankHoliday = false;

    /**
     * @var bool
     */
    protected $nationalAcceptedParty = false;


    /**
     * @var string
     */
    protected $normalizedDescription = false;

    /**
     * @var int
     */
    protected $year;

    public function __construct($year)
    {
        $this->year     = $year;
        $this->zeroDate = new \DateTime();
        $this->zeroDate->setDate(1, 1, 1);

        $this->startDate = new \DateTime();
        $this->startDate->setDate(1, 1, 1);

        $this->endDate = new \DateTime();
        $this->endDate->setDate(1, 1, 1);

        $this->generate();

        if ($this->startDate->format('Y') . '-' . $this->startDate->format('m') . '-' . $this->startDate->format('d') === '0001-01-01') {

            $this->valid = false;
            $this->totalLength = 0;
        }

        if ($this->endDate->format('Y') . '-' . $this->endDate->format('m') . '-' . $this->endDate->format('d') === '0001-01-01') {

            $this->valid = false;
            $this->totalLength = 0;
        }

        $this->normalizeDescription();
    }

    protected function normalizeDescription()
    {
        $string = iconv('UTF-8', 'ASCII//TRANSLIT', $this->description);
        $string = trim($string);
        $string = preg_replace("/[^a-zA-Z0-9_| -]/", ' ', $string);
        $string = preg_replace("/[| -]+/", '-', $string);
        $string = strtolower(trim($string, '-'));
        $string = preg_replace('/-{2,}/', ' ', $string);

        $this->normalizedDescription = $string;
    }

    /**
     * @param $year
     * @param $month
     * @param $day
     *
     * @return \DateTime
     */
    protected function generateDateTime($year, $month, $day)
    {
        $date = \DateTime::createFromFormat('Y-m-d', $year . '-' . $month . '-' . $day);

        if ($date === false) {
            return $this->zeroDate;
        }

        return $date;
    }

    protected function setupDateTimeObjects(\DateTime $start, \DateTime $end = null)
    {
        $this->startDate = $start;

        if ($end === null) {
            $this->endDate = $start;
        } else {
            $this->endDate = $end;
        }

        $interval = $this->startDate->diff($this->endDate);

        $this->totalLength = (int)$interval->format('%R%a') + 1;
    }

    /**
     * @return \DateTime
     */
    public function getStartDate()
    {
        if (!$this->valid || $this->startDate == null) {
            return $this->zeroDate;
        }

        return $this->startDate;
    }

    /**
     * @return \DateTime
     */
    public function getEndDate()
    {
        if (!$this->valid || $this->endDate == null) {
            return $this->zeroDate;
        }

        return $this->endDate;
    }

    /**
     * @return string
     */
    public function getDescription()
    {
        return $this->description;
    }

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

    /**
     * @return bool
     */
    public function isValid()
    {
        return $this->valid;
    }

    /**
     * @return boolean
     */
    public function isBankHoliday()
    {
        return $this->bankHoliday;
    }

    /**
     * @return boolean
     */
    public function isNationalAcceptedParty()
    {
        return $this->nationalAcceptedParty;
    }

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

    /**
     * @return string
     */
    public function getNormalizedDescription()
    {
        return $this->normalizedDescription;
    }

    abstract protected function generate();
}


1			<?php
2
3			namespace Stefanius\SpecialDates\SDK;
4
5			abstract class AbstractSpecialDate implements SpecialDateInterface
6			{
7			/**
8			* @var \DateTime
9			*/
10			protected $startDate;
11
12			/**
13			* @var \DateTime
14			*/
15			protected $endDate;
16
17			/**
18			* @var \DateTime
19			*/
20			protected $zeroDate;
21
22			/**
23			* @var string
24			*/
25			protected $description;
26
27			/**
28			* @var int
29			*/
30			protected $totalLength = 0;
31
32			/**
33			* @var bool
34			*/
35			protected $valid = true;
36
37			/**
38			* @var bool
39			*/
40			protected $bankHoliday = false;
41
42			/**
43			* @var bool
44			*/
45			protected $nationalAcceptedParty = false;
			0 ignored issues – show Comprehensibility Naming introduced 2016-05-19 20:02 UTC by Report Bug Copy Issue Report The variable name `$nationalAcceptedParty` exceeds the maximum configured length of `20`. Very long variable names usually make code harder to read. It is therefore recommended not to make variable names too verbose. Loading history...
46
47			/**
48			* @var string
49			*/
50			protected $normalizedDescription = false;
			0 ignored issues – show Comprehensibility Naming introduced 2016-05-19 20:02 UTC by Report Bug Copy Issue Report The variable name `$normalizedDescription` exceeds the maximum configured length of `20`. Very long variable names usually make code harder to read. It is therefore recommended not to make variable names too verbose. Loading history...
51			/**
52			* @var int
53			*/
54			protected $year;
55
56	69		public function __construct($year)
57			{
58	69		$this->year = $year;
59	69		$this->zeroDate = new \DateTime();
60	69		$this->zeroDate->setDate(1, 1, 1);
61
62	69		$this->startDate = new \DateTime();
63	69		$this->startDate->setDate(1, 1, 1);
64
65	69		$this->endDate = new \DateTime();
66	69		$this->endDate->setDate(1, 1, 1);
67
68	69		$this->generate();
69
70	69	View Code Duplication	if ($this->startDate->format('Y') . '-' . $this->startDate->format('m') . '-' . $this->startDate->format('d') === '0001-01-01') {
			0 ignored issues – show Duplication introduced 2015-12-29 21:33 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
71	28		$this->valid = false;
72	28		$this->totalLength = 0;
73			}
74
75	69	View Code Duplication	if ($this->endDate->format('Y') . '-' . $this->endDate->format('m') . '-' . $this->endDate->format('d') === '0001-01-01') {
			0 ignored issues – show Duplication introduced 2015-12-29 21:33 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
76	28		$this->valid = false;
77	28		$this->totalLength = 0;
78			}
79
80	69		$this->normalizeDescription();
81	69		}
82
83	69		protected function normalizeDescription()
84			{
85	69		$string = iconv('UTF-8', 'ASCII//TRANSLIT', $this->description);
86	69		$string = trim($string);
87	69		$string = preg_replace("/[^a-zA-Z0-9_\| -]/", ' ', $string);
88	69		$string = preg_replace("/[\| -]+/", '-', $string);
89	69		$string = strtolower(trim($string, '-'));
90	69		$string = preg_replace('/-{2,}/', ' ', $string);
91
92	69		$this->normalizedDescription = $string;
93	69		}
94
95			/**
96			* @param $year
97			* @param $month
98			* @param $day
99			*
100			* @return \DateTime
101			*/
102	19		protected function generateDateTime($year, $month, $day)
103			{
104	19		$date = \DateTime::createFromFormat('Y-m-d', $year . '-' . $month . '-' . $day);
105
106	19		if ($date === false) {
107			return $this->zeroDate;
108			}
109
110	19		return $date;
111			}
112
113	25		protected function setupDateTimeObjects(\DateTime $start, \DateTime $end = null)
114			{
115	25		$this->startDate = $start;
116
117	25		if ($end === null) {
118	21		$this->endDate = $start;
119			} else {
120	4		$this->endDate = $end;
121			}
122
123	25		$interval = $this->startDate->diff($this->endDate);
124
125	25		$this->totalLength = (int)$interval->format('%R%a') + 1;
126	25		}
127
128			/**
129			* @return \DateTime
130			*/
131	69		public function getStartDate()
132			{
133	69		if (!$this->valid \|\| $this->startDate == null) {
134	28		return $this->zeroDate;
135			}
136
137	44		return $this->startDate;
138			}
139
140			/**
141			* @return \DateTime
142			*/
143	66		public function getEndDate()
144			{
145	66		if (!$this->valid \|\| $this->endDate == null) {
146	25		return $this->zeroDate;
147			}
148
149	41		return $this->endDate;
150			}
151
152			/**
153			* @return string
154			*/
155	66		public function getDescription()
156			{
157	66		return $this->description;
158			}
159
160			/**
161			* @return int
162			*/
163	66		public function getTotalLength()
164			{
165	66		return $this->totalLength;
166			}
167
168			/**
169			* @return bool
170			*/
171	66		public function isValid()
172			{
173	66		return $this->valid;
174			}
175
176			/**
177			* @return boolean
178			*/
179	66		public function isBankHoliday()
180			{
181	66		return $this->bankHoliday;
182			}
183
184			/**
185			* @return boolean
186			*/
187			public function isNationalAcceptedParty()
188			{
189			return $this->nationalAcceptedParty;
190			}
191
192			/**
193			* @return int
194			*/
195			public function getYear()
196			{
197			return $this->year;
198			}
199
200			/**
201			* @return string
202			*/
203	1		public function getNormalizedDescription()
204			{
205	1		return $this->normalizedDescription;
206			}
207
208			abstract protected function generate();
209			}
210

stefanius / special-dates

Pull Request — master (#7)

AbstractSpecialDate::getEndDate() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like