Context - Code Metrics - Inspection of "Feature swoole" - SeasX/seas-logger - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#12)

by wujunze

created 2019-09-11 10:30 UTC

Context A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
eloc	5
c	1
b	0
f	1
dl	0
loc	35
rs	10
wmc	5

4 Methods

Rating	Name	Size	Complexity
A	has()	3	1
A	get()	3	2
A	delete()	3	1
A	set()	3	1

<?php
declare(strict_types=1);

namespace Seasx\SeasLogger;


use Co;

/**
 * Class Context
 * @package Seasx\SeasLogger
 */
class Context
{
    /**
     * @param string $name
     * @param $value
     */
    public static function set(string $name, $value): void
    {
        Co::getContext()[$name] = $value;
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }

    /**
     * @param string $name
     * @return mixed
     */
    public static function get(string $name)
    {
        return isset(Co::getContext()[$name]) ? Co::getContext()[$name] : null;
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }

    /**
     * @param string $name
     * @return bool
     */
    public static function has(string $name): bool
    {
        return isset(Co::getContext()[$name]);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }

    /**
     * @param string $name
     */
    public static function delete(string $name): void
    {
        unset(Co::getContext()[$name]);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }
}

1			<?php
2			declare(strict_types=1);
3
4			namespace Seasx\SeasLogger;
5
6
7			use Co;
8
9			/**
10			* Class Context
11			* @package Seasx\SeasLogger
12			*/
13			class Context
14			{
15			/**
16			* @param string $name
17			* @param $value
18			*/
19			public static function set(string $name, $value): void
20			{
21			Co::getContext()[$name] = $value;
			0 ignored issues – show Bug introduced 2019-09-11 10:33 UTC by Report Bug Copy Issue Report Are you sure the usage of `Co::getContext()` targeting `Swoole\Coroutine::getContext()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
22			}
23
24			/**
25			* @param string $name
26			* @return mixed
27			*/
28			public static function get(string $name)
29			{
30			return isset(Co::getContext()[$name]) ? Co::getContext()[$name] : null;
			0 ignored issues – show Bug introduced 2019-09-11 10:33 UTC by Report Bug Copy Issue Report Are you sure the usage of `Co::getContext()` targeting `Swoole\Coroutine::getContext()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
31			}
32
33			/**
34			* @param string $name
35			* @return bool
36			*/
37			public static function has(string $name): bool
38			{
39			return isset(Co::getContext()[$name]);
			0 ignored issues – show Bug introduced 2019-09-11 10:33 UTC by Report Bug Copy Issue Report Are you sure the usage of `Co::getContext()` targeting `Swoole\Coroutine::getContext()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
40			}
41
42			/**
43			* @param string $name
44			*/
45			public static function delete(string $name): void
46			{
47			unset(Co::getContext()[$name]);
			0 ignored issues – show Bug introduced 2019-09-11 10:33 UTC by Report Bug Copy Issue Report Are you sure the usage of `Co::getContext()` targeting `Swoole\Coroutine::getContext()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
48			}
49			}

SeasX / seas-logger

Pull Request — master (#12)

Context A

Complexity

Size/Duplication

Importance

4 Methods

Duplication Side-by-Side

Filter issues like