JonnyW\PhantomJs\Procedure\ChainProcedureLoader

Complexity

Total Complexity

8

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	addLoader()	0	4	1
A	load()	0	17	3
A	loadTemplate()	0	17	3

<?php

/*
 * This file is part of the php-phantomjs.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace JonnyW\PhantomJs\Procedure;

/**
 * PHP PhantomJs
 *
 * @author Jon Wenmoth <contact@jonnyw.me>
 */
class ChainProcedureLoader implements ProcedureLoaderInterface
{
    /**
     * Procedure loader storage.
     *
     * @var array
     * @access protected
     */
    protected $procedureLoaders;

    /**
     * Internal constructor.
     *
     * @access public
     * @param array $procedureLoaders
     */
    public function __construct(array $procedureLoaders)
    {
        $this->procedureLoaders = $procedureLoaders;
    }

    /**
     * Add procedure loader.
     *
     * @access public
     * @param  \JonnyW\PhantomJs\Procedure\ProcedureLoaderInterface $procedureLoader
     * @return void
     */
    public function addLoader(ProcedureLoaderInterface $procedureLoader)
    {
        array_unshift($this->procedureLoaders, $procedureLoader);
    }

    /**
     * Load procedure instance by id.
     *
     * @access public
     * @param  string                                         $id
     * @throws \InvalidArgumentException
     * @return \JonnyW\PhantomJs\Procedure\ProcedureInterface
     */
    public function load($id)
    {
        /** @var \JonnyW\PhantomJs\Procedure\ProcedureLoaderInterface $loader **/
        foreach ($this->procedureLoaders as $loader) {

            try {

                $procedure = $loader->load($id);

                return $procedure;

            } catch (\Exception $e) {}

Consider adding a comment why this CATCH block is empty.

        }

        throw new \InvalidArgumentException(sprintf('No valid procedure loader could be found to load the \'%s\' procedure.', $id));
    }

    /**
     * Load procedure template by id.
     *
     * @access public
     * @param  string                    $id
     * @param  string                    $extension (default: 'proc')
     * @throws \InvalidArgumentException
     * @return string
     */
    public function loadTemplate($id, $extension = 'proc')
    {
        /** @var \JonnyW\PhantomJs\Procedure\ProcedureLoaderInterface $loader **/
        foreach ($this->procedureLoaders as $loader) {

            try {

                $template = $loader->loadTemplate($id, $extension);

                return $template;

            } catch (\Exception $e) {}

Consider adding a comment why this CATCH block is empty.

        }

        throw new \InvalidArgumentException(sprintf('No valid procedure loader could be found to load the \'%s\' procedure template.', $id));
    }
}