Encoder::encode() - Code Metrics - Inspection of "Create specific exceptions" - AlexMasterov/msgpack.php - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( 9ec3c0...c630a6 )

by Alex

created 2018-03-13 06:41 UTC

Encoder::encode() C

↳ Parent: Encoder

Complexity

Conditions	11
Paths	11

Size

Total Lines	33
Code Lines	22

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	22
CRAP Score	11

Importance

Changes

Metric	Value
cc	11
eloc	22
nc	11
nop	1
dl	0
loc	33
ccs	22
cts	22
cp	1
crap	11
rs	5.2653
c	0
b	0
f	0

How to fix Complexity

<?php
declare(strict_types=1);

namespace MessagePack;

use MessagePack\{
    Exception\UnsupportedType,
    Ext
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php
};
use const MessagePack\CHR;
use function array_values;
use function count;
use function is_array;
use function is_bool;
use function is_float;
use function is_int;
use function is_string;
use function pack;
use function strlen;

final class Encoder
{
    public const FORCE_AUTO = 0b00;
    public const FORCE_ARR = 0b01;
    public const FORCE_MAP = 0b10;

    /** @var int */
    private $typeDetectionMode = self::FORCE_AUTO;

    public function __construct(int $typeDetectionMode = null)
    {
        if (null !== $typeDetectionMode) {
            $this->typeDetectionMode |= $typeDetectionMode;
        }
    }

    public function encode($value): string
    {
        if (is_int($value)) {
            return $this->encodeInt($value);
        }
        if (is_string($value)) {
            return $this->encodeStr($value);
        }
        if (is_array($value)) {
            if ($this->typeDetectionMode ^ self::FORCE_AUTO) {
                return $this->typeDetectionMode & self::FORCE_MAP
                    ? $this->encodeMap($value)
                    : $this->encodeArray($value);
            }
            return array_values($value) === $value
                ? $this->encodeArray($value)
                : $this->encodeMap($value);
        }
        if (is_float($value)) {
            return $this->encodeFloat($value);
        }
        if (is_bool($value)) {
            return $this->encodeBool($value);
        }
        if (null === $value) {
            return $this->encodeNil();
        }
        if ($value instanceof Ext) {
            return $this->encodeExt($value);
        }

        throw UnsupportedType::withValue($value);
    }

    public function encodeNil(): string
    {
        return "\xc0";
    }

    public function encodeBool(bool $value): string
    {
        return $value ? "\xc3" : "\xc2";
    }

    public function encodeFloat(float $num): string
    {
        $b1 = pack('E', $num);

        return "\xcb${b1}";
    }

    public function encodeInt(int $num): string
    {
        if ($num >= 0) {
            // positive fixint
            if ($num <= 0x7f) {
                return CHR[$num];
            }
            // uint 8
            if ($num <= 0xff) {
                $b1 = CHR[$num];
                return "\xcc${b1}";
            }
            // uint 16
            if ($num <= 0xffff) {
                $b1 = CHR[$num >> 8];
                $b2 = CHR[$num & 0xff];
                return "\xcd${b1}${b2}";
            }
            // uint 32
            if ($num <= 0xffffffff) {
                $b1 = CHR[$num >> 24];
                $b2 = CHR[$num >> 16 & 0xff];
                $b3 = CHR[$num >> 8 & 0xff];
                $b4 = CHR[$num & 0xff];
                return "\xce${b1}${b2}${b3}${b4}";
            }
            // uint 64
            return self::encodeUint64("\xcf", $num);
        }
        // negative fixint
        if ($num >= -0x20) {
            return CHR[$num & 0xff];
        }
        // int 8
        if ($num >= -0x80) {
            $b1 = CHR[$num & 0xff];
            return "\xd0${b1}";
        }
        // int 16
        if ($num >= -0x8000) {
            $b1 = CHR[$num >> 8 & 0xff];
            $b2 = CHR[$num & 0xff];
            return "\xd1${b1}${b2}";
        }
        // int 32
        if ($num >= -0x80000000) {
            $num |= 0x100000000;
            $b1 = CHR[$num >> 24 & 0xff];
            $b2 = CHR[$num >> 16 & 0xff];
            $b3 = CHR[$num >> 8 & 0xff];
            $b4 = CHR[$num & 0xff];
            return "\xd2${b1}${b2}${b3}${b4}";
        }
        // int 64
        return self::encodeUint64("\xd3", $num);
    }

    public function encodeStr(string $str): string
    {
        $len = strlen($str);
        // fixstr
        if ($len < 0x20) {
            $b0 = CHR[0xa0 | $len];
            return "${b0}${str}";
        }
        // str 8
        if ($len <= 0xff) {
            $b1 = CHR[$len];
            return "\xd9${b1}${str}";
        }
        // str 16
        if ($len <= 0xffff) {
            $b1 = CHR[$len >> 8];
            $b2 = CHR[$len & 0xff];
            return "\xda${b1}${b2}${str}";
        }
        // str 32
        $b1 = CHR[$len >> 24];
        $b2 = CHR[$len >> 16];
        $b3 = CHR[$len >> 8];
        $b4 = CHR[$len & 0xff];
        return "\xdb${b1}${b2}${b3}${b4}${str}";
    }

    public function encodeArray(array $array): string
    {
        $size = count($array);
        $data = self::encodeArrayHeader($size);

        foreach ($array as $value) {
            $data .= $this->encode($value);
        }

        return $data;
    }

    public function encodeMap(array $map): string
    {
        $size = count($map);
        $data = self::encodeMapHeader($size);

        foreach ($map as $key => $value) {
            $data .= "{$this->encode($key)}{$this->encode($value)}";
        }

        return $data;
    }

    public function encodeExt(Ext $ext): string
    {
        $type = CHR[$ext->type()];
        $data = $ext->data();

        $len = strlen($data);

        // fixext 1/2/4/8/16
        switch ($len) {
            case 1: return "\xd4${type}${data}";
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 2: return "\xd5${type}${data}";
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 4: return "\xd6${type}${data}";
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 8: return "\xd7${type}${data}";
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 16: return "\xd8${type}${data}";
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
        }
        // ext 8
        if ($len <= 0xff) {
            $b1 = CHR[$len];
            return "\xc7${b1}${type}${data}";
        }
        // ext 16
        if ($len <= 0xffff) {
            $b2 = CHR[$len & 0xff];
            $b1 = CHR[$len >> 8];
            return "\xc8${b1}${b2}${type}${data}";
        }
        // ext 32
        $b1 = CHR[$len >> 24 & 0xff];
        $b2 = CHR[$len >> 16 & 0xff];
        $b3 = CHR[$len >> 8 & 0xff];
        $b4 = CHR[$len & 0xff];
        return "\xc9${b1}${b2}${b3}${b4}${type}${data}";
    }

    private static function encodeArrayHeader(int $size): string
    {
        // fixarray
        if ($size <= 0xf) {
            return CHR[0x90 | $size];
        }
        // array 16
        if ($size <= 0xffff) {
            $b1 = CHR[$size >> 8];
            $b2 = CHR[$size & 0xff];
            return "\xdc${b1}${b2}";
        }
        // array 32
        $b1 = CHR[$size >> 24];
        $b2 = CHR[$size >> 16];
        $b3 = CHR[$size >> 8];
        $b4 = CHR[$size & 0xff];
        return "\xdd${b1}${b2}${b3}${b4}";
    }

    private static function encodeMapHeader(int $size): string
    {
        // fixmap
        if ($size <= 0xf) {
            return CHR[0x80 | $size];
        }
        // map 16
        if ($size <= 0xffff) {
            $b1 = CHR[$size >> 8];
            $b2 = CHR[$size & 0xff];
            return "\xde${b1}${b2}";
        }
        // map 32
        $b1 = CHR[$size >> 24];
        $b2 = CHR[$size >> 16];
        $b3 = CHR[$size >> 8];
        $b4 = CHR[$size & 0xff];
        return "\xdf${b1}${b2}${b3}${b4}";
    }

    private static function encodeUint64(string $byte, int $value): string
    {
        $hi = ($value & 0xffffffff00000000) >> 32;
        $lo = $value & 0x00000000ffffffff;

        $b1 = CHR[$hi >> 24 & 0xff];
        $b2 = CHR[$hi >> 16 & 0xff];
        $b3 = CHR[$hi >> 8 & 0xff];
        $b4 = CHR[$hi & 0xff];

        $b5 = CHR[$lo >> 24 & 0xff];
        $b6 = CHR[$lo >> 16 & 0xff];
        $b7 = CHR[$lo >> 8 & 0xff];
        $b8 = CHR[$lo & 0xff];

        return "${byte}${b1}${b2}${b3}${b4}${b5}${b6}${b7}${b8}";
    }
}


1		<?php
2		declare(strict_types=1);
3
4		namespace MessagePack;
5
6		use MessagePack\{
7		Exception\UnsupportedType,
8		Ext
		0 ignored issues – show Bug introduced 2018-03-13 07:22 UTC by Report Bug Copy Issue Report This use statement conflicts with another class in this namespace, `MessagePack\Ext`. Let’s assume that you have a directory layout like this: . \|-- OtherDir \| \|-- Bar.php \| `-- Foo.php `-- SomeDir `-- Foo.php and let’s assume the following content of `Bar.php`: // Bar.php namespace OtherDir; use SomeDir\Foo; // This now conflicts the class OtherDir\Foo If both files `OtherDir/Foo.php` and `SomeDir/Foo.php` are loaded in the same runtime, you will see a PHP error such as the following: PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php However, as `OtherDir/Foo.php` does not necessarily have to be loaded and the error is only triggered if it is loaded before `OtherDir/Bar.php`, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias: // Bar.php namespace OtherDir; use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore. Loading history...
9		};
10		use const MessagePack\CHR;
11		use function array_values;
12		use function count;
13		use function is_array;
14		use function is_bool;
15		use function is_float;
16		use function is_int;
17		use function is_string;
18		use function pack;
19		use function strlen;
20
21		final class Encoder
22		{
23		public const FORCE_AUTO = 0b00;
24		public const FORCE_ARR = 0b01;
25		public const FORCE_MAP = 0b10;
26
27		/** @var int */
28		private $typeDetectionMode = self::FORCE_AUTO;
29
30	65	public function __construct(int $typeDetectionMode = null)
31		{
32	65	if (null !== $typeDetectionMode) {
33	5	$this->typeDetectionMode \|= $typeDetectionMode;
34		}
35	65	}
36
37	65	public function encode($value): string
38		{
39	65	if (is_int($value)) {
40	35	return $this->encodeInt($value);
41		}
42	45	if (is_string($value)) {
43	7	return $this->encodeStr($value);
44		}
45	38	if (is_array($value)) {
46	16	if ($this->typeDetectionMode ^ self::FORCE_AUTO) {
47	3	return $this->typeDetectionMode & self::FORCE_MAP
48	2	? $this->encodeMap($value)
49	3	: $this->encodeArray($value);
50		}
51	13	return array_values($value) === $value
52	8	? $this->encodeArray($value)
53	13	: $this->encodeMap($value);
54		}
55	22	if (is_float($value)) {
56	6	return $this->encodeFloat($value);
57		}
58	16	if (is_bool($value)) {
59	3	return $this->encodeBool($value);
60		}
61	13	if (null === $value) {
62	1	return $this->encodeNil();
63		}
64	12	if ($value instanceof Ext) {
65	9	return $this->encodeExt($value);
66		}
67
68	3	throw UnsupportedType::withValue($value);
69		}
70
71	1	public function encodeNil(): string
72		{
73	1	return "\xc0";
74		}
75
76	3	public function encodeBool(bool $value): string
77		{
78	3	return $value ? "\xc3" : "\xc2";
79		}
80
81	6	public function encodeFloat(float $num): string
82		{
83	6	$b1 = pack('E', $num);
84
85	6	return "\xcb${b1}";
86		}
87
88		public function encodeInt(int $num): string
89		{
90	35	if ($num >= 0) {
91		// positive fixint
92	25	if ($num <= 0x7f) {
93	17	return CHR[$num];
94		}
95		// uint 8
96	10	if ($num <= 0xff) {
97	4	$b1 = CHR[$num];
98	4	return "\xcc${b1}";
99		}
100		// uint 16
101	8	if ($num <= 0xffff) {
102	4	$b1 = CHR[$num >> 8];
103	4	$b2 = CHR[$num & 0xff];
104	4	return "\xcd${b1}${b2}";
105		}
106		// uint 32
107	5	if ($num <= 0xffffffff) {
108	3	$b1 = CHR[$num >> 24];
109	3	$b2 = CHR[$num >> 16 & 0xff];
110	3	$b3 = CHR[$num >> 8 & 0xff];
111	3	$b4 = CHR[$num & 0xff];
112	3	return "\xce${b1}${b2}${b3}${b4}";
113		}
114		// uint 64
115	2	return self::encodeUint64("\xcf", $num);
116		}
117		// negative fixint
118	10	if ($num >= -0x20) {
119	2	return CHR[$num & 0xff];
120		}
121		// int 8
122	8	if ($num >= -0x80) {
123	2	$b1 = CHR[$num & 0xff];
124	2	return "\xd0${b1}";
125		}
126		// int 16
127	6	if ($num >= -0x8000) {
128	2	$b1 = CHR[$num >> 8 & 0xff];
129	2	$b2 = CHR[$num & 0xff];
130	2	return "\xd1${b1}${b2}";
131		}
132		// int 32
133	4	if ($num >= -0x80000000) {
134	2	$num \|= 0x100000000;
135	2	$b1 = CHR[$num >> 24 & 0xff];
136	2	$b2 = CHR[$num >> 16 & 0xff];
137	2	$b3 = CHR[$num >> 8 & 0xff];
138	2	$b4 = CHR[$num & 0xff];
139	2	return "\xd2${b1}${b2}${b3}${b4}";
140		}
141		// int 64
142	2	return self::encodeUint64("\xd3", $num);
143		}
144
145		public function encodeStr(string $str): string
146		{
147	7	$len = strlen($str);
148		// fixstr
149	7	if ($len < 0x20) {
150	2	$b0 = CHR[0xa0 \| $len];
151	2	return "${b0}${str}";
152		}
153		// str 8
154	5	if ($len <= 0xff) {
155	2	$b1 = CHR[$len];
156	2	return "\xd9${b1}${str}";
157		}
158		// str 16
159	3	if ($len <= 0xffff) {
160	2	$b1 = CHR[$len >> 8];
161	2	$b2 = CHR[$len & 0xff];
162	2	return "\xda${b1}${b2}${str}";
163		}
164		// str 32
165	1	$b1 = CHR[$len >> 24];
166	1	$b2 = CHR[$len >> 16];
167	1	$b3 = CHR[$len >> 8];
168	1	$b4 = CHR[$len & 0xff];
169	1	return "\xdb${b1}${b2}${b3}${b4}${str}";
170		}
171
172		public function encodeArray(array $array): string
173		{
174	9	$size = count($array);
175	9	$data = self::encodeArrayHeader($size);
176
177	9	foreach ($array as $value) {
178	8	$data .= $this->encode($value);
179		}
180
181	9	return $data;
182		}
183
184		public function encodeMap(array $map): string
185		{
186	7	$size = count($map);
187	7	$data = self::encodeMapHeader($size);
188
189	7	foreach ($map as $key => $value) {
190	7	$data .= "{$this->encode($key)}{$this->encode($value)}";
191		}
192
193	7	return $data;
194		}
195
196		public function encodeExt(Ext $ext): string
197		{
198	9	$type = CHR[$ext->type()];
199	9	$data = $ext->data();
200
201	9	$len = strlen($data);
202
203		// fixext 1/2/4/8/16
204	9	switch ($len) {
205	9	case 1: return "\xd4${type}${data}";
		0 ignored issues – show Coding Style introduced 2018-03-11 23:51 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history... Coding Style introduced 2018-03-11 23:51 UTC by Report Bug Copy Issue Report Terminating statement must be on a line by itself As per the PSR-2 coding standard, the `break` (or other terminating) statement must be on a line of its own. switch ($expr) { case "A": doSomething(); break; //wrong case "B": doSomething(); break; //right case "C:": doSomething(); return true; //right } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
206	8	case 2: return "\xd5${type}${data}";
		0 ignored issues – show Coding Style introduced 2018-03-11 23:51 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history... Coding Style introduced 2018-03-11 23:51 UTC by Report Bug Copy Issue Report Terminating statement must be on a line by itself As per the PSR-2 coding standard, the `break` (or other terminating) statement must be on a line of its own. switch ($expr) { case "A": doSomething(); break; //wrong case "B": doSomething(); break; //right case "C:": doSomething(); return true; //right } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
207	7	case 4: return "\xd6${type}${data}";
		0 ignored issues – show Coding Style introduced 2018-03-11 23:51 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history... Coding Style introduced 2018-03-11 23:51 UTC by Report Bug Copy Issue Report Terminating statement must be on a line by itself As per the PSR-2 coding standard, the `break` (or other terminating) statement must be on a line of its own. switch ($expr) { case "A": doSomething(); break; //wrong case "B": doSomething(); break; //right case "C:": doSomething(); return true; //right } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
208	6	case 8: return "\xd7${type}${data}";
		0 ignored issues – show Coding Style introduced 2018-03-11 23:51 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history... Coding Style introduced 2018-03-11 23:51 UTC by Report Bug Copy Issue Report Terminating statement must be on a line by itself As per the PSR-2 coding standard, the `break` (or other terminating) statement must be on a line of its own. switch ($expr) { case "A": doSomething(); break; //wrong case "B": doSomething(); break; //right case "C:": doSomething(); return true; //right } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
209	5	case 16: return "\xd8${type}${data}";
		0 ignored issues – show Coding Style introduced 2018-03-11 23:51 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history... Coding Style introduced 2018-03-11 23:51 UTC by Report Bug Copy Issue Report Terminating statement must be on a line by itself As per the PSR-2 coding standard, the `break` (or other terminating) statement must be on a line of its own. switch ($expr) { case "A": doSomething(); break; //wrong case "B": doSomething(); break; //right case "C:": doSomething(); return true; //right } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
210		}
211		// ext 8
212	4	if ($len <= 0xff) {
213	1	$b1 = CHR[$len];
214	1	return "\xc7${b1}${type}${data}";
215		}
216		// ext 16
217	3	if ($len <= 0xffff) {
218	2	$b2 = CHR[$len & 0xff];
219	2	$b1 = CHR[$len >> 8];
220	2	return "\xc8${b1}${b2}${type}${data}";
221		}
222		// ext 32
223	1	$b1 = CHR[$len >> 24 & 0xff];
224	1	$b2 = CHR[$len >> 16 & 0xff];
225	1	$b3 = CHR[$len >> 8 & 0xff];
226	1	$b4 = CHR[$len & 0xff];
227	1	return "\xc9${b1}${b2}${b3}${b4}${type}${data}";
228		}
229
230		private static function encodeArrayHeader(int $size): string
231		{
232		// fixarray
233	9	if ($size <= 0xf) {
234	6	return CHR[0x90 \| $size];
235		}
236		// array 16
237	3	if ($size <= 0xffff) {
238	2	$b1 = CHR[$size >> 8];
239	2	$b2 = CHR[$size & 0xff];
240	2	return "\xdc${b1}${b2}";
241		}
242		// array 32
243	1	$b1 = CHR[$size >> 24];
244	1	$b2 = CHR[$size >> 16];
245	1	$b3 = CHR[$size >> 8];
246	1	$b4 = CHR[$size & 0xff];
247	1	return "\xdd${b1}${b2}${b3}${b4}";
248		}
249
250		private static function encodeMapHeader(int $size): string
251		{
252		// fixmap
253	7	if ($size <= 0xf) {
254	4	return CHR[0x80 \| $size];
255		}
256		// map 16
257	3	if ($size <= 0xffff) {
258	2	$b1 = CHR[$size >> 8];
259	2	$b2 = CHR[$size & 0xff];
260	2	return "\xde${b1}${b2}";
261		}
262		// map 32
263	1	$b1 = CHR[$size >> 24];
264	1	$b2 = CHR[$size >> 16];
265	1	$b3 = CHR[$size >> 8];
266	1	$b4 = CHR[$size & 0xff];
267	1	return "\xdf${b1}${b2}${b3}${b4}";
268		}
269
270		private static function encodeUint64(string $byte, int $value): string
271		{
272	4	$hi = ($value & 0xffffffff00000000) >> 32;
273	4	$lo = $value & 0x00000000ffffffff;
274
275	4	$b1 = CHR[$hi >> 24 & 0xff];
276	4	$b2 = CHR[$hi >> 16 & 0xff];
277	4	$b3 = CHR[$hi >> 8 & 0xff];
278	4	$b4 = CHR[$hi & 0xff];
279
280	4	$b5 = CHR[$lo >> 24 & 0xff];
281	4	$b6 = CHR[$lo >> 16 & 0xff];
282	4	$b7 = CHR[$lo >> 8 & 0xff];
283	4	$b8 = CHR[$lo & 0xff];
284
285	4	return "${byte}${b1}${b2}${b3}${b4}${b5}${b6}${b7}${b8}";
286		}
287		}
288

AlexMasterov / msgpack.php

Push — master ( 9ec3c0...c630a6 )

Encoder::encode() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like