GenerationBench

Complexity

Total Complexity

6

Size/Duplication

Total Lines	60
Duplicated Lines	33.33 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	benchAssembleStaticNip()	0	7	1
A	benchAssembleStandardNip()	0	7	1
A	benchAssembleStaticSymfony()	0	7	1
A	benchAssembleDynamicSymfony()	0	7	1
A	init()	20	20	2

<?php

use Nip\Router\Route\Route;
use Nip\Router\RouteFactory;
use Nip\Router\Router;
use Nip\Router\Tests\Fixtures\Application\Library\Router\Route\StandardRoute;

/**
 * Class GeneratingBench
 * @Iterations(5)
 * @Revs(100)
 * @BeforeMethods({"init"})
 */
class GenerationBench

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * @var Router
     */
    protected $router;

    public function benchAssembleStaticNip()
    {
        $this->router->assemble(
            'index.999',
            ['controller' => 'posts', 'action' => 'create', 'title' => 'test']
        );
    }

    public function benchAssembleStandardNip()
    {
        $this->router->assemble(
            'index.standard999',
            ['controller' => 'posts', 'action' => 'create', 'title' => 'test']
        );
    }

    public function benchAssembleStaticSymfony()
    {
        $this->router->generate(
            'index.999',
            ['controller' => 'posts', 'action' => 'create', 'title' => 'test']
        );
    }

    public function benchAssembleDynamicSymfony()
    {
        $this->router->generate(
            'index.standard999',
            ['controller' => 'posts', 'action' => 'create', 'title' => 'test']
        );
    }

    public function init()

This method seems to be duplicated in your project.

    {
        $this->router = new Router();
        $collection = $this->router->getRoutes();

        for ($i = 0; $i < 1000; ++$i) {
            RouteFactory::generateLiteralRoute(
                $collection,

It seems like $collection defined by $this->router->getRoutes() on line 56 can also be of type array<integer,object<Nip\Router\Route\Route>>; however, Nip\Router\RouteFactory::generateLiteralRoute() does only seem to accept object<Nip\Router\RouteCollection>, maybe add an additional type check?

                "index." . $i,
                Route::class,
                "",
                "/index" . $i);

            RouteFactory::generateStandardRoute(
                $collection,

It seems like $collection defined by $this->router->getRoutes() on line 56 can also be of type array<integer,object<Nip\Router\Route\Route>>; however, Nip\Router\RouteFactory::generateStandardRoute() does only seem to accept object<Nip\Router\RouteCollection>, maybe add an additional type check?

                "index.standard" . $i,
                StandardRoute::class,
                "/" . $i);
        }
    }
}