Yandex\Webmaster\Models\Hosts

Complexity

Total Complexity

4

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	2
Bugs	1	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	2
A	add()	0	13	1
A	getAll()	0	4	1

<?php
namespace Yandex\Webmaster\Models;

use Yandex\Common\ObjectModel;

/**
 * Class Hosts
 *
 * @package Yandex\Webmaster\Models
 */
class Hosts extends ObjectModel
{
    /**
     * @var array $collection a collection of Host objects
     */
    protected $collection = [];

    /**
     * @var array $mappingClasses
     */
    protected $mappingClasses = [];

    /**
     * @var array $propNameMap
     */
    protected $propNameMap = [];

    /**
     * Hosts constructor.
     *
     * @param \SimpleXMLIterator $data
     */
    public function __construct(\SimpleXMLIterator $data)
    {
        for ($data->rewind(); $data->valid(); $data->next()) {
            $this->add($data->current());
        }

        return $this;

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

    }

    /**
     * Add item
     *
     * @param \SimpleXMLIterator $hostXmlNode
     * @return $this
     */
    public function add(\SimpleXMLIterator $hostXmlNode)
    {
        $host = new Host($hostXmlNode);

        $hostPathArray = explode('/', parse_url($host->getHref(), PHP_URL_PATH));
        $hostId = array_pop($hostPathArray);

        $host->setId($hostId);

        $this->collection[] = $host;

        return $this;
    }

    /**
     * Get items
     *
     * @return array
     */
    public function getAll()
    {
        return $this->collection;
    }
}