Hoooklife\DynamodbPodm\Collection

Complexity

Total Complexity

12

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	offsetUnset()	0	3	1
A	__get()	0	4	1
A	__construct()	0	3	1
A	getMarshaler()	0	6	2
A	offsetGet()	0	6	2
A	offsetExists()	0	3	1
A	toArray()	0	8	2
A	offsetSet()	0	3	1
A	first()	0	4	1

<?php
/**
 * Created by PhpStorm.
 * User: hooklife
 * Date: 2018/12/5
 * Time: 下午4:59
 */

namespace Hoooklife\DynamodbPodm;


use Aws\DynamoDb\Marshaler;
use Aws\Result;

class Collection implements \ArrayAccess
{
    /** @var Result $data */
    private $data;
    private $marshaler;

    public function __construct($data)
    {
        $this->data = $data;

    }

    public function __get($var)
    {
        var_dump(1);

var_dump(1) looks like debug code. Are you sure you do not want to remove it?

        die;

Using exit here is not recommended.

    }

    private function getMarshaler()
    {
        if (!$this->marshaler) {
            $this->marshaler = new Marshaler();
        }
        return $this->marshaler;

    }


    public function offsetExists($offset)
    {
        return isset($this->data[$offset]);
    }


    public function offsetGet($offset)
    {
        if (isset($this->data[$offset])) {
            return $this->getMarshaler()->unmarshalItem($this->data[$offset]);
        }
        return null;
    }


    public function offsetSet($offset, $value)
    {
        $this->data[$offset] = $value;
    }


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

    public function toArray()
    {

        $result = [];
        foreach ($this->data["Items"] as $item) {
            $result[] = $this->getMarshaler()->unmarshalItem($item);
        }
        return $result;
    }

    public function first()
    {
        $item = reset($this->data["Items"]);

It seems like $this->data['Items'] can also be of type null; however, parameter $array of reset() does only seem to accept array, maybe add an additional type check?

        return $this->getMarshaler()->unmarshalItem($item);
    }
}