RedisTrait::transaction() - Code Metrics - Inspection of "Merge pull request #1 from alex-foleon/001-basic" - alex-foleon/service-library-catalog-php - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — main (4cb13d)

by Mr.

created 2020-12-24 15:19 UTC

RedisTrait::transaction() A

↳ Parent: RedisTrait

Complexity

Conditions	3
Paths	3

Size

Total Lines	16
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	4.4609

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	11
c	1
b	0
f	0
nc	3
nop	4
dl	0
loc	16
ccs	5
cts	11
cp	0.4545
crap	4.4609
rs	9.9

<?php

declare(strict_types=1);

namespace LibraryCatalog\Service\Repository;

use Predis\Client;

trait RedisTrait
{
    /**
     * @param Client $predis
     * @param string $key
     * @param int $ttl
     */
    public function lock(Client $predis, string $key, int $ttl): void
    {
        while ($predis->get($key)) {
            usleep(50000);
        }
        $predis->setex($key, $ttl, true);
    }

    /**
     * @param Client $predis
     * @param string $key
     */
    public function unlock(Client $predis, string $key): void
    {
        $predis->del($key);
    }

    /**
     * @param Client $predis
     * @param string $key
     * @param int $ttl
     * @param \Closure $closure
     * @throws \Throwable
     * @return mixed
     */
    public function transaction(Client $predis, string $key, int $ttl, \Closure $closure)
    {
        $this->lock($predis, $key, $ttl);
        try {
            $res = $closure();
        } catch (\Exception $e) {
            $this->unlock($predis, $key);
            throw $e;
        } catch (\Throwable $e) {
            $this->unlock($predis, $key);
            throw $e;
        }
        // Unlock in correct flow.
        $this->unlock($predis, $key);

        return $res;
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace LibraryCatalog\Service\Repository;
6
7		use Predis\Client;
8
9		trait RedisTrait
10		{
11		/**
12		* @param Client $predis
13		* @param string $key
14		* @param int $ttl
15		*/
16	7	public function lock(Client $predis, string $key, int $ttl): void
17		{
18	7	while ($predis->get($key)) {
19		usleep(50000);
20		}
21	7	$predis->setex($key, $ttl, true);
22	7	}
23
24		/**
25		* @param Client $predis
26		* @param string $key
27		*/
28	7	public function unlock(Client $predis, string $key): void
29		{
30	7	$predis->del($key);
31	7	}
32
33		/**
34		* @param Client $predis
35		* @param string $key
36		* @param int $ttl
37		* @param \Closure $closure
38		* @throws \Throwable
39		* @return mixed
40		*/
41	7	public function transaction(Client $predis, string $key, int $ttl, \Closure $closure)
42		{
43	7	$this->lock($predis, $key, $ttl);
44		try {
45	7	$res = $closure();
46		} catch (\Exception $e) {
47		$this->unlock($predis, $key);
48		throw $e;
49		} catch (\Throwable $e) {
50		$this->unlock($predis, $key);
51		throw $e;
52		}
53		// Unlock in correct flow.
54	7	$this->unlock($predis, $key);
55
56	7	return $res;
57		}
58		}
59

alex-foleon / service-library-catalog-php

Branch — main (4cb13d)

RedisTrait::transaction() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like