Scriptotek\GoogleBooks\Model

Complexity

Total Complexity

13

Size/Duplication

Total Lines	104
Duplicated Lines	21.15 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	setData()	0	4	1
A	isSearchResult()	0	4	1
A	expandToFullRecord()	0	4	1
A	getIterator()	0	3	1
A	get()	11	11	3
A	has()	11	11	3
A	__toString()	0	4	1
A	__get()	0	4	1

<?php

namespace Scriptotek\GoogleBooks;


abstract class Model implements \IteratorAggregate
{
    protected $client;
    private $_data;

    public function __construct(GoogleBooks $client, $data)
    {
        $this->client = $client;
        $this->setData($data);
    }

    /**
     * Set data for this model
     */
    protected function setData(\stdClass $data)
    {
        $this->_data = $data;
    }

    /**
     * Returns true if the model is created from a search result response
     * (and thus do not contain all the data of the full record).
     *
     * @return bool
     */
    public function isSearchResult()
    {
        return $this->has('searchInfo');
    }

    /**
     * Expand a search result response object to a full record.
     */
    public function expandToFullRecord()
    {
        $this->setData($this->client->getItem($this->client->removeBaseUrl($this->selfLink)));

The property selfLink does not exist on object<Scriptotek\GoogleBooks\Model>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * Special method that allows the object to be iterated over, for example
     * with a foreach statement.
     */
    public function getIterator() {
        return new \ArrayIterator($this->_data);
    }

    /**
     * Get an item from an array using "dot" notation.
     *
     * @param  string  $key
     * @param  mixed   $default
     * @return mixed
     */
    public function get($key, $default = null)

This method seems to be duplicated in your project.

    {
        $current = $this->_data;
        foreach (explode('.', $key) as $segment) {
            if (!isset($current->{$segment})) {
                return $default;
            }
            $current = $current->{$segment};
        }
        return $current;
    }

    /**
     * Check if an item or items exist in an array using "dot" notation.
     *
     * @param  string  $key
     * @return mixed
     */
    public function has($key)

This method seems to be duplicated in your project.

    {
        $current = $this->_data;
        foreach (explode('.', $key) as $segment) {
            if (!isset($current->{$segment})) {
                return false;
            }
            $current = $current->{$segment};
        }
        return true;
    }

    /**
     * Get a string representation of the object
     *
     * @return string
     */
    public function __toString()
    {
        return json_encode($this);
    }

    /**
     * Provide object-like access to the data.
     *
     * @param  string  $key
     * @return mixed
     */
    public function __get($key)
    {
        return $this->get($key);
    }
}