Find::getUsersByPage() - Code Metrics - FernandoCalmet/php-slim-rest-api - Measure and Improve Code Quality continuously with Scrutinizer

Find::getUsersByPage() A
last analyzed 2023-02-21 00:05 UTC

↳ Parent: Find

Complexity

Conditions	3
Paths	4

Size

Total Lines	18
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	9
c	1
b	0
f	0
nc	4
nop	4
dl	0
loc	18
ccs	8
cts	8
cp	1
crap	3
rs	9.9666

<?php

declare(strict_types=1);

namespace App\Service\User;

final class Find extends Base
{
    /**
     * @return array<string>
     */
    public function getAll(): array
    {
        return $this->userRepository->getAllUsers();
    }

    /**
     * @return array<string>
     */
    public function getUsersByPage(
        int $page,
        int $perPage,
        ?string $name,
        ?string $email
    ): array {
        if ($page < 1) {
            $page = 1;
        }
        if ($perPage < 1) {
            $perPage = self::DEFAULT_PER_PAGE_PAGINATION;
        }

        return $this->userRepository->getUsersByPage(

            $page,
            $perPage,
            $name,
            $email
        );
    }

    public function getOne(int $userId): object
    {
        if (self::isRedisEnabled() === true) {
            $user = $this->getUserFromCache($userId);
        } else {
            $user = $this->getUserFromDb($userId)->toJson();
        }

        return $user;
    }

    public function search(string $usersName): array
    {
        return $this->userRepository->search($usersName);
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace App\Service\User;
6
7		final class Find extends Base
8		{
9	1	/**
10		* @return array<string>
11	1	*/
12		public function getAll(): array
13		{
14	2	return $this->userRepository->getAllUsers();
15		}
16
17		/**
18		* @return array<string>
19		*/
20	2	public function getUsersByPage(
21	1	int $page,
22		int $perPage,
23	2	?string $name,
24	1	?string $email
25		): array {
26		if ($page < 1) {
27	2	$page = 1;
28	2	}
29		if ($perPage < 1) {
30		$perPage = self::DEFAULT_PER_PAGE_PAGINATION;
31		}
32
33		return $this->userRepository->getUsersByPage(
		0 ignored issues – show Bug Best Practice introduced 2022-03-06 07:50 UTC by Report Bug Copy Issue Report The expression `return $this->userReposi...perPage, $name, $email)` returns the type `array<string,array\|array<string,double\|integer>>` which is incompatible with the documented return type `string[]`. Loading history...
34		$page,
35	3	$perPage,
36		$name,
37	3	$email
38	3	);
39		}
40
41		public function getOne(int $userId): object
42		{
43	2	if (self::isRedisEnabled() === true) {
44		$user = $this->getUserFromCache($userId);
45		} else {
46	2	$user = $this->getUserFromDb($userId)->toJson();
47		}
48	2
49		return $user;
50		}
51
52		public function search(string $usersName): array
53		{
54		return $this->userRepository->search($usersName);
55		}
56		}
57

FernandoCalmet / php-slim-rest-api

Find::getUsersByPage() A last analyzed 2023-02-21 00:05 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

Find::getUsersByPage() A
last analyzed 2023-02-21 00:05 UTC