Innmind\AMQP\Transport\Frame\Value\ShortString

Complexity

Total Complexity

4

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	fromStream()	0	5	1
A	__toString()	0	3	1
A	original()	0	3	1

<?php
declare(strict_types = 1);

namespace Innmind\AMQP\Transport\Frame\Value;

use Innmind\AMQP\Transport\Frame\Value;
use Innmind\Math\Algebra\Integer;
use Innmind\Stream\Readable;
use Innmind\Immutable\Str;

final class ShortString implements Value
{
    private $value;
    private $original;

    public function __construct(Str $string)
    {
        $this->original = $string;
        $string = $string->toEncoding('ASCII');
        $this->value = (string) new UnsignedOctet(
            new Integer($string->length())
        );
        $this->value .= $string;
    }

    public static function fromStream(Readable $stream): Value
    {
        $length = UnsignedOctet::fromStream($stream)->original();

        return new self($stream->read($length->value()));

It seems like $length->value() can also be of type double; however, parameter $length of Innmind\Stream\Readable::read() does only seem to accept integer|null, maybe add an additional type check?

    }

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

    public function __toString(): string
    {
        return $this->value;
    }
}