Period::__toString() - Code Metrics - jeancsil/flight-spy - Measure and Improve Code Quality continuously with Scrutinizer

Period::__toString() A
last analyzed 2017-04-10 08:24 UTC

↳ Parent: Period

Complexity

Conditions	2
Paths	2

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	18
ccs	0
cts	11
cp	0
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	11
nc	2
nop	0
crap	6

<?php
/**
 * @author Jean Silva <[email protected]>
 * @license MIT
 */
namespace Jeancsil\FlightSpy\Api\DataTransfer;

use Doctrine\Common\Collections\ArrayCollection;

class Period
{
    /**
     * @var int
     */
    private $durationInDays;

    /**
     * @var \Datetime
     */
    private $dateFrom;

    /**
     * @var \Datetime
     */
    private $dateTo;

    /**
     * @var ArrayCollection
     */
    private $combinations;

    /**
     * @param int $durationInDays
     * @param \Datetime $dateFrom
     * @param \Datetime $dateTo
     */
    public function __construct($durationInDays, \Datetime $dateFrom, \Datetime $dateTo)
    {
        if (!is_int($durationInDays) || $durationInDays <= 0) {
            throw new \InvalidArgumentException('The duration should be bigger than 0');
        }

        $this->durationInDays = $durationInDays;
        $this->dateFrom = $dateFrom;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->dateTo = $dateTo;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->combinations = new ArrayCollection();
$a = "a";
$ab = "ab";
$abc = "abc";
    }

    /**
     * @return ArrayCollection

     */
    public function generateDateCombinations()
    {
        $possibleDays = $this->dateFrom->diff($this->dateTo)->days;

        if ($possibleDays < $this->durationInDays) {
            throw new \InvalidArgumentException('There are no combinations possible');
        }

        $initialDate = clone $this->dateFrom;
        $endDate = clone $this->dateFrom->add(new \DateInterval(sprintf('P%dD', $this->durationInDays)));
$a = "a";
$ab = "ab";
$abc = "abc";

        for ($i = 0; $i < $possibleDays; $i++) {
            $this->combinations->add([
                'outboundDate' => $initialDate->format('Y-m-d'),
                'inboundDate' => $endDate->format('Y-m-d')
            ]);

            if ($endDate->diff($this->dateTo)->days == 0) {
                return $this->combinations;
            }

            $initialDate->add($this->getOneDayMore());
            $endDate->add($this->getOneDayMore());
        }
    }

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

    /**
     * @return \Datetime
     */
    public function getDateFrom()
    {
        return $this->dateFrom;
    }

    /**
     * @return \Datetime
     */
    public function getDateTo()
    {
        return $this->dateTo;
    }

    /**
     * @return string
     */
    public function __toString()
    {
        $string = sprintf(
            "Duration: %d days\nBetween %s and %s (inclusive)",
            $this->durationInDays,
            $this->dateFrom->format('d/m/Y'),
            $this->dateTo->format('d/m/Y')
        );

        if (!$this->combinations->isEmpty()) {
            $string .= sprintf(
                "\nResulted in these combinations:\n %s",
                var_export($this->combinations->toArray(), true)
            );
        }

        return $string;
    }

    /**
     * @return \DateInterval
     */
    private function getOneDayMore()
    {
        return new \DateInterval('P1D');
    }
}


1		<?php
2		/**
3		* @author Jean Silva <[email protected]>
4		* @license MIT
5		*/
6		namespace Jeancsil\FlightSpy\Api\DataTransfer;
7
8		use Doctrine\Common\Collections\ArrayCollection;
9
10		class Period
11		{
12		/**
13		* @var int
14		*/
15		private $durationInDays;
16
17		/**
18		* @var \Datetime
19		*/
20		private $dateFrom;
21
22		/**
23		* @var \Datetime
24		*/
25		private $dateTo;
26
27		/**
28		* @var ArrayCollection
29		*/
30		private $combinations;
31
32		/**
33		* @param int $durationInDays
34		* @param \Datetime $dateFrom
35		* @param \Datetime $dateTo
36		*/
37	4	public function __construct($durationInDays, \Datetime $dateFrom, \Datetime $dateTo)
38		{
39	4	if (!is_int($durationInDays) \|\| $durationInDays <= 0) {
40	3	throw new \InvalidArgumentException('The duration should be bigger than 0');
41		}
42
43	1	$this->durationInDays = $durationInDays;
44	1	$this->dateFrom = $dateFrom;
		0 ignored issues – show Coding Style introduced 2017-01-14 14:13 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
45	1	$this->dateTo = $dateTo;
		0 ignored issues – show Coding Style introduced 2017-01-14 14:13 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 9 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
46	1	$this->combinations = new ArrayCollection();
		0 ignored issues – show Coding Style introduced 2017-01-14 17:55 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
47	1	}
48
49		/**
50		* @return ArrayCollection
		0 ignored issues – show Documentation introduced 2017-01-14 14:13 UTC by Report Bug Copy Issue Report Should the return type not be `ArrayCollection\|null`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
51		*/
52	3	public function generateDateCombinations()
53		{
54	3	$possibleDays = $this->dateFrom->diff($this->dateTo)->days;
55
56	3	if ($possibleDays < $this->durationInDays) {
57	1	throw new \InvalidArgumentException('There are no combinations possible');
58		}
59
60	2	$initialDate = clone $this->dateFrom;
61	2	$endDate = clone $this->dateFrom->add(new \DateInterval(sprintf('P%dD', $this->durationInDays)));
		0 ignored issues – show Coding Style introduced 2017-01-14 14:13 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
62
63	2	for ($i = 0; $i < $possibleDays; $i++) {
64	2	$this->combinations->add([
65	2	'outboundDate' => $initialDate->format('Y-m-d'),
66	2	'inboundDate' => $endDate->format('Y-m-d')
67		]);
68
69	2	if ($endDate->diff($this->dateTo)->days == 0) {
70	2	return $this->combinations;
71		}
72
73	2	$initialDate->add($this->getOneDayMore());
74	2	$endDate->add($this->getOneDayMore());
75		}
76		}
77
78		/**
79		* @return int
80		*/
81		public function getDurationInDays()
82		{
83		return $this->durationInDays;
84		}
85
86		/**
87		* @return \Datetime
88		*/
89		public function getDateFrom()
90		{
91		return $this->dateFrom;
92		}
93
94		/**
95		* @return \Datetime
96		*/
97		public function getDateTo()
98		{
99		return $this->dateTo;
100		}
101
102		/**
103		* @return string
104		*/
105		public function __toString()
106		{
107		$string = sprintf(
108		"Duration: %d days\nBetween %s and %s (inclusive)",
109		$this->durationInDays,
110		$this->dateFrom->format('d/m/Y'),
111		$this->dateTo->format('d/m/Y')
112		);
113
114		if (!$this->combinations->isEmpty()) {
115		$string .= sprintf(
116		"\nResulted in these combinations:\n %s",
117		var_export($this->combinations->toArray(), true)
118		);
119		}
120
121		return $string;
122		}
123
124		/**
125		* @return \DateInterval
126		*/
127	2	private function getOneDayMore()
128		{
129	2	return new \DateInterval('P1D');
130		}
131		}
132

jeancsil / flight-spy

Period::__toString() A last analyzed 2017-04-10 08:24 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

Period::__toString() A
last analyzed 2017-04-10 08:24 UTC