File::store() - Code Metrics - bluetree-service/simple-log - Measure and Improve Code Quality continuously with Scrutinizer

File::store() A
last analyzed 2021-11-23 20:46 UTC

↳ Parent: File

Complexity

Conditions	5
Paths	9

Size

Total Lines	24
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	5

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	5
eloc	12
c	1
b	0
f	0
nc	9
nop	2
dl	0
loc	24
rs	9.5555
ccs	13
cts	13
cp	1
crap	5

<?php

declare(strict_types=1);

namespace SimpleLog\Storage;

use SimpleLog\LogException;

class File implements StorageInterface
{
    /**
     * @var array
     */
    protected $params = [];

    /**
     * @param array $params
     */
    public function __construct(array $params)
    {
        $this->params = $params;
    }

    /**
     * @param string $message
     * @param string $level
     * @throws LogException
     * @return $this
     */
    public function store(string $message, string $level): StorageInterface
    {
        $flag = 0;
        $logFile = $this->params['log_path'] . DIRECTORY_SEPARATOR . $level . '.log';

        if (!\file_exists($this->params['log_path'])) {
            $bool = @\mkdir($this->params['log_path']);

            if (!$bool) {
                throw new LogException('Unable to create log directory: ' . $this->params['log_path']);
            }
        }

        if (\file_exists($logFile)) {
            $flag = FILE_APPEND;
        }

        $bool = @\file_put_contents($logFile, $message, $flag | LOCK_EX);

        if (!$bool) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
            throw new LogException('Unable to save log file: ' . $logFile);
        }

        return $this;
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace SimpleLog\Storage;
6
7		use SimpleLog\LogException;
8
9		class File implements StorageInterface
10		{
11		/**
12		* @var array
13		*/
14		protected $params = [];
15
16		/**
17		* @param array $params
18		*/
19	17	public function __construct(array $params)
20		{
21	17	$this->params = $params;
22	17	}
23
24		/**
25		* @param string $message
26		* @param string $level
27		* @throws LogException
28		* @return $this
29		*/
30	15	public function store(string $message, string $level): StorageInterface
31		{
32	15	$flag = 0;
33	15	$logFile = $this->params['log_path'] . DIRECTORY_SEPARATOR . $level . '.log';
34
35	15	if (!\file_exists($this->params['log_path'])) {
36	13	$bool = @\mkdir($this->params['log_path']);
37
38	13	if (!$bool) {
39	1	throw new LogException('Unable to create log directory: ' . $this->params['log_path']);
40		}
41		}
42
43	14	if (\file_exists($logFile)) {
44	3	$flag = FILE_APPEND;
45		}
46
47	14	$bool = @\file_put_contents($logFile, $message, $flag \| LOCK_EX);
48
49	14	if (!$bool) {
		0 ignored issues – show Bug Best Practice introduced 2021-11-23 20:19 UTC by Report Bug Copy Issue Report The expression `$bool` of type `false\|integer` is loosely compared to `false`; this is ambiguous if the integer can be 0. You might want to explicitly use `=== false` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
50	1	throw new LogException('Unable to save log file: ' . $logFile);
51		}
52
53	13	return $this;
54		}
55		}
56

bluetree-service / simple-log

File::store() A last analyzed 2021-11-23 20:46 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

File::store() A
last analyzed 2021-11-23 20:46 UTC