AutoScan::doIteration() - Code Metrics - kakserpom/phpdaemon - Measure and Improve Code Quality continuously with Scrutinizer

AutoScan::doIteration() B
last analyzed 2021-06-02 17:17 UTC

↳ Parent: AutoScan

Complexity

Conditions	6
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	6
dl	0
loc	24
rs	8.9137
c	0
b	0
f	0
nc	2
nop	0

<?php
namespace PHPDaemon\Clients\Redis;

use PHPDaemon\Core\CallbackWrapper;

/**
 * @package    NetworkClients
 * @subpackage RedisClient
 * @author     Efimenko Dmitriy <[email protected]>
 */
class AutoScan
{
    use \PHPDaemon\Traits\ClassWatchdog;
    use \PHPDaemon\Traits\StaticObjectWatchdog;

    protected $conn;

    protected $cmd;

    protected $cursor = 0;

    protected $args;

    protected $limit = false;

    protected $num = 0;

    protected $isFreeze = false;

    protected $cb;

    protected $cbEnd;

    /**
     * Constructor
     * @param  Pool $pool Redis pool or connection
     * @param  string $cmd Command
     * @param  array $args Arguments
     * @param  cllable $cbEnd Callback

     * @param  integer $limit Limit

     */
    public function __construct($pool, $cmd, $args = [], $cbEnd = null, $limit = false)
    {
        $this->conn = $pool;
        $this->cmd = $cmd;
        $this->args = empty($args) ? [] : $args;
        $this->limit = $limit;
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
        if (is_numeric($this->args[0])) {
            array_shift($this->args);
        }
        for ($i = sizeof($this->args) - 1; $i >= 0; --$i) {
            $a = $this->args[$i];
            if ((is_array($a) || is_object($a)) && is_callable($a)) {
                $this->cb = CallbackWrapper::wrap($a);
                $this->args = array_slice($this->args, 0, $i);
                break;
            } elseif ($a !== null) {
                break;
            }
        }
        if ($cbEnd !== null) {
            $this->cbEnd = CallbackWrapper::wrap($cbEnd);
        }
        $this->doIteration();
    }

    protected function doIteration()
    {
        if ($this->isFreeze) {
            return;
        }

        $args = $this->args;
        array_unshift($args, $this->cursor);
        $args[] = function ($redis) {
            $this->conn = $redis;
            $this->cursor = $redis->result[0];
            $func = $this->cb;
            $func($redis);

            if (!is_numeric($redis->result[0]) || !$redis->result[0] || ($this->limit && ++$this->num > $this->limit)) {
                $func = $this->cbEnd;
                $func($redis, $this);
                return;
            }
            $this->doIteration();
        };
        $func = [$this->conn, $this->cmd];
        $func(... $args);
    }

    public function freeze()
    {
        $this->isFreeze = true;
    }

    public function run()
    {
        $this->isFreeze = false;
        $this->doIteration();
    }

    public function reset()
    {
        $this->num = 0;
        $this->isFreeze = false;
    }
}


1			<?php
2			namespace PHPDaemon\Clients\Redis;
3
4			use PHPDaemon\Core\CallbackWrapper;
5
6			/**
7			* @package NetworkClients
8			* @subpackage RedisClient
9			* @author Efimenko Dmitriy <[email protected]>
10			*/
11			class AutoScan
12			{
13			use \PHPDaemon\Traits\ClassWatchdog;
14			use \PHPDaemon\Traits\StaticObjectWatchdog;
15
16			protected $conn;
17
18			protected $cmd;
19
20			protected $cursor = 0;
21
22			protected $args;
23
24			protected $limit = false;
25
26			protected $num = 0;
27
28			protected $isFreeze = false;
29
30			protected $cb;
31
32			protected $cbEnd;
33
34			/**
35			* Constructor
36			* @param Pool $pool Redis pool or connection
37			* @param string $cmd Command
38			* @param array $args Arguments
39			* @param cllable $cbEnd Callback
			0 ignored issues – show Documentation introduced 2015-12-07 02:43 UTC by Report Bug Copy Issue Report Should the type for parameter `$cbEnd` not be `cllable\|null`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
40			* @param integer $limit Limit
			0 ignored issues – show Documentation introduced 2015-12-07 02:43 UTC by Report Bug Copy Issue Report Should the type for parameter `$limit` not be `false\|integer`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
41			*/
42			public function __construct($pool, $cmd, $args = [], $cbEnd = null, $limit = false)
43			{
44			$this->conn = $pool;
45			$this->cmd = $cmd;
46			$this->args = empty($args) ? [] : $args;
47			$this->limit = $limit;
			0 ignored issues – show Documentation Bug introduced 2015-12-07 02:43 UTC by Report Bug Copy Issue Report It seems like `$limit` can also be of type `integer`. However, the property `$limit` is declared as type `boolean`. Maybe add an additional type check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable `$accountId` that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the `id` property of an instance of the `Account` class. This class holds a proper account, so the id value must no longer be false. Either this assignment is in error or a type check should be added for that assignment. class Id { public $id; public function __construct($id) { $this->id = $id; } } class Account { /** @var Id $id */ public $id; } $account_id = false; if (starsAreRight()) { $account_id = new Id(42); } $account = new Account(); if ($account instanceof Id) { $account->id = $account_id; } Loading history...
48			if (is_numeric($this->args[0])) {
49			array_shift($this->args);
50			}
51			for ($i = sizeof($this->args) - 1; $i >= 0; --$i) {
52			$a = $this->args[$i];
53			if ((is_array($a) \|\| is_object($a)) && is_callable($a)) {
54			$this->cb = CallbackWrapper::wrap($a);
55			$this->args = array_slice($this->args, 0, $i);
56			break;
57			} elseif ($a !== null) {
58			break;
59			}
60			}
61			if ($cbEnd !== null) {
62			$this->cbEnd = CallbackWrapper::wrap($cbEnd);
63			}
64			$this->doIteration();
65			}
66
67			protected function doIteration()
68			{
69			if ($this->isFreeze) {
70			return;
71			}
72
73			$args = $this->args;
74			array_unshift($args, $this->cursor);
75			$args[] = function ($redis) {
76			$this->conn = $redis;
77			$this->cursor = $redis->result[0];
78			$func = $this->cb;
79			$func($redis);
80
81			if (!is_numeric($redis->result[0]) \|\| !$redis->result[0] \|\| ($this->limit && ++$this->num > $this->limit)) {
82			$func = $this->cbEnd;
83			$func($redis, $this);
84			return;
85			}
86			$this->doIteration();
87			};
88			$func = [$this->conn, $this->cmd];
89			$func(... $args);
90			}
91
92			public function freeze()
93			{
94			$this->isFreeze = true;
95			}
96
97			public function run()
98			{
99			$this->isFreeze = false;
100			$this->doIteration();
101			}
102
103			public function reset()
104			{
105			$this->num = 0;
106			$this->isFreeze = false;
107			}
108			}
109

kakserpom / phpdaemon

AutoScan::doIteration() B last analyzed 2021-06-02 17:17 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

AutoScan::doIteration() B
last analyzed 2021-06-02 17:17 UTC