Gacela\Router\Request

Complexity

Total Complexity

7

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Test Coverage

Coverage

78.95%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	path()	0	6	1
A	__construct()	0	5	1
A	instance()	0	6	2
A	resetCache()	0	3	1
A	isMethod()	0	4	1
A	get()	0	5	1

<?php

declare(strict_types=1);

namespace Gacela\Router;

final class Request
{
    public const METHOD_CONNECT = 'CONNECT';
    public const METHOD_DELETE = 'DELETE';
    public const METHOD_GET = 'GET';
    public const METHOD_HEAD = 'HEAD';
    public const METHOD_OPTIONS = 'OPTIONS';
    public const METHOD_PATCH = 'PATCH';
    public const METHOD_POST = 'POST';
    public const METHOD_PUT = 'PUT';
    public const METHOD_TRACE = 'TRACE';
    private static ?self $instance = null;

    private function __construct(
        private array $query,
        private array $request,
        private array $server,
    ) {
    }

    public static function resetCache(): void
    {
        self::$instance = null;
    }

    public static function instance(): self
    {
        if (self::$instance === null) {
            self::$instance = new self($_GET, $_POST, $_SERVER);
        }
        return self::$instance;

The expression return self::instance could return the type null which is incompatible with the type-hinted return Gacela\Router\Request. Consider adding an additional type-check to rule them out.

    }

    public function isMethod(string $method): bool
    {
        /** @psalm-suppress PossiblyUndefinedArrayOffset */
        return (string)$this->server['REQUEST_METHOD'] === $method;
    }

    public function path(): string
    {
        /** @psalm-suppress PossiblyUndefinedArrayOffset */
        return (string)parse_url(
            (string)$this->server['REQUEST_URI'],
            PHP_URL_PATH,
        );
    }

    public function get(string $key): mixed
    {
        return $this->request[$key]
            ?? $this->query[$key]
            ?? null;
    }
}