UniMan\Drivers\Redis\DataManager\RedisSortedSetDataManager

Complexity

Total Complexity

15

Size/Duplication

Total Lines	71
Duplicated Lines	84.51 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	itemsCount()	25	25	6
B	items()	35	35	8

<?php

namespace UniMan\Drivers\Redis\DataManager;

use RedisProxy\RedisProxy;
use UniMan\Core\Utils\Filter;

class RedisSortedSetDataManager
{
    private $connection;

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

    public function itemsCount(string $table, array $filter): int

This method seems to be duplicated in your project.

    {
        if (!$filter) {

The expression $filter of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return $this->connection->zcard($table);
        }
        $iterator = '';
        $totalItems = 0;
        do {
            $res = $this->connection->zscan($table, $iterator, null, 1000);
            if (!$res) {
                return $totalItems;
            }
            foreach ($res as $member => $score) {
                $item = [
                    'member' => $member,
                    'length' => strlen($member),
                    'score' => $score,
                ];
                if (Filter::apply($item, $filter)) {
                    $totalItems++;
                }
            }
        } while ($iterator !== 0);

The strict comparison !== seems to always evaluate to true as the types of $iterator (string) and 0 (integer) can never be identical. Maybe you want to use a loose comparison != instead?

        return $totalItems;
    }

    public function items(string $table, int $page, int $onPage, array $filter = []): array

This method seems to be duplicated in your project.

    {
        $skipped = 0;
        $offset = ($page - 1) * $onPage;
        $items = [];
        $iterator = '';
        do {
            $pattern = null;
            $res = $this->connection->zscan($table, $iterator, $pattern, $onPage * 10);
            if (!$res) {
                return $items;
            }
            foreach ($res as $member => $score) {
                $item = [
                    'member' => $member,
                    'length' => strlen($member),
                    'score' => $score,
                ];
                if (!Filter::apply($item, $filter)) {
                    continue;
                }

                if ($skipped < $offset) {
                    $skipped++;
                    continue;
                }

                $items[$member] = $item;
                if (count($items) === $onPage) {
                    break;
                }
            }
        } while ($iterator !== 0 && count($items) < $onPage);

The strict comparison !== seems to always evaluate to true as the types of $iterator (string) and 0 (integer) can never be identical. Maybe you want to use a loose comparison != instead?

        return $items;
    }
}