Bavix\Iterator\Traits\ArrayAccess

Complexity

Total Complexity

5

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	offsetExists()	0	3	1
A	offsetUnset()	0	3	1
A	offsetGet()	0	3	1
A	offsetSet()	0	10	2

<?php

namespace Bavix\Iterator\Traits;

trait ArrayAccess
{

    /**
     * @inheritdoc
     * @param string $offset
     */
    public function offsetExists($offset)
    {
        return isset($this->data[$offset]);
    }

    /**
     * @inheritdoc
     * @param string $offset
     */
    public function offsetGet($offset)
    {
        return $this->data[$offset];
    }

    /**
     * @inheritdoc
     * @param string $offset
     */
    public function offsetSet($offset, $value)
    {
        if ($offset === null)

The condition $offset === null is always false.

        {
            $this->data[] = $value;

            return;
        }

        $this->data[$offset] = $value;

The property data does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    /**
     * @inheritdoc
     */
    public function offsetUnset($offset)
    {
        unset($this->data[$offset]);
    }

}