PhpBoot\Controller\ControllerContainer

Complexity

Total Complexity

17

Size/Duplication

Total Lines	179
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

85.37%

Importance

Changes

0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	addRoute()	0	5	2
A	getRoutes()	0	4	1
A	setRoutes()	0	4	1
A	getRoute()	0	7	2
A	dispatch()	0	10	1
A	getClassName()	0	4	1
A	getFileName()	0	4	1
A	setFileName()	0	4	1
A	setDescription()	0	4	1
A	getDescription()	0	4	1
A	getSummary()	0	4	1
A	setSummary()	0	4	1
A	getUriPrefix()	0	4	1
A	setUriPrefix()	0	4	1

<?php

namespace PhpBoot\Controller;
use PhpBoot\Application;
use Symfony\Component\HttpFoundation\Request;

/**
 * Class ControllerContainer
 */
class ControllerContainer
{
    public function __construct($className)
    {
        $this->className = $className;
    }
    /**
     * 添加路由
     * @param Route $route
     * @param string $actionName class method
     * @return void
     */
    public function addRoute($actionName, Route $route)
    {
        !array_key_exists($actionName, $this->routes) or \PhpBoot\abort("repeated @route for {$this->className}::$actionName");

Using logical operators such as or instead of || is generally not recommended.

        $this->routes[$actionName] = $route;
    }
    /**
     * 获取路由列表
     * @return Route[]
     */
    public function getRoutes()
    {
        return $this->routes;
    }

    /**
     * 获取路由列表
     * @param Route[] $routes
     */
    public function setRoutes($routes)
    {
        $this->routes = $routes;
    }
    /**
     * 获取指定名称的路由
     * @param $actionName
     * @return Route|false
     */
    public function getRoute($actionName)
    {
        if (array_key_exists($actionName, $this->routes)){
            return $this->routes[$actionName];
        }
        return false;
    }

    static public function dispatch(

As per PSR2, the static declaration should come after the visibility declaration.

        Application $app,
        $className,
        $actionName,
        Route $route,
        Request $request)
    {
        $ctrl = $app->get($className);
        return $route->invoke($app, [$ctrl, $actionName], $request);
    }
    
//    /**

44% of this comment could be valid code. Did you maybe forget this after debugging?

//     * 应用路由, 使路由生效
//     * @param RouteCollector $r
//     * @return RouteCollector
//     */
//    public function build(RouteCollector $r)
//    {
//        foreach ($this->routes as $actionName=>$route){
//            $r->addRoute(
//                $route->getMethod(),
//                $route->getUri(),
//                new SerializableFunc(self::class.'::dispatch', $this->className, $actionName, $route)
//            );
//        }
//        return $r;
//    }

    /**
     * @return string
     */
    public function getClassName()
    {
        return $this->className;
    }

    /**
     * 获取uri前缀
     * @return string
     */
    public function getUriPrefix()
    {
        return $this->prefix;
    }

    /**
     * 设置uri前缀
     * @param string $prefix
     */
    public function setUriPrefix($prefix)
    {
        $this->prefix = $prefix;
    }

    /**
     * @return string
     */
    public function getFileName()
    {
        return $this->fileName;
    }

    /**
     * @param string $fileName
     */
    public function setFileName($fileName)
    {
        $this->fileName = $fileName;
    }

    /**
     * @param string $description
     */
    public function setDescription($description)
    {
        $this->description = $description;
    }

    /**
     * @return string
     */
    public function getDescription()
    {
        return $this->description;
    }

    /**
     * @return string
     */
    public function getSummary()
    {
        return $this->summary;
    }

    /**
     * @param string $summary
     */
    public function setSummary($summary)
    {
        $this->summary = $summary;
    }

    /**
     * @var string
     * the prefix path for all routes of the controller
     */
    private $prefix;

    /**
     * @var string
     */
    private $className;

    /**
     * @var Route[]
     */
    private $routes=[];

    /**
     * @var string
     */
    private $description='';
    /**
     * @var string
     */
    private $summary='';

    /**
     * @var string
     */
    private $fileName;
}