MJanssen\Route\Factory

Complexity

Total Complexity

10

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	7

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
D	fromArray()	0	32	10

<?php

namespace MJanssen\Route;

use InvalidArgumentException;

class Factory
{
    /**
     * @param array $route
     *
     * @return Route
     */
    public static function fromArray(array $route)
    {
        if (!isset($route['pattern'])) {
            throw new InvalidArgumentException('Required parameter pattern is not set.');
        }

        if (!isset($route['method'])) {
            throw new InvalidArgumentException('Required parameter method is not set.');
        }

        if (!isset($route['controller'])) {
            throw new InvalidArgumentException('Required parameter controller is not set.');
        }

        if (isset($route['value']) && !is_array($route['value'])) {
            $route['value'] = [];
        }

        $methods = !is_array($route['method']) ? [$route['method']] : $route['method'];
        $assert = isset($route['assert']) ? $route['assert'] : [];
        $value = isset($route['value']) ? $route['value'] : [];
        $name = isset($route['name']) ? $route['name'] : '';

        return new Route(
            new Methods($methods),
            new Pattern($route['pattern']),
            new Controller($route['controller']),
            new Asserts($assert),
            new Values($value),
            new Name($name)

It seems like $name defined by isset($route['name']) ? $route['name'] : '' on line 35 can also be of type array; however, MJanssen\Route\Name::__construct() does only seem to accept string, maybe add an additional type check?

        );
    }
}