StackCallbacks::executeAll() - Code Metrics - Inspection of "Feature/up code quality stack callbacks" - kakserpom/phpdaemon - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#239)

by Дмитрий

created 2016-05-13 22:58 UTC

StackCallbacks::executeAll() A

↳ Parent: StackCallbacks

Complexity

Conditions	4
Paths	4

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Metric	Value
cc	4
eloc	10
nc	4
nop	1
dl	0
loc	18
rs	9.2

<?php
namespace PHPDaemon\Structures;

use PHPDaemon\Core\CallbackWrapper;

/**
 * StackCallbacks
 * @package PHPDaemon\Structures
 * @author  Vasily Zorin <[email protected]>
 */
class StackCallbacks extends \SplStack
{
    use \PHPDaemon\Traits\ClassWatchdog;
    use \PHPDaemon\Traits\StaticObjectWatchdog;

    /**
     * Push callback to the bottom of stack
     * @param  callable $cb Callback
     * @return void
     */
    public function push($cb)
    {
        parent::push(CallbackWrapper::wrap($cb));
    }

    /**
     * Push callback to the top of stack
     * @param  callable $cb Callback
     * @return void
     */
    public function unshift($cb)
    {
        parent::unshift(CallbackWrapper::wrap($cb));
    }

    /**
     * Executes one callback from the top with given arguments
     * @param  mixed   ...$args Arguments
     * @return boolean
     */
    public function executeOne()

    {
        if ($this->isEmpty()) {
            return false;
        }
        $cb = $this->shift();
        if ($cb) {
            $cb(...func_get_args());
            if ($cb instanceof CallbackWrapper) {
                $cb->cancel();
            }
        }
        return true;
    }

    /**
     * Executes one callback from the top with given arguments without taking it out
     * @param  mixed   ...$args Arguments
     * @return boolean
     */
    public function executeAndKeepOne()

    {
        if ($this->isEmpty()) {
            return false;
        }
        $cb = $this->shift();
        $this->unshift($cb);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        if ($cb) {
            $cb(...func_get_args());
        }
        return true;
    }

    /**
     * Executes all callbacks with given arguments
     * @param array $args
     * @return int
     */
    public function executeAll(...$args)
    {
        if ($this->isEmpty()) {
            return 0;
        }

        $n = 0;
        
        foreach ($this as $cb) {
            $cb(...$args);
            ++$n;
            if ($cb instanceof CallbackWrapper) {
                $cb->cancel();
            }
        }

        return $n;
    }

    /**
     * Return array
     * @return array
     */
    public function toArray()
    {
        $arr = [];
        while (!$this->isEmpty()) {
            $arr[] = $this->shift();
        }
        return $arr;
    }

    /**
     * Shifts all callbacks sequentially
     * @return void
     */
    public function reset()
    {
        while (!$this->isEmpty()) {
            $this->shift();
        }
    }
}


1		<?php
2		namespace PHPDaemon\Structures;
3
4		use PHPDaemon\Core\CallbackWrapper;
5
6		/**
7		* StackCallbacks
8		* @package PHPDaemon\Structures
9		* @author Vasily Zorin <[email protected]>
10		*/
11		class StackCallbacks extends \SplStack
12		{
13		use \PHPDaemon\Traits\ClassWatchdog;
14		use \PHPDaemon\Traits\StaticObjectWatchdog;
15
16		/**
17		* Push callback to the bottom of stack
18		* @param callable $cb Callback
19		* @return void
20		*/
21		public function push($cb)
22		{
23		parent::push(CallbackWrapper::wrap($cb));
24		}
25
26		/**
27		* Push callback to the top of stack
28		* @param callable $cb Callback
29		* @return void
30		*/
31		public function unshift($cb)
32		{
33		parent::unshift(CallbackWrapper::wrap($cb));
34		}
35
36		/**
37		* Executes one callback from the top with given arguments
38		* @param mixed ...$args Arguments
39		* @return boolean
40		*/
41	View Code Duplication	public function executeOne()
		0 ignored issues – show Duplication introduced 2015-12-06 14:42 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
42		{
43		if ($this->isEmpty()) {
44		return false;
45		}
46		$cb = $this->shift();
47		if ($cb) {
48		$cb(...func_get_args());
49		if ($cb instanceof CallbackWrapper) {
50		$cb->cancel();
51		}
52		}
53		return true;
54		}
55
56		/**
57		* Executes one callback from the top with given arguments without taking it out
58		* @param mixed ...$args Arguments
59		* @return boolean
60		*/
61	View Code Duplication	public function executeAndKeepOne()
		0 ignored issues – show Duplication introduced 2015-12-06 14:42 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
62		{
63		if ($this->isEmpty()) {
64		return false;
65		}
66		$cb = $this->shift();
67		$this->unshift($cb);
		0 ignored issues – show Documentation introduced 2015-12-06 14:42 UTC by Report Bug Copy Issue Report `$cb` is of type `*`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
68		if ($cb) {
69		$cb(...func_get_args());
70		}
71		return true;
72		}
73
74		/**
75		* Executes all callbacks with given arguments
76		* @param array $args
77		* @return int
78		*/
79		public function executeAll(...$args)
80		{
81		if ($this->isEmpty()) {
82		return 0;
83		}
84
85		$n = 0;
86
87		foreach ($this as $cb) {
88		$cb(...$args);
89		++$n;
90		if ($cb instanceof CallbackWrapper) {
91		$cb->cancel();
92		}
93		}
94
95		return $n;
96		}
97
98		/**
99		* Return array
100		* @return array
101		*/
102		public function toArray()
103		{
104		$arr = [];
105		while (!$this->isEmpty()) {
106		$arr[] = $this->shift();
107		}
108		return $arr;
109		}
110
111		/**
112		* Shifts all callbacks sequentially
113		* @return void
114		*/
115		public function reset()
116		{
117		while (!$this->isEmpty()) {
118		$this->shift();
119		}
120		}
121		}
122

kakserpom / phpdaemon

Pull Request — master (#239)

StackCallbacks::executeAll() A

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like