Notifier::notify() - Code Metrics - Inspection of "Merge pull request #17 from phpextra/5.0" - phpextra/event-manager - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( b6becd...dd5d3f )

by Jacek

created 2019-03-26 15:51 UTC

Notifier::notify() A

↳ Parent: Notifier

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	11
c	0
b	0
f	0
rs	9.9
cc	3
nc	3
nop	2

<?php

declare(strict_types=1);

namespace PHPExtra\EventManager;

/**
 * @internal This class is for internal use only. You should not depend on it in your code.
 */
final class Notifier
{
    public function notify(Listener $listener, Event $event): void
    {
        $eventClass = get_class($event);
        $reflection = new \ReflectionObject($listener);

        foreach ($reflection->getMethods(\ReflectionMethod::IS_PUBLIC) as $method) {
            if ($this->supports($method, $eventClass)) {
                $this->call($listener, $method->getName(), $event);

            }
        }
    }

    private function supports(\ReflectionMethod $method, string $eventClass): bool
    {
        if ($method->getNumberOfParameters() !== 1) {
            return false;
        }

        $paramClass = $method->getParameters()[0]->getClass();

        if ($paramClass === null) {
            return false;
        }

        if (!is_a($eventClass, $paramClass->getName(), true)) {
            return false;
        }

        return true;
    }

    private function call(Listener $listener, string $method, Event $event): void
    {
        $listener->{$method}($event);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace PHPExtra\EventManager;
6
7			/**
8			* @internal This class is for internal use only. You should not depend on it in your code.
9			*/
10			final class Notifier
11			{
12			public function notify(Listener $listener, Event $event): void
13			{
14			$eventClass = get_class($event);
15			$reflection = new \ReflectionObject($listener);
16
17			foreach ($reflection->getMethods(\ReflectionMethod::IS_PUBLIC) as $method) {
18			if ($this->supports($method, $eventClass)) {
19			$this->call($listener, $method->getName(), $event);
			0 ignored issues – show Bug introduced 2019-03-22 16:32 UTC by Report Bug Copy Issue Report Consider using `$method->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
20			}
21			}
22			}
23
24			private function supports(\ReflectionMethod $method, string $eventClass): bool
25			{
26			if ($method->getNumberOfParameters() !== 1) {
27			return false;
28			}
29
30			$paramClass = $method->getParameters()[0]->getClass();
31
32			if ($paramClass === null) {
33			return false;
34			}
35
36			if (!is_a($eventClass, $paramClass->getName(), true)) {
37			return false;
38			}
39
40			return true;
41			}
42
43			private function call(Listener $listener, string $method, Event $event): void
44			{
45			$listener->{$method}($event);
46			}
47			}
48

phpextra / event-manager

Push — master ( b6becd...dd5d3f )

Notifier::notify() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like