Cmobi\RabbitmqBundle\Routing\MethodCollection

Complexity

Total Complexity

15

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	3
Bugs	2	Features	1

10 Methods

Rating	Name	Duplication	Size	Complexity
A	get()	0	8	2
A	remove()	0	6	2
A	addCollection()	0	11	2
A	getResources()	0	4	1
A	addResource()	0	4	1
A	getIterator()	0	4	1
A	count()	0	4	1
A	add()	0	6	1
A	all()	0	4	1
A	addDefaults()	0	11	3

<?php

namespace Cmobi\RabbitmqBundle\Routing;

use Symfony\Component\Config\Resource\ResourceInterface;

class MethodCollection implements \IteratorAggregate, \Countable
{
    private $methods = [];
    private  $resources = [];

    public function getIterator()
    {
        return new \ArrayIterator($this->methods);
    }

    public function count()
    {
        return count($this->methods);
    }

    public function add($name, Method $method)
    {
        unset($this->methods[$name]);

        $this->methods[$name] = $method;
    }

    public function all()
    {
        return $this->methods;
    }


    public function get($name)
    {
        if (isset($this->methods[$name])) {
            return $this->methods[$name];
        }

        return null;
    }

    /**
     * Removes a route or an array of routes by name from the collection.
     *
     * @param string|array $name The route name or an array of route names
     */
    public function remove($name)
    {
        foreach ((array) $name as $n) {
            unset($this->methods[$n]);
        }
    }

    public function addCollection(MethodCollection $collection)
    {
        // we need to remove all routes with the same names first because just replacing them
        // would not place the new route at the end of the merged array
        foreach ($collection->all() as $name => $route) {
            unset($this->methods[$name]);
            $this->methods[$name] = $route;
        }

        $this->resources = array_merge($this->resources, $collection->getResources());
    }

    public function getResources()
    {
        return array_unique($this->resources);
    }

    public function addResource(ResourceInterface $resource)
    {
        $this->resources[] = $resource;
    }

    public function addDefaults(array $defaults)
    {
        if ($defaults) {

The expression $defaults of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            /**
             * @var Method $method
             */
            foreach ($this->methods as $method) {
                $method->addDefaults($defaults);
            }
        }
    }
}