Innmind\AMQP\Transport\Frame\Value\LongString::fromString()

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	LongString::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 LongString implements Value
{
    private $value;
    private $original;

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

    public static function fromStream(Readable $stream): Value
    {
        $length = UnsignedLongInteger::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;
    }
}