ApcuRateLimiter::updateCounter() - Code Metrics - Inspection of "new limiters" - nikolaposa/rate-limit - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#24)

by Julián

created 2020-05-09 00:07 UTC

ApcuRateLimiter::updateCounter() A

↳ Parent: ApcuRateLimiter

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	10
rs	9.9332
c	0
b	0
f	0
cc	3
nc	4
nop	3

<?php

declare(strict_types=1);

namespace RateLimit;

final class ApcuRateLimiter extends AbstractElapsedTimeRateLimiter
{
    public function __construct(string $keyPrefix = '')
    {
        if (!\extension_loaded('apcu')) {
            throw new \RuntimeException('APCu extension is not loaded.');
        }

        if (\ini_get('apc.use_request_time') === '1') {
            throw new \RuntimeException('APCu ini configuration "apc.use_request_time" should be set to "0".');
        }

        parent::__construct($keyPrefix);
    }

    protected function getCurrent(string $valueKey): int
    {
        return (int) \apcu_fetch($valueKey);
    }

    protected function updateCounter(string $valueKey, string $timeKey, int $interval): int
    {
        $current = \apcu_inc($valueKey, 1, $success, $interval);

        if ($current === 1) {
            \apcu_store($timeKey, \time(), $interval);
        }

        return $current === false ? 1 : $current;
    }

    protected function getElapsedTime(string $timeKey): int
    {
        return \time() - (int) \apcu_fetch($timeKey);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace RateLimit;
6
7			final class ApcuRateLimiter extends AbstractElapsedTimeRateLimiter
8			{
9			public function __construct(string $keyPrefix = '')
10			{
11			if (!\extension_loaded('apcu')) {
12			throw new \RuntimeException('APCu extension is not loaded.');
13			}
14
15			if (\ini_get('apc.use_request_time') === '1') {
16			throw new \RuntimeException('APCu ini configuration "apc.use_request_time" should be set to "0".');
17			}
18
19			parent::__construct($keyPrefix);
20			}
21
22			protected function getCurrent(string $valueKey): int
23			{
24			return (int) \apcu_fetch($valueKey);
25			}
26
27			protected function updateCounter(string $valueKey, string $timeKey, int $interval): int
28			{
29			$current = \apcu_inc($valueKey, 1, $success, $interval);
30
31			if ($current === 1) {
32			\apcu_store($timeKey, \time(), $interval);
33			}
34
35			return $current === false ? 1 : $current;
36			}
37
38			protected function getElapsedTime(string $timeKey): int
39			{
40			return \time() - (int) \apcu_fetch($timeKey);
41			}
42			}
43

nikolaposa / rate-limit

Pull Request — master (#24)

ApcuRateLimiter::updateCounter() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like