Boundary::validate() - Code Metrics - Inspection of "add post, fix some more CS (declare calls)" - genkgo/mail - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#48)

by Frederik

created 2018-03-06 17:30 UTC

Boundary::validate() A

↳ Parent: Boundary

Complexity

Conditions	4
Paths	4

Size

Total Lines	17
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	4.0218

Importance

Changes

Metric	Value
dl	0
loc	17
ccs	8
cts	9
cp	0.8889
rs	9.2
c	0
b	0
f	0
cc	4
eloc	9
nc	4
nop	1
crap	4.0218

<?php
declare(strict_types=1);

namespace Genkgo\Mail\Mime;

final class Boundary
{
    /**
     * @var string
     */
    private $identifier;

    /**
     * @param string $identifier
     */
    public function __construct(string $identifier)
    {
        $this->assert($identifier);

        $this->identifier = $identifier;
    }

    /**
     * @return string
     */
    public function __toString(): string
    {
        return $this->identifier;
    }

    /**
     * @param string $line
     * @return bool
     */
    public function isOpening(string $line): bool
    {
        return '--' . $this->identifier === $line;
    }

    /**
     * @param string $line
     * @return bool
     */
    public function isClosing(string $line): bool
    {
        return '--' . $this->identifier . '--' === $line;
    }

    /**
     * @return Boundary
     */
    public static function newRandom()
    {
        return new self('GenkgoMailV2Part' . \bin2hex(\random_bytes(6)));
    }

    /**
     * @param string $value
     */
    private function assert(string $value): void
    {
        if ($this->validate($value) === false) {
            throw new \InvalidArgumentException('Invalid header value ' . $value);
        }
    }

    /**
     * @param string $value
     * @return bool
     */
    private function validate(string $value): bool
    {
        if ($value === '') {
            return false;
        }

        $length = \strlen($value);
        if ($length > 70) {
            return false;
        }

        if ($value[$length - 1] === ' ') {
            return false;
        }

        return \preg_match('/[^A-Za-z0-9\'\,\(\)\+\_\,\-\.\/\:\=\? ]/', $value) !== 1;
    }
}


1		<?php
2		declare(strict_types=1);
3
4		namespace Genkgo\Mail\Mime;
5
6		final class Boundary
7		{
8		/**
9		* @var string
10		*/
11		private $identifier;
12
13		/**
14		* @param string $identifier
15		*/
16	25	public function __construct(string $identifier)
17		{
18	25	$this->assert($identifier);
19
20	19	$this->identifier = $identifier;
21	19	}
22
23		/**
24		* @return string
25		*/
26	16	public function __toString(): string
27		{
28	16	return $this->identifier;
29		}
30
31		/**
32		* @param string $line
33		* @return bool
34		*/
35	5	public function isOpening(string $line): bool
36		{
37	5	return '--' . $this->identifier === $line;
38		}
39
40		/**
41		* @param string $line
42		* @return bool
43		*/
44	5	public function isClosing(string $line): bool
45		{
46	5	return '--' . $this->identifier . '--' === $line;
47		}
48
49		/**
50		* @return Boundary
51		*/
52	4	public static function newRandom()
53		{
54	4	return new self('GenkgoMailV2Part' . \bin2hex(\random_bytes(6)));
55		}
56
57		/**
58		* @param string $value
59		*/
60	25	private function assert(string $value): void
61		{
62	25	if ($this->validate($value) === false) {
63	6	throw new \InvalidArgumentException('Invalid header value ' . $value);
64		}
65	19	}
66
67		/**
68		* @param string $value
69		* @return bool
70		*/
71	25	private function validate(string $value): bool
72		{
73	25	if ($value === '') {
74		return false;
75		}
76
77	25	$length = \strlen($value);
78	25	if ($length > 70) {
79	1	return false;
80		}
81
82	24	if ($value[$length - 1] === ' ') {
83	1	return false;
84		}
85
86	23	return \preg_match('/[^A-Za-z0-9\'\,\(\)\+\_\,\-\.\/\:\=\? ]/', $value) !== 1;
87		}
88		}
89

genkgo / mail

Pull Request — master (#48)

Boundary::validate() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like