Nymfonya\Component\Http\Kernel

Complexity

Total Complexity

12

Size/Duplication

Total Lines	116
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

95.35%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	setNameSpace()	0	5	1
A	getBundleClassname()	0	4	1
A	run()	0	21	4
A	setAction()	0	4	2
A	send()	0	22	2
A	shutdown()	0	4	1
A	__construct()	0	5	1

<?php

namespace Nymfonya\Component\Http;

use Nymfonya\Component\Http\Response;
use Nymfonya\Component\Http\Headers;

class Kernel
{

    const PATH_CONFIG = '/../config/';

    use \Nymfonya\Component\Http\Reuse\TKernel;

    /**
     * instanciate
     * @return Kernel

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(string $env, string $path)
    {
        $this->init($env, $path);
        return $this;

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

    }

    /**
     * set controller namespace
     *
     * @param string $ctrlNamespace
     * @return Kernel
     */
    public function setNameSpace(string $ctrlNamespace): Kernel
    {
        $this->spacename = $ctrlNamespace;
        return $this;
    }

    /**
     * retrieve kernel instance classname from container
     *
     * @return string
     */
    public function getBundleClassname():string
    {
        return get_called_class();
    }

    /**
     * run app
     *
     * @param array $groups
     * @return Kernel
     */
    public function run(array $groups = []): Kernel
    {
        $routerGroups = (empty($groups))
            ? $this->router->compile()
            : $groups;
        if ($routerGroups) {

The expression $routerGroups 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.

            $this->setClassname($routerGroups);
            if (class_exists($this->className)) {
                $this->setController();
                $this->setReflector();
                $this->setActions();
                $this->setAction($routerGroups);
                //->setActionAnnotations();
                $this->setMiddleware();
            } else {
                $this->error = true;
                $this->errorCode = Response::HTTP_SERVICE_UNAVAILABLE;
            }
        }
        return $this;
    }

    /**
     * set controller action from router groups
     *
     * @param array $routerGrps
     * @return void
     */
    public function setAction(array $routerGrps)
    {
        $this->action = isset($routerGrps[1]) ? strtolower($routerGrps[1]) : '';
    }

    /**
     * dispatch response
     *
     * @return Kernel
     */
    public function send(): Kernel
    {
        if ($this->getError()) {
            $this->res
                ->setCode($this->errorCode)
                ->setContent([
                    Response::_ERROR => $this->error,
                    Response::_ERROR_CODE => $this->errorCode,
                    Response::_ERROR_MSG => $this->errorMsg,
                ])
                ->getHeaderManager()
                ->add(
                    Headers::CONTENT_TYPE,
                    'application/json; charset=utf-8'
                );
            $this->getLogger()->warning($this->errorMsg);
        } else {
            $this->getLogger()->debug('Response sent');
        }
        $this->getResponse()->send();
        return $this;
    }

    /**
     * shutdown kernel
     *
     * @return void

Consider making the return type a bit more specific; maybe use NoType.

     */
    public function shutdown(int $code = 0)
    {
        throw new \Exception('shutdown', $code);
    }
}