StackProcessor::buildCallable() - Code Metrics - Inspection of "Minor optimisation on processor" - b2pweb/bdf-pipeline - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 4598b7...aecae1 )

by Sébastien

created 2018-03-09 08:04 UTC

StackProcessor::buildCallable() A

↳ Parent: StackProcessor::getCallable()

Complexity

Conditions	3
Paths	2

Size

Total Lines	17
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	17
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	8
nc	2
nop	2

<?php

namespace Bdf\Pipeline\Processor;

use Bdf\Pipeline\ProcessorInterface;

/**
 * StackProcessor
 *
 * @author Sébastien Tanneux
 */
class StackProcessor implements ProcessorInterface
{
    /**
     * The pipes callable
     *
     * @var callable
     */
    private $callable;

    /**
     * {@inheritdoc}
     */
    public function process(array $pipes, array $payload, callable $outlet = null)
    {
        $callable = $this->getCallable($pipes, $outlet);

        return $callable(...$payload);
    }

    /**
     * Build the chain callable
     *
     * @param array $pipes
     * @param callable|null $outlet
     *
     * @return \Closure
     */
    private function getCallable($pipes, $outlet)
    {
        if ($this->callable !== null) {
            return $this->callable;
        }

        if ($outlet !== null) {
            $callable = $outlet;
        } else {
            $callable = function ($payload) {
                return $payload;
            };
        };

        while ($pipe = array_pop($pipes)) {
            $callable = function (...$payload) use ($pipe, $callable) {
                return $pipe($callable, ...$payload);
            };
        }

        return $callable;
    }

    /**
     * {@inheritdoc}
     */
    public function clearCache()
    {
        $this->callable = null;
    }
}


1			<?php
2
3			namespace Bdf\Pipeline\Processor;
4
5			use Bdf\Pipeline\ProcessorInterface;
6
7			/**
8			* StackProcessor
9			*
10			* @author Sébastien Tanneux
11			*/
12			class StackProcessor implements ProcessorInterface
13			{
14			/**
15			* The pipes callable
16			*
17			* @var callable
18			*/
19			private $callable;
20
21			/**
22			* {@inheritdoc}
23			*/
24			public function process(array $pipes, array $payload, callable $outlet = null)
25			{
26			$callable = $this->getCallable($pipes, $outlet);
27
28			return $callable(...$payload);
29			}
30
31			/**
32			* Build the chain callable
33			*
34			* @param array $pipes
35			* @param callable\|null $outlet
36			*
37			* @return \Closure
38			*/
39			private function getCallable($pipes, $outlet)
40			{
41			if ($this->callable !== null) {
42			return $this->callable;
43			}
44
45			if ($outlet !== null) {
46			$callable = $outlet;
47			} else {
48			$callable = function ($payload) {
49			return $payload;
50			};
51			};
52
53			while ($pipe = array_pop($pipes)) {
54			$callable = function (...$payload) use ($pipe, $callable) {
55			return $pipe($callable, ...$payload);
56			};
57			}
58
59			return $callable;
60			}
61
62			/**
63			* {@inheritdoc}
64			*/
65			public function clearCache()
66			{
67			$this->callable = null;
68			}
69			}
70

b2pweb / bdf-pipeline

Push — master ( 4598b7...aecae1 )

StackProcessor::buildCallable() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like