Pause::__construct() - Code Metrics - Inspection of "add getDuration() method for Pause" - aminebenhariz/Attendance - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 26a050...b0c7d0 )

by Amine

created 2015-12-15 10:42 UTC

Pause::__construct() A

↳ Parent: Pause

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes	2
Bugs	0	Features	0

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

<?php

namespace AmineBenHariz\Attendance;

/**
 * Class Pause
 * @package AmineBenHariz\Attendance
 */
class Pause
{
    /**
     * @var \DateTime
     */
    private $start;

    /**
     * @var \DateTime
     */
    private $end;

    /**
     * Pause constructor.
     * @param \DateTime $start
     * @param \DateTime $end
     */
    public function __construct(\DateTime $start, \DateTime $end)
    {
        if ($start > $end) {
            throw new \InvalidArgumentException("Pause can't start after ending");
        }

        if ($start->format('Y-m-d') !== $end->format('Y-m-d')) {
            throw new \InvalidArgumentException("Pause must end in the same day");
        }

        $this->start = $start;
        $this->end = $end;
    }

    /**
     * @return \DateTime
     */
    public function getStart()
    {
        return $this->start;
    }

    /**
     * @return \DateTime
     */
    public function getEnd()
    {
        return $this->end;
    }

    /**
     * @return \DateInterval
     * @throws \Exception
     */
    public function getDuration()
    {
        $duration = $this->getStart()->diff($this->getEnd());
        if ($duration === false) {
            throw new \Exception('Error while calculating pause duration');
        }

        return $duration;
    }

    /**
     * @param Pause $pause
     * @return bool
     */
    public function isOverlapping(Pause $pause)
    {
        if ($this->getEnd() <= $pause->getStart()) {
            return false;
        }

        if ($this->getStart() >= $pause->getEnd()) {
            return false;
        }

        return true;
    }
}


1		<?php
2
3		namespace AmineBenHariz\Attendance;
4
5		/**
6		* Class Pause
7		* @package AmineBenHariz\Attendance
8		*/
9		class Pause
10		{
11		/**
12		* @var \DateTime
13		*/
14		private $start;
15
16		/**
17		* @var \DateTime
18		*/
19		private $end;
20
21		/**
22		* Pause constructor.
23		* @param \DateTime $start
24		* @param \DateTime $end
25		*/
26	16	public function __construct(\DateTime $start, \DateTime $end)
27		{
28	16	if ($start > $end) {
29	4	throw new \InvalidArgumentException("Pause can't start after ending");
30		}
31
32	12	if ($start->format('Y-m-d') !== $end->format('Y-m-d')) {
33	4	throw new \InvalidArgumentException("Pause must end in the same day");
34		}
35
36	8	$this->start = $start;
37	8	$this->end = $end;
38	8	}
39
40		/**
41		* @return \DateTime
42		*/
43	24	public function getStart()
44		{
45	24	return $this->start;
46		}
47
48		/**
49		* @return \DateTime
50		*/
51	24	public function getEnd()
52		{
53	24	return $this->end;
54		}
55
56		/**
57		* @return \DateInterval
58		* @throws \Exception
59		*/
60	4	public function getDuration()
61		{
62	4	$duration = $this->getStart()->diff($this->getEnd());
63	4	if ($duration === false) {
64		throw new \Exception('Error while calculating pause duration');
65		}
66
67	4	return $duration;
68		}
69
70		/**
71		* @param Pause $pause
72		* @return bool
73		*/
74	16	public function isOverlapping(Pause $pause)
75		{
76	16	if ($this->getEnd() <= $pause->getStart()) {
77	8	return false;
78		}
79
80	16	if ($this->getStart() >= $pause->getEnd()) {
81	8	return false;
82		}
83
84	8	return true;
85		}
86		}
87

aminebenhariz / Attendance

Push — master ( 26a050...b0c7d0 )

Pause::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like