Imanghafoori\HeyMan\Reactions\ReactionFactory

Complexity

Total Complexity

7

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	makeReaction()	0	14	2
A	methodsToCall()	0	12	3
A	make()	0	5	1

<?php

namespace Imanghafoori\HeyMan\Reactions;

use Imanghafoori\HeyMan\Chain;

class ReactionFactory
{
    /**
     * @var \Imanghafoori\HeyMan\Chain
     */
    private $chain;

    /**
     * ListenerFactory constructor.
     *
     * @param Chain $chain
     */
    public function __construct(Chain $chain)
    {
        $this->chain = $chain;
    }

    /**
     * @return \Closure
     */
    public function make(): \Closure
    {
        $responder = app(ResponderFactory::class)->make();

        return $this->makeReaction($responder);
    }

    /**
     * @param $responder
     *
     * @return \Closure
     */
    private function makeReaction(callable $responder): \Closure
    {
        $beforeResponse = $this->methodsToCall();

        $cb = $this->chain->predicate;
        $this->chain->reset();

        return function (...$f) use ($responder, $cb, $beforeResponse) {
            if ($cb($f)) {
                return true;
            }

            $beforeResponse();
            $responder();
        };
    }

    private function methodsToCall(): \Closure
    {
        $calls = $this->chain->beforeResponse;

        if (!$calls) {

The expression $calls of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return function () {
            };
        }

        return function () use ($calls) {
            foreach ($calls as $call) {
                $call();
            }
        };
    }
}