Jerodev\PhpIrcClient\Helpers\EventHandlerCollection

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	addHandler()	0	15	3
A	invoke()	0	8	2

<?php

declare(strict_types=1);

namespace Jerodev\PhpIrcClient\Helpers;

class EventHandlerCollection
{
    /** @var array<string, array<int, callable>> */
    private array $eventHandlers = [];

    /**
     * Register an event handler.
     *
     * @param callable|string $event The name of the event to listen for. Pass a callable to this parameter to catch all events.
     * @param callable|null $function The callable that will be invoked on event
     */
    public function addHandler(
        callable | string $event,
        ?callable $function
    ): void {
        if (is_callable($event)) {
            $function = $event;
            $event = '*';
        }

        if (!array_key_exists($event, $this->eventHandlers)) {

It seems like $event can also be of type callable; however, parameter $key of array_key_exists() does only seem to accept integer|string, maybe add an additional type check?

            $this->eventHandlers[$event] = [];
        }

        /** @psalm-suppress InvalidPropertyAssignmentValue */
        $this->eventHandlers[$event][] = $function;
    }

    /**
     *  Invoke all handlers for a specific event.
     */
    public function invoke(Event $event): void
    {
        $handlers = array_merge(
            $this->eventHandlers['*'] ?? [],
            $this->eventHandlers[$event->getEvent()] ?? []
        );
        foreach ($handlers as $handler) {
            $handler(...$event->getArguments());
        }
    }
}