Dspbee\Core\BaseRoute

Complexity

Total Complexity

5

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	3
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getResponse()	0	4	1
B	default()	0	24	3

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 14 and the first side effect is on line 60.

/**
 * @license MIT
 * @author Igor Sorokin <dspbee@pivasic.com>
 */
namespace Dspbee\Core;

/**
 * Base functions to custom route.
 *
 * Class Route
 * @package Dspbee\Core
 */
class BaseRoute
{
    public function __construct()
    {
        $this->response = null;
    }

    /**
     * Get object of Response.
     *
     * @return Response|null.

The doc-type Response|null. could not be parsed: Unknown type name "null." at position 9. (view supported doc-types)

     */
    public function getResponse()
    {
        return $this->response;
    }

    /**
     * @param string $packageRoot
     * @param Request $request
     */
    public function default($packageRoot, Request $request)

Possible parse error: non-abstract method defined as abstract

It is generally advisable to only define one property per statement.

    {
        $packageRoot = rtrim($packageRoot, '/');
        $path = $packageRoot . '/Route/' . $request->route() . '/' . $request->method() . '.php';

It is generally advisable to only define one property per statement.

The visibility should be declared for property $path.

        if (file_exists($path)) {

It is generally advisable to only define one property per statement.

The visibility should be declared for property $path.

            require $path;
            $controllerClass = $request->package() . '\\Route_' . str_replace('/', '_', $request->route()) . '\\' . $request->method();

It is generally advisable to only define one property per statement.

The visibility should be declared for property $controllerClass.

            /**
             * @var BaseController $controller
             */
            $controller = new $controllerClass($packageRoot, $request);

It is generally advisable to only define one property per statement.

The visibility should be declared for property $controller.

            /**
             * Call handler.
             */
            $handler = $_POST['handler'] ?? $_GET['handler'] ?? 'index';

It is generally advisable to only define one property per statement.

The visibility should be declared for property $handler.

            $handler = str_replace('.', '', $handler);

It is generally advisable to only define one property per statement.

The visibility should be declared for property $handler.

            if (method_exists($controllerClass, $handler)) {

It is generally advisable to only define one property per statement.

The visibility should be declared for property $controllerClass.

                $controller->$handler();

                $this->response = $controller->getResponse();

It is generally advisable to only define one property per statement.

The visibility should be declared for property $this.

            }
        }
    }

    protected $response;
}