PhpProfiler\Lib\ByteStream\CDataByteReader

Complexity

Total Complexity

6

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	createSliceAsString()	0	7	2
A	offsetGet()	0	3	1
A	offsetExists()	0	6	2
A	__construct()	0	3	1

<?php

/**
 * This file is part of the sj-i/php-profiler package.
 *
 * (c) sji <sji@sj-i.dev>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace PhpProfiler\Lib\ByteStream;

use FFI\CArray;
use FFI\CData;

final class CDataByteReader implements ByteReaderInterface
{
    use ByteReaderDisableWriteAccessTrait;

    /** @var CArray */
    private CData $source;

    /**
     * CDataByteReader constructor.
     * @param CArray $source
     */
    public function __construct(CData $source)
    {
        $this->source = $source;

$source is of type FFI\CData, but the property $source was declared to be of type FFI\CArray. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    public function offsetExists($offset): bool
    {
        if (count($this->source) <= $offset) {

$this->source of type FFI\CData is incompatible with the type Countable|array expected by parameter $var of count().

            return false;
        }
        return !is_null($this->source[$offset]);
    }

    public function offsetGet($offset): int
    {
        return $this->source[$offset];
    }

    public function createSliceAsString(int $offset, int $size): string
    {
        $result = '';
        for ($i = $offset, $last_offset = $offset + $size; $i < $last_offset; $i++) {
            $result .= chr($this->source[$i]);
        }
        return $result;
    }
}