ApiListTrait - Code Metrics - Inspection of "Merge pull request #4 from hidehalo/wip" - dazzle-php/redis - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 93ca92...ff64f1 )

by Kamil

created 2017-07-11 16:33 UTC

ApiListTrait A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	248
Duplicated Lines	33.06 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

97.67%

Importance

Changes

Metric	Value
dl	82
loc	248
c	0
b	0
f	0
wmc	19
lcom	1
cbo	1
ccs	84
cts	86
cp	0.9767
rs	10

18 Methods

Rating	Name	Duplication	Size	Complexity
	dispatch()	0	1	?
A	blPop()	22	22	2
A	brPop()	22	22	2
A	brPopLPush()	0	8	1
A	lIndex()	0	8	1
A	lInsert()	0	8	1
A	lLen()	7	8	1
A	lPop()	8	8	1
A	lPush()	0	7	1
A	lPushX()	0	7	1
A	lRange()	7	7	1
A	lRem()	0	8	1
A	lSet()	0	8	1
A	lTrim()	0	8	1
A	rPop()	7	7	1
A	rPopLPush()	8	8	1
A	rPush()	0	8	1
A	rPushX()	0	7	1

How to fix Duplicated Code

<?php

namespace Dazzle\Redis\Command\Compose;

use Dazzle\Redis\Command\Builder;
use Dazzle\Redis\Command\Enum;
use Dazzle\Redis\Driver\Request;

trait ApiListTrait
{
    /**
     * @param Request $request
     * @return mixed
     */
    abstract function dispatch(Request $request);


    /**
     * @override
     * @inheritDoc
     */
    public function blPop(array $keys, $timeout)

    {
        // TODO: Implement blPop() method.
        $command = Enum::BLPOP;
        $keys[] = $timeout;
        $args = $keys;
        $promise = $this->dispatch(Builder::build($command, $args));
        $promise = $promise->then(function ($value) {
            if (is_array($value)) {
                list($k,$v) = $value;

                return [
                    'key'=>$k,
                    'value'=>$v
                ];
            }

            return $value;
        });

        return $promise;
    }

    /**
     * @override
     * @inheritDoc
     */
    public function brPop(array $keys, $timeout)

    {
        // TODO: Implement brPop() method.
        $command = Enum::BRPOP;
        $keys[] = $timeout;
        $args = $keys;
        $promise = $this->dispatch(Builder::build($command, $args));
        $promise = $promise->then(function ($value) {
            if (is_array($value)) {
                list($k,$v) = $value;

                return [
                    'key'=>$k,
                    'value'=>$v
                ];
            }

            return $value;
        });

        return $promise;
    }

    /**
     * @override
     * @inheritDoc
     */
    public function brPopLPush($src, $dst, $timeout)
    {
        // TODO: Implement brPopLPush() method.
        $command = Enum::BRPOPLPUSH;
        $args = [$src, $dst, $timeout];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function lIndex($key, $index)
    {
        // TODO: Implement lIndex() method.
        $command = Enum::LINDEX;
        $args = [$key, $index];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function lInsert($key, $action, $pivot, $value)
    {
        // TODO: Implement lInsert() method.
        $command = Enum::LINSERT;
        $args = [$key, $action, $pivot, $value];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function lLen($key)

    {
        // TODO: Implement lLen() method.
        $command = Enum::LLEN;
        $args = [$key];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function lPop($key)

    {
        // TODO: Implement lPop() method.
        $command = Enum::LPOP;
        $args = [$key];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function lPush($key,...$values)
    {
        $command = Enum::LPUSH;
        array_unshift($values, $key);

        return $this->dispatch(Builder::build($command, $values));
    }

    public function lPushX($key, $value)
    {
        $command = Enum::LPUSHX;
        $args = [$key, $value];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function lRange($key, $start = 0, $stop = -1)

    {
        $command = Enum::LRANGE;
        $args = [$key, $start, $stop];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function lRem($key, $count, $value)
    {
        // TODO: Implement lRem() method.
        $command = Enum::LREM;
        $args = [$key, $count, $value];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function lSet($key, $index, $value)
    {
        // TODO: Implement lSet() method.
        $command = Enum::LSET;
        $args = [$key, $index, $value];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function lTrim($key, $start, $stop)
    {
        // TODO: Implement lTrim() method.
        $command = Enum::LTRIM;
        $args = [$key, $start, $stop];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function rPop($key)

    {
        $command = Enum::RPOP;
        $args = [$key];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function rPopLPush($src, $dst)

    {
        // TODO: Implement rPopLPush() method.
        $command = Enum::RPOPLPUSH;
        $args = [$src, $dst];

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function rPush($key, ...$values)
    {
        $command = Enum::RPUSH;
        $args = [$key];
        $args = array_merge($args, $values);

        return $this->dispatch(Builder::build($command, $args));
    }

    /**
     * @override
     * @inheritDoc
     */
    public function rPushX($key, $value)
    {
        $command = Enum::RPUSHX;
        $args = [$key, $value];

        return $this->dispatch(Builder::build($command, $args));
    }
}


1			<?php
2
3			namespace Dazzle\Redis\Command\Compose;
4
5			use Dazzle\Redis\Command\Builder;
6			use Dazzle\Redis\Command\Enum;
7			use Dazzle\Redis\Driver\Request;
8
9			trait ApiListTrait
10			{
11			/**
12			* @param Request $request
13			* @return mixed
14			*/
15			abstract function dispatch(Request $request);
			0 ignored issues – show Best Practice introduced 2017-07-02 15:04 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
16
17			/**
18			* @override
19			* @inheritDoc
20			*/
21	1	View Code Duplication	public function blPop(array $keys, $timeout)
			0 ignored issues – show Duplication introduced 2017-07-01 22:13 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...
22			{
23			// TODO: Implement blPop() method.
24	1		$command = Enum::BLPOP;
25	1		$keys[] = $timeout;
26	1		$args = $keys;
27	1		$promise = $this->dispatch(Builder::build($command, $args));
28			$promise = $promise->then(function ($value) {
29	1		if (is_array($value)) {
30	1		list($k,$v) = $value;
31
32			return [
33	1		'key'=>$k,
34	1		'value'=>$v
35			];
36			}
37
38			return $value;
39	1		});
40
41	1		return $promise;
42			}
43
44			/**
45			* @override
46			* @inheritDoc
47			*/
48	1	View Code Duplication	public function brPop(array $keys, $timeout)
			0 ignored issues – show Duplication introduced 2017-07-01 22:13 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...
49			{
50			// TODO: Implement brPop() method.
51	1		$command = Enum::BRPOP;
52	1		$keys[] = $timeout;
53	1		$args = $keys;
54	1		$promise = $this->dispatch(Builder::build($command, $args));
55	1		$promise = $promise->then(function ($value) {
56	1		if (is_array($value)) {
57	1		list($k,$v) = $value;
58
59			return [
60	1		'key'=>$k,
61	1		'value'=>$v
62			];
63			}
64
65			return $value;
66	1		});
67
68	1		return $promise;
69			}
70
71			/**
72			* @override
73			* @inheritDoc
74			*/
75	1		public function brPopLPush($src, $dst, $timeout)
76			{
77			// TODO: Implement brPopLPush() method.
78	1		$command = Enum::BRPOPLPUSH;
79	1		$args = [$src, $dst, $timeout];
80
81	1		return $this->dispatch(Builder::build($command, $args));
82			}
83
84			/**
85			* @override
86			* @inheritDoc
87			*/
88	5		public function lIndex($key, $index)
89			{
90			// TODO: Implement lIndex() method.
91	5		$command = Enum::LINDEX;
92	5		$args = [$key, $index];
93
94	5		return $this->dispatch(Builder::build($command, $args));
95			}
96
97			/**
98			* @override
99			* @inheritDoc
100			*/
101	1		public function lInsert($key, $action, $pivot, $value)
102			{
103			// TODO: Implement lInsert() method.
104	1		$command = Enum::LINSERT;
105	1		$args = [$key, $action, $pivot, $value];
106
107	1		return $this->dispatch(Builder::build($command, $args));
108			}
109
110			/**
111			* @override
112			* @inheritDoc
113			*/
114	1	View Code Duplication	public function lLen($key)
			0 ignored issues – show Duplication introduced 2017-07-02 15:04 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...
115			{
116			// TODO: Implement lLen() method.
117	1		$command = Enum::LLEN;
118	1		$args = [$key];
119
120	1		return $this->dispatch(Builder::build($command, $args));
121			}
122
123			/**
124			* @override
125			* @inheritDoc
126			*/
127	1	View Code Duplication	public function lPop($key)
			0 ignored issues – show Duplication introduced 2017-07-02 15:04 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...
128			{
129			// TODO: Implement lPop() method.
130	1		$command = Enum::LPOP;
131	1		$args = [$key];
132
133	1		return $this->dispatch(Builder::build($command, $args));
134			}
135
136			/**
137			* @override
138			* @inheritDoc
139			*/
140	15		public function lPush($key,...$values)
141			{
142	15		$command = Enum::LPUSH;
143	15		array_unshift($values, $key);
144
145	15		return $this->dispatch(Builder::build($command, $values));
146			}
147
148	1		public function lPushX($key, $value)
149			{
150	1		$command = Enum::LPUSHX;
151	1		$args = [$key, $value];
152
153	1		return $this->dispatch(Builder::build($command, $args));
154			}
155
156			/**
157			* @override
158			* @inheritDoc
159			*/
160	1	View Code Duplication	public function lRange($key, $start = 0, $stop = -1)
			0 ignored issues – show Duplication introduced 2017-07-05 16:51 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...
161			{
162	1		$command = Enum::LRANGE;
163	1		$args = [$key, $start, $stop];
164
165	1		return $this->dispatch(Builder::build($command, $args));
166			}
167
168			/**
169			* @override
170			* @inheritDoc
171			*/
172	1		public function lRem($key, $count, $value)
173			{
174			// TODO: Implement lRem() method.
175	1		$command = Enum::LREM;
176	1		$args = [$key, $count, $value];
177
178	1		return $this->dispatch(Builder::build($command, $args));
179			}
180
181			/**
182			* @override
183			* @inheritDoc
184			*/
185	1		public function lSet($key, $index, $value)
186			{
187			// TODO: Implement lSet() method.
188	1		$command = Enum::LSET;
189	1		$args = [$key, $index, $value];
190
191	1		return $this->dispatch(Builder::build($command, $args));
192			}
193
194			/**
195			* @override
196			* @inheritDoc
197			*/
198	1		public function lTrim($key, $start, $stop)
199			{
200			// TODO: Implement lTrim() method.
201	1		$command = Enum::LTRIM;
202	1		$args = [$key, $start, $stop];
203
204	1		return $this->dispatch(Builder::build($command, $args));
205			}
206
207			/**
208			* @override
209			* @inheritDoc
210			*/
211	1	View Code Duplication	public function rPop($key)
			0 ignored issues – show Duplication introduced 2017-07-02 15:04 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...
212			{
213	1		$command = Enum::RPOP;
214	1		$args = [$key];
215
216	1		return $this->dispatch(Builder::build($command, $args));
217			}
218
219			/**
220			* @override
221			* @inheritDoc
222			*/
223	1	View Code Duplication	public function rPopLPush($src, $dst)
			0 ignored issues – show Duplication introduced 2017-07-01 22:13 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...
224			{
225			// TODO: Implement rPopLPush() method.
226	1		$command = Enum::RPOPLPUSH;
227	1		$args = [$src, $dst];
228
229	1		return $this->dispatch(Builder::build($command, $args));
230			}
231
232			/**
233			* @override
234			* @inheritDoc
235			*/
236	2		public function rPush($key, ...$values)
237			{
238	2		$command = Enum::RPUSH;
239	2		$args = [$key];
240	2		$args = array_merge($args, $values);
241
242	2		return $this->dispatch(Builder::build($command, $args));
243			}
244
245			/**
246			* @override
247			* @inheritDoc
248			*/
249	1		public function rPushX($key, $value)
250			{
251	1		$command = Enum::RPUSHX;
252	1		$args = [$key, $value];
253
254	1		return $this->dispatch(Builder::build($command, $args));
255			}
256			}
257

dazzle-php / redis

Push — master ( 93ca92...ff64f1 )

ApiListTrait A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

18 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like