Logger::getEntries() - Code Metrics - Inspection of "fixed cs issues" - sweetrdf/in-memory-store-sqlite - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — extract-store (f24e42)

by Konrad

created 2021-03-16 10:18 UTC

Logger::getEntries() A

↳ Parent: Logger

Complexity

Conditions	4
Paths	3

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	4

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	5
nc	3
nop	1
dl	0
loc	9
ccs	6
cts	6
cp	1
crap	4
rs	10
c	1
b	0
f	0

<?php

namespace sweetrdf\InMemoryStoreSqlite;

use Exception;

final class Logger
{
    /**
     * @var array<string,array<integer,mixed>>
     */
    private array $entries = [];

    public function __construct()
    {
        $this->entries = [
            'error' => [],
            'warning' => [],
        ];
    }

    public function error(string $message, array $context = []): void
    {
        $this->log('error', $message, $context);
    }

    public function warning(string $message, array $context = []): void
    {
        $this->log('warning', $message, $context);
    }

    private function log($level, $message, array $context = []): void
    {
        if (!isset($this->entries[$level])) {
            $this->entries[$level] = [];
        }

        $this->entries[$level][] = ['message' => $message, 'context' => $context];
    }

    public function getEntries(?string $level = null): array
    {
        if (null !== $level && isset($this->entries[$level])) {
            return $this->entries[$level];
        } elseif (null == $level) {

            return $this->entries;
        }

        throw new Exception('Level '.$level.' not set.');
    }

    public function hasEntries(?string $level = null): bool
    {
        return 0 < \count($this->getEntries($level));
    }
}


1		<?php
2
3		namespace sweetrdf\InMemoryStoreSqlite;
4
5		use Exception;
6
7		final class Logger
8		{
9		/**
10		* @var array<string,array<integer,mixed>>
11		*/
12		private array $entries = [];
13
14	117	public function __construct()
15		{
16	117	$this->entries = [
17		'error' => [],
18		'warning' => [],
19		];
20	117	}
21
22	3	public function error(string $message, array $context = []): void
23		{
24	3	$this->log('error', $message, $context);
25	3	}
26
27	3	public function warning(string $message, array $context = []): void
28		{
29	3	$this->log('warning', $message, $context);
30	3	}
31
32	3	private function log($level, $message, array $context = []): void
33		{
34	3	if (!isset($this->entries[$level])) {
35		$this->entries[$level] = [];
36		}
37
38	3	$this->entries[$level][] = ['message' => $message, 'context' => $context];
39	3	}
40
41	5	public function getEntries(?string $level = null): array
42		{
43	5	if (null !== $level && isset($this->entries[$level])) {
44	2	return $this->entries[$level];
45	5	} elseif (null == $level) {
		0 ignored issues – show Bug introduced 2021-03-12 09:14 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing `$level` of type `null\|string` against `null`; this is ambiguous if the string can be empty. Consider using a strict comparison `===` instead. Loading history...
46	3	return $this->entries;
47		}
48
49	2	throw new Exception('Level '.$level.' not set.');
50		}
51
52	2	public function hasEntries(?string $level = null): bool
53		{
54	2	return 0 < \count($this->getEntries($level));
55		}
56		}
57

sweetrdf / in-memory-store-sqlite

Branch — extract-store (f24e42)

Logger::getEntries() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like