UnsignedLongInteger::fromStream() - Code Metrics - Inspection of "Performance improvement" - Innmind/AMQP - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — develop (#1)

by Baptiste

created 2019-02-10 18:30 UTC

UnsignedLongInteger::fromStream() A

↳ Parent: UnsignedLongInteger

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	1
dl	0
loc	5
ccs	3
cts	3
cp	1
crap	1
rs	10
c	0
b	0
f	0

<?php
declare(strict_types = 1);

namespace Innmind\AMQP\Transport\Frame\Value;

use Innmind\AMQP\{
    Transport\Frame\Value,
    Exception\OutOfRangeValue,
};
use Innmind\Math\{
    Algebra\Integer,
    DefinitionSet\Set,
    DefinitionSet\Range,
};
use Innmind\Stream\Readable;

final class UnsignedLongInteger implements Value
{
    private static $definitionSet;

    private $value;
    private $original;

    public function __construct(Integer $value)
    {
        if (!self::definitionSet()->contains($value)) {
            throw new OutOfRangeValue($value, self::definitionSet());
        }

        $this->original = $value;
    }

    public static function fromStream(Readable $stream): Value
    {
        [, $value] = unpack('N', (string) $stream->read(4));

        return new self(new Integer($value));
    }

    public function original(): Integer
    {
        return $this->original;
    }

    public function __toString(): string
    {
        return $this->value ?? $this->value = pack('N', $this->original->value());
    }

    public static function definitionSet(): Set
    {
        return self::$definitionSet ?? self::$definitionSet = Range::inclusive(
            new Integer(0),
            new Integer(4294967295)
        );
    }
}


1		<?php
2		declare(strict_types = 1);
3
4		namespace Innmind\AMQP\Transport\Frame\Value;
5
6		use Innmind\AMQP\{
7		Transport\Frame\Value,
8		Exception\OutOfRangeValue,
9		};
10		use Innmind\Math\{
11		Algebra\Integer,
12		DefinitionSet\Set,
13		DefinitionSet\Range,
14		};
15		use Innmind\Stream\Readable;
16
17		final class UnsignedLongInteger implements Value
18		{
19		private static $definitionSet;
20
21		private $value;
22		private $original;
23
24	135	public function __construct(Integer $value)
25		{
26	135	if (!self::definitionSet()->contains($value)) {
27	2	throw new OutOfRangeValue($value, self::definitionSet());
28		}
29
30	133	$this->original = $value;
31	133	}
32
33	76	public static function fromStream(Readable $stream): Value
34		{
35	76	[, $value] = unpack('N', (string) $stream->read(4));
36
37	76	return new self(new Integer($value));
38		}
39
40	68	public function original(): Integer
41		{
42	68	return $this->original;
43		}
44
45	132	public function __toString(): string
46		{
47	132	return $this->value ?? $this->value = pack('N', $this->original->value());
48		}
49
50	135	public static function definitionSet(): Set
51		{
52	135	return self::$definitionSet ?? self::$definitionSet = Range::inclusive(
53		new Integer(0),
54	135	new Integer(4294967295)
55		);
56		}
57		}
58

Innmind / AMQP

GitHub Access Token became invalid

Pull Request — develop (#1)

UnsignedLongInteger::fromStream() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like