TopicName::setTopicName() - Code Metrics - Inspection of "Separate TopicFilter and TopicName" - unreal4u/mqtt - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d37c8b...c70816 )

by Camilo

created 2018-04-05 21:52 UTC

TopicName::setTopicName() A

↳ Parent: TopicName

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	11
ccs	6
cts	6
cp	1
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	1
crap	2

<?php

declare(strict_types=1);

namespace unreal4u\MQTT\DataTypes;

use unreal4u\MQTT\Internals\GeneralTopicRules;

/**
 * When the client wants to send a message to a topic, this is done by adding a topic NAME.
 */
final class TopicName extends GeneralTopicRules
{
    /**
     * The TopicName identifies the information channel to which payload data is published.
     *
     * @see http://docs.oasis-open.org/mqtt/mqtt/v3.1.1/os/mqtt-v3.1.1-os.html#_Toc398718106
     * @var string
     */
    private $topicName;

    /**
     * TopicName constructor.
     * @param string $topicName
     * @throws \OutOfBoundsException
     * @throws \unreal4u\MQTT\Exceptions\InvalidQoSLevel
     * @throws \InvalidArgumentException
     */
    public function __construct(string $topicName)
    {
        $this->setTopicName($topicName);
    }

    /**
     * Contains the name of the TopicFilter Filter
     *
     * @param string $topicName
     * @return TopicFilter
     * @throws \OutOfBoundsException
     * @throws \InvalidArgumentException
     */
    private function setTopicName(string $topicName): self
    {
        $this->generalRulesCheck($topicName);

        // A topic name has some additional checks, as no wildcard characters are allowed
        if (strpbrk($topicName, '#+') !== false) {
            throw new \InvalidArgumentException('Topic names can not contain wildcard characters');
        }

        $this->topicName = $topicName;
        return $this;

    }

    /**
     * @return string
     */
    public function getTopicName(): string
    {
        return $this->topicName;
    }

    public function __toString()
    {
        return $this->getTopicName();
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace unreal4u\MQTT\DataTypes;
6
7		use unreal4u\MQTT\Internals\GeneralTopicRules;
8
9		/**
10		* When the client wants to send a message to a topic, this is done by adding a topic NAME.
11		*/
12		final class TopicName extends GeneralTopicRules
13		{
14		/**
15		* The TopicName identifies the information channel to which payload data is published.
16		*
17		* @see http://docs.oasis-open.org/mqtt/mqtt/v3.1.1/os/mqtt-v3.1.1-os.html#_Toc398718106
18		* @var string
19		*/
20		private $topicName;
21
22		/**
23		* TopicName constructor.
24		* @param string $topicName
25		* @throws \OutOfBoundsException
26		* @throws \unreal4u\MQTT\Exceptions\InvalidQoSLevel
27		* @throws \InvalidArgumentException
28		*/
29	48	public function __construct(string $topicName)
30		{
31	48	$this->setTopicName($topicName);
32	47	}
33
34		/**
35		* Contains the name of the TopicFilter Filter
36		*
37		* @param string $topicName
38		* @return TopicFilter
39		* @throws \OutOfBoundsException
40		* @throws \InvalidArgumentException
41		*/
42	48	private function setTopicName(string $topicName): self
43		{
44	48	$this->generalRulesCheck($topicName);
45
46		// A topic name has some additional checks, as no wildcard characters are allowed
47	48	if (strpbrk($topicName, '#+') !== false) {
48	1	throw new \InvalidArgumentException('Topic names can not contain wildcard characters');
49		}
50
51	47	$this->topicName = $topicName;
52	47	return $this;
		0 ignored issues – show Bug Best Practice introduced 2018-04-05 21:55 UTC by Report Bug Copy Issue Report The expression `return $this` returns the type `unreal4u\MQTT\DataTypes\TopicName` which is incompatible with the documented return type `unreal4u\MQTT\DataTypes\TopicFilter`. Loading history...
53		}
54
55		/**
56		* @return string
57		*/
58	15	public function getTopicName(): string
59		{
60	15	return $this->topicName;
61		}
62
63	1	public function __toString()
64		{
65	1	return $this->getTopicName();
66		}
67		}
68

unreal4u / mqtt

Push — master ( d37c8b...c70816 )

TopicName::setTopicName() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like