Bouhnosaure\Dogecoin\ReadOnlyArrayTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	offsetSet()	0	4	1
A	offsetExists()	0	4	1
A	offsetUnset()	0	4	1
A	offsetGet()	0	5	2

<?php

namespace Bouhnosaure\Dogecoin;

trait ReadOnlyArrayTrait
{
    /**
     * Assigns a value to the specified offset.
     *
     * @param mixed $offset
     * @param mixed $value
     *
     * @return void
     */
    public function offsetSet($offset, $value)

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

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

    {
        throw new Exceptions\ClientException('Cannot modify readonly object');
    }

    /**
     * Whether or not an offset exists.
     *
     * @param mixed $offset
     *
     * @return bool
     */
    public function offsetExists($offset)
    {
        return isset($this->result()[$offset]);

It seems like result() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Unsets the offset.
     *
     * @param mixed $offset
     *
     * @return void
     */
    public function offsetUnset($offset)

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

    {
        throw new Exceptions\ClientException('Cannot modify readonly object');
    }

    /**
     * Returns the value at specified offset.
     *
     * @param mixed $offset
     *
     * @return mixed
     */
    public function offsetGet($offset)
    {
        return isset($this->result()[$offset]) ?

It seems like result() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->result()[$offset] : null;

It seems like result() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
}