TreeHouse\WorkerBundle\ExecutorPool

Complexity

Total Complexity

6

Size/Duplication

Total Lines	71
Duplicated Lines	39.44 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	hasExecutor()	0	4	1
A	addExecutor()	15	15	2
A	getExecutor()	13	13	2
A	getExecutors()	0	4	1

<?php

namespace TreeHouse\WorkerBundle;

use InvalidArgumentException;
use OutOfBoundsException;
use TreeHouse\WorkerBundle\Executor\ExecutorInterface;

class ExecutorPool
{
    /**
     * @var ExecutorInterface[]
     */
    protected $executors = [];

    /**
     * @param string $action
     *
     * @return bool
     */
    public function hasExecutor($action)
    {
        return array_key_exists($action, $this->executors);
    }

    /**
     * Add an executor.
     *
     * @param ExecutorInterface $executor
     *
     * @throws InvalidArgumentException
     */
    public function addExecutor(ExecutorInterface $executor)

This method seems to be duplicated in your project.

    {
        $action = $executor->getName();

        if ($this->hasExecutor($action)) {
            throw new InvalidArgumentException(
                sprintf(
                    'There is already an executor registered for action "%s".',
                    $action
                )
            );
        }

        $this->executors[$action] = $executor;
    }

    /**
     * Returns a registered executor for given action.
     *
     * @param string $action
     *
     * @return ExecutorInterface
     *
     * @throws OutOfBoundsException
     */
    public function getExecutor($action)

This method seems to be duplicated in your project.

    {
        if (!$this->hasExecutor($action)) {
            throw new OutOfBoundsException(
                sprintf(
                    'There is no executor registered for action "%s".',
                    $action
                )
            );
        }

        return $this->executors[$action];
    }

    /**
     * @return ExecutorInterface[]
     */
    public function getExecutors()
    {
        return $this->executors;
    }
}