TypedCData\TypedCDataArray

Complexity

Total Complexity

9

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	offsetUnset()	0	2	1
A	offsetGet()	0	6	2
A	__construct()	0	5	1
A	fromCData()	0	3	1
A	toCData()	0	7	2
A	offsetExists()	0	3	1
A	offsetSet()	0	3	1

<?php

/**
 * This file is part of the sj-i/typed-cdata 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 TypedCData;

use FFI\CData;

/**
 * @template T of TypedCDataInterface
 */
class TypedCDataArray implements \ArrayAccess
{
    /** @var \FFI\CDataArray */
    private CData $cdata;
    /** @var class-string<T> $typed_cdata_class */

The doc comment class-string<T> at position 0 could not be parsed: Unknown type name 'class-string' at position 0 in class-string<T>.

    private string $typed_cdata_class;
    /** @var array<int, T> */
    private array $typed_element_cache;

    /**
     * TypedCDataArray constructor.
     * @param \FFI\CDataArray $cdata
     * @param class-string<T> $typed_cdata_class

The doc comment class-string<T> at position 0 could not be parsed: Unknown type name 'class-string' at position 0 in class-string<T>.

     */
    public function __construct(CData $cdata, string $typed_cdata_class)
    {
        $this->cdata = $cdata;

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

        $this->typed_cdata_class = $typed_cdata_class;
        $this->typed_element_cache = [];
    }

    public function offsetExists($offset)
    {
        return true;
    }

    /**
     * @param int $offset
     */
    public function offsetGet($offset): TypedCDataInterface
    {
        if (!isset($this->typed_element_cache[$offset])) {
            $this->typed_element_cache[$offset] = $this->typed_cdata_class::fromCData($this->cdata[$offset]);
        }
        return $this->typed_element_cache[$offset];
    }

    /**
     * @param int $offset
     * @param TypedCDataInterface $value
     */
    public function offsetSet($offset, $value)
    {
        $value->toCData($this->cdata[$offset]);
    }

    public function offsetUnset($offset)
    {
    }

    /**
     * @template T2 of TypedCDataInterface
     * @param \FFI\CDataArray $cdata
     * @param class-string<T2> $typed_cdata_class

The doc comment class-string<T2> at position 0 could not be parsed: Unknown type name 'class-string' at position 0 in class-string<T2>.

     * @return self<T2>
     */
    public static function fromCData(CData $cdata, string $typed_cdata_class): TypedCDataArray
    {
        return new self($cdata, $typed_cdata_class);
    }

    public function toCData(?CData $cdata): CData

The parameter $cdata is not used and could be removed.

    {
        foreach ($this->typed_element_cache as $offset => $item) {
            assert($item instanceof TypedCDataInterface);
            $item->toCData($this->cdata[$offset]);
        }
        return $this->cdata;
    }
}