Partial::__construct() - Code Metrics - Inspection of "Implement imap client" - genkgo/mail - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#42)

by Frederik

created 2018-03-03 14:11 UTC

Partial::__construct() A

↳ Parent: Partial

Complexity

Conditions	3
Paths	2

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	3

Importance

Changes

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

<?php
declare(strict_types=1);

namespace Genkgo\Mail\Protocol\Imap\MessageData;

final class Partial
{
    /**
     * @var int
     */
    private $firstByte;
    /**
     * @var int
     */
    private $lastByte;

    /**
     * @param int $firstByte
     * @param int $lastByte
     */
    public function __construct(int $firstByte, int $lastByte)
    {
        if ($firstByte < 0 || $lastByte < $firstByte) {
            throw new \InvalidArgumentException(
                'Last byte must be greater than first byte, and first byte must be greater than zero'
            );
        }

        $this->firstByte = $firstByte;
        $this->lastByte = $lastByte;
    }

    /**
     * @return string
     */
    public function __toString(): string
    {
        return sprintf(
            '<%s>',
            $this->firstByte === $this->lastByte ? $this->firstByte : $this->firstByte . '.' . $this->lastByte
        );
    }

    /**
     * @param string $partial
     * @return Partial
     */
    public static function fromString(string $partial): self
    {
        $result = preg_match('/^<([0-9]+)(\.([0-9]+))?>$/', $partial, $matches);

        if ($result !== 1) {
            throw new \InvalidArgumentException('String is not a partial');
        }

        return new self((int)$matches[1], (int)($matches[3] ?? (int)$matches[1]));
    }
}

1		<?php
2		declare(strict_types=1);
3
4		namespace Genkgo\Mail\Protocol\Imap\MessageData;
5
6		final class Partial
7		{
8		/**
9		* @var int
10		*/
11		private $firstByte;
12		/**
13		* @var int
14		*/
15		private $lastByte;
16
17		/**
18		* @param int $firstByte
19		* @param int $lastByte
20		*/
21	8	public function __construct(int $firstByte, int $lastByte)
22		{
23	8	if ($firstByte < 0 \|\| $lastByte < $firstByte) {
24	2	throw new \InvalidArgumentException(
25	2	'Last byte must be greater than first byte, and first byte must be greater than zero'
26		);
27		}
28
29	6	$this->firstByte = $firstByte;
30	6	$this->lastByte = $lastByte;
31	6	}
32
33		/**
34		* @return string
35		*/
36	5	public function __toString(): string
37		{
38	5	return sprintf(
39	5	'<%s>',
40	5	$this->firstByte === $this->lastByte ? $this->firstByte : $this->firstByte . '.' . $this->lastByte
41		);
42		}
43
44		/**
45		* @param string $partial
46		* @return Partial
47		*/
48	3	public static function fromString(string $partial): self
49		{
50	3	$result = preg_match('/^<([0-9]+)(\.([0-9]+))?>$/', $partial, $matches);
51
52	3	if ($result !== 1) {
53	1	throw new \InvalidArgumentException('String is not a partial');
54		}
55
56	2	return new self((int)$matches[1], (int)($matches[3] ?? (int)$matches[1]));
57		}
58		}

genkgo / mail

Pull Request — master (#42)

Partial::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like