GenericMessage::getRaw() - Code Metrics - Buttress/IRC - Measure and Improve Code Quality continuously with Scrutinizer

GenericMessage::getRaw() B
last analyzed 2017-02-27 18:54 UTC

↳ Parent: GenericMessage

Complexity

Conditions	4
Paths	6

Size

Total Lines	22
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	13
nc	6
nop	0
dl	0
loc	22
rs	8.9197
c	0
b	0
f	0

<?php
namespace Buttress\IRC\Message;

use Buttress\IRC\Connection\ConnectionInterface;

class GenericMessage implements MessageInterface
{

    /**
     * @var array
     */
    protected $params;

    /**
     * @var String
     */
    protected $command;

    /**
     * @var String
     */
    protected $prefix;

    /**
     * @var ConnectionInterface
     */
    protected $connection;

    function __construct($command, $prefix = "", array $params = array(), $connection = null)

    {
        $this->command = $command;
        $this->connection = $connection;
        $this->params = $params;
        $this->prefix = $prefix;
    }

    /**
     * @return ConnectionInterface
     */
    public function getConnection()
    {
        return $this->connection;
    }

    /**
     * @param ConnectionInterface $connection
     * @return void
     */
    public function setConnection(ConnectionInterface $connection)
    {
        $this->connection = $connection;
    }

    public function getRaw()
    {
        $command = strtoupper($this->getCommand());

        if ($prefix = $this->getPrefix()) {
            $raw = ":{$prefix} {$command}";
        } else {
            $raw = $this->getCommand();
        }

        if ($params = $this->getParams()) {
            $last = array_pop($params);

            $param_string = implode(' ', $params);
            if ($param_string) {
                $raw .= " {$param_string}";
            }
            $raw .= " :{$last}";
        }

        return $raw;
    }

    /**
     * @return string
     */
    public function getCommand()
    {
        return $this->command;
    }

    /**
     * @return string
     */
    public function getPrefix()
    {
        return $this->prefix;
    }

    /**
     * @param string $prefix
     * @return void
     */
    public function setPrefix($prefix)
    {
        $this->prefix = $prefix;
    }

    /**
     * @return array
     */
    public function getParams()
    {
        return $this->params;
    }

}


1		<?php
2		namespace Buttress\IRC\Message;
3
4		use Buttress\IRC\Connection\ConnectionInterface;
5
6		class GenericMessage implements MessageInterface
7		{
8
9		/**
10		* @var array
11		*/
12		protected $params;
13
14		/**
15		* @var String
16		*/
17		protected $command;
18
19		/**
20		* @var String
21		*/
22		protected $prefix;
23
24		/**
25		* @var ConnectionInterface
26		*/
27		protected $connection;
28
29	View Code Duplication	function __construct($command, $prefix = "", array $params = array(), $connection = null)
		0 ignored issues – show Best Practice introduced 2017-02-22 21:07 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history... Duplication introduced 2017-02-22 21:07 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
30		{
31		$this->command = $command;
32		$this->connection = $connection;
33		$this->params = $params;
34		$this->prefix = $prefix;
35		}
36
37		/**
38		* @return ConnectionInterface
39		*/
40		public function getConnection()
41		{
42		return $this->connection;
43		}
44
45		/**
46		* @param ConnectionInterface $connection
47		* @return void
48		*/
49		public function setConnection(ConnectionInterface $connection)
50		{
51		$this->connection = $connection;
52		}
53
54		public function getRaw()
55		{
56		$command = strtoupper($this->getCommand());
57
58		if ($prefix = $this->getPrefix()) {
59		$raw = ":{$prefix} {$command}";
60		} else {
61		$raw = $this->getCommand();
62		}
63
64		if ($params = $this->getParams()) {
65		$last = array_pop($params);
66
67		$param_string = implode(' ', $params);
68		if ($param_string) {
69		$raw .= " {$param_string}";
70		}
71		$raw .= " :{$last}";
72		}
73
74		return $raw;
75		}
76
77		/**
78		* @return string
79		*/
80		public function getCommand()
81		{
82		return $this->command;
83		}
84
85		/**
86		* @return string
87		*/
88		public function getPrefix()
89		{
90		return $this->prefix;
91		}
92
93		/**
94		* @param string $prefix
95		* @return void
96		*/
97		public function setPrefix($prefix)
98		{
99		$this->prefix = $prefix;
100		}
101
102		/**
103		* @return array
104		*/
105		public function getParams()
106		{
107		return $this->params;
108		}
109
110		}
111

Buttress / IRC

GenericMessage::getRaw() B last analyzed 2017-02-27 18:54 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

GenericMessage::getRaw() B
last analyzed 2017-02-27 18:54 UTC