HnrAzevedo\Router\CheckTrait

Complexity

Total Complexity

13

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Importance

Changes	6
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	checkMethod()	0	10	5
A	isInPseudGroup()	0	4	2
A	hasRouteName()	0	4	2
A	existRouteName()	0	4	2
A	isInNameGroup()	0	4	2

<?php

namespace HnrAzevedo\Router;

trait CheckTrait
{
    protected array $routers = [];
    protected array $routesName = [];
    protected ?string $group = null;
    
    protected function hasRouteName(string $name): void
    {
        if(!isset($this->routesName[$name])){
            throw new \RuntimeException("There is no route named with {$name}");
        }
    }

    protected function isInNameGroup(): void
    {
        if(!is_null($this->group)){
            throw new \RuntimeException("It is not allowed to assign names to groups");
        }
    }

    protected function isInPseudGroup(): void
    {
        if(!is_null($this->group)){
            throw new \RuntimeException("To assign actions before or after the execution of the route, use beforeGroup or afterGroup");
        }
    }

    protected function existRouteName(string $name): void
    {
        if(isset($this->routesName[$name])){
            throw new \RuntimeException("There is already a route named with {$name}");
        }
    }

    protected function checkMethod(array $route, $method): void
    {
        $hasMethod = false;
        foreach(explode('|',$route['method']) as $routeMethod){
            if(@preg_match("/{$routeMethod}/",$method) == true || $method === '*'){

It seems like you are loosely comparing @preg_match('/'.$routeMethod.'/', $method) of type integer to the boolean true. If you are specifically checking for non-zero, consider using something more explicit like > 0 or !== 0 instead.

                $hasMethod = true;
            }
        }
        if(!$hasMethod){
            throw new \Exception('This route is not released for the accessed method');
        }
        
    }

}