Swift_CharacterStream_NgCharacterStream - Code Metrics - Inspection of "Merge pull request #18 from voku/analysis-Xpne6z" - voku/swiftmailer - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 5.x ( 0b26a1...a498b0 )

by Lars

created 2016-01-14 12:31 UTC

Swift_CharacterStream_NgCharacterStream A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	257
Duplicated Lines	4.28 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

81.82%

Metric	Value
wmc	26
lcom	1
cbo	3
dl	11
loc	257
ccs	81
cts	99
cp	0.8182
rs	10

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	setCharacterSet()	0	6	1
A	setCharacterReaderFactory()	0	4	1
A	flushContents()	0	8	1
A	importString()	0	5	1
A	readBytes()	11	11	2
C	read()	0	52	12
A	setPointer()	0	7	2
A	write()	0	20	3
A	importByteStream()	0	9	2

How to fix Duplicated Code

<?php

/*
 * This file is part of SwiftMailer.
 * (c) 2004-2009 Chris Corbyn
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

/**
 * A CharacterStream implementation which stores characters in an internal array.
 *
 * @author Xavier De Cock <[email protected]>
 */
class Swift_CharacterStream_NgCharacterStream implements Swift_CharacterStream
namespace YourVendor;

class YourClass { }
{
    /**
     * The char reader (lazy-loaded) for the current charset.
     *
     * @var Swift_CharacterReader
     */
    private $_charReader;

    /**
     * A factory for creating CharacterReader instances.
     *
     * @var Swift_CharacterReaderFactory
     */
    private $_charReaderFactory;

    /**
     * The character set this stream is using.
     *
     * @var string
     */
    private $_charset;

    /**
     * The data's stored as-is.
     *
     * @var string
     */
    private $_datas = '';

    /**
     * Number of bytes in the stream.
     *
     * @var int
     */
    private $_datasSize = 0;

    /**
     * Map.
     *
     * @var mixed
     */
    private $_map;

    /**
     * Map Type.
     *
     * @var int
     */
    private $_mapType = 0;

    /**
     * Number of characters in the stream.
     *
     * @var int
     */
    private $_charCount = 0;

    /**
     * Position in the stream.
     *
     * @var int
     */
    private $_currentPos = 0;

    /**
     * Constructor.
     *
     * @param Swift_CharacterReaderFactory $factory
     * @param string                       $charset
     */
    public function __construct(Swift_CharacterReaderFactory $factory, $charset)
    {
        $this->setCharacterReaderFactory($factory);
        $this->setCharacterSet($charset);
    }

    /* -- Changing parameters of the stream -- */

    /**
     * Set the character set used in this CharacterStream.
     *
     * @param string $charset
     */
    public function setCharacterSet($charset)
    {
        $this->_charset = $charset;
        $this->_charReader = null;
        $this->_mapType = 0;
    }

    /**
     * Set the CharacterReaderFactory for multi charset support.
     *
     * @param Swift_CharacterReaderFactory $factory
     */
    public function setCharacterReaderFactory(Swift_CharacterReaderFactory $factory)
    {
        $this->_charReaderFactory = $factory;
    }

    /**
     * @see Swift_CharacterStream::flushContents()
     */
    public function flushContents()
    {
        $this->_datas = null;
        $this->_map = null;
        $this->_charCount = 0;
        $this->_currentPos = 0;
        $this->_datasSize = 0;
    }

    /**
     * @see Swift_CharacterStream::importByteStream()
     *
     * @param Swift_OutputByteStream $os
     */
    public function importByteStream(Swift_OutputByteStream $os)
    {
        $this->flushContents();
        $blocks = 512;
        $os->setReadPointer(0);
        while (false !== ($read = $os->read($blocks))) {
            $this->write($read);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
        }
    }

    /**
     * @see Swift_CharacterStream::importString()
     *
     * @param string $string
     */
    public function importString($string)
    {
        $this->flushContents();
        $this->write($string);
    }

    /**
     * @see Swift_CharacterStream::read()
     *
     * @param int $length
     *
     * @return string|false false on error
     */
    public function read($length)
    {
        if ($this->_currentPos >= $this->_charCount) {
            return false;
class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}
        }

        $ret = false;

        $length = $this->_currentPos + $length > $this->_charCount ? $this->_charCount - $this->_currentPos : $length;
        switch ($this->_mapType) {
            case Swift_CharacterReader::MAP_TYPE_FIXED_LEN:
                $len = $length * $this->_map;
                $ret = substr($this->_datas, $this->_currentPos * $this->_map, $len);
                $this->_currentPos += $length;
                break;

            case Swift_CharacterReader::MAP_TYPE_INVALID:
                $ret = '';
                for (; $this->_currentPos < $length; ++$this->_currentPos) {
                    if (isset($this->_map[$this->_currentPos])) {
                        $ret .= '?';
                    } else {
                        $ret .= $this->_datas[$this->_currentPos];
                    }
                }
                break;

            case Swift_CharacterReader::MAP_TYPE_POSITIONS:
                $end = $this->_currentPos + $length;
                $end = $end > $this->_charCount ? $this->_charCount : $end;
                $ret = '';

                $start = 0;
                if ($this->_currentPos > 0) {
                    $start = $this->_map['p'][$this->_currentPos - 1];
                }

                $to = $start;
                for (; $this->_currentPos < $end; ++$this->_currentPos) {
                    if (isset($this->_map['i'][$this->_currentPos])) {
                        $ret .= substr($this->_datas, $start, $to - $start) . '?';
                        $start = $this->_map['p'][$this->_currentPos];
                    } else {
                        $to = $this->_map['p'][$this->_currentPos];
                    }
                }

                $ret .= substr($this->_datas, $start, $to - $start);
                break;
        }

        return $ret;
    }

    /**
     * @see Swift_CharacterStream::readBytes()
     *
     * @param int $length
     *
     * @return integer[]|false false on error
     */
    public function readBytes($length)

    {
        $read = $this->read($length);
        if ($read !== false) {
            $ret = array_map('ord', str_split($read, 1));

            return $ret;
        }

        return false;
class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}
    }

    /**
     * @see Swift_CharacterStream::setPointer()
     *
     * @param int $charOffset
     */
    public function setPointer($charOffset)
    {
        if ($this->_charCount < $charOffset) {
            $charOffset = $this->_charCount;
        }
        $this->_currentPos = $charOffset;
    }

    /**
     * @see Swift_CharacterStream::write()
     *
     * @param string $chars
     */
    public function write($chars)
    {
        if (!isset($this->_charReader)) {
            $this->_charReader = $this->_charReaderFactory->getReaderFor($this->_charset);
            $this->_map = array();
            $this->_mapType = $this->_charReader->getMapType();
        }

        $ignored = '';
        $this->_datas .= $chars;
        $this->_charCount += $this->_charReader->getCharPositions(
            substr($this->_datas, $this->_datasSize), $this->_datasSize, $this->_map, $ignored
        );

        if ($ignored !== false) {
            $this->_datasSize = strlen($this->_datas) - strlen($ignored);
        } else {
            $this->_datasSize = strlen($this->_datas);
        }
    }
}


1			<?php
2
3			/*
4			* This file is part of SwiftMailer.
5			* (c) 2004-2009 Chris Corbyn
6			*
7			* For the full copyright and license information, please view the LICENSE
8			* file that was distributed with this source code.
9			*/
10
11			/**
12			* A CharacterStream implementation which stores characters in an internal array.
13			*
14			* @author Xavier De Cock <[email protected]>
15			*/
16			class Swift_CharacterStream_NgCharacterStream implements Swift_CharacterStream
			0 ignored issues – show Coding Style Compatibility introduced 2015-11-22 11:38 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
17			{
18			/**
19			* The char reader (lazy-loaded) for the current charset.
20			*
21			* @var Swift_CharacterReader
22			*/
23			private $_charReader;
24
25			/**
26			* A factory for creating CharacterReader instances.
27			*
28			* @var Swift_CharacterReaderFactory
29			*/
30			private $_charReaderFactory;
31
32			/**
33			* The character set this stream is using.
34			*
35			* @var string
36			*/
37			private $_charset;
38
39			/**
40			* The data's stored as-is.
41			*
42			* @var string
43			*/
44			private $_datas = '';
45
46			/**
47			* Number of bytes in the stream.
48			*
49			* @var int
50			*/
51			private $_datasSize = 0;
52
53			/**
54			* Map.
55			*
56			* @var mixed
57			*/
58			private $_map;
59
60			/**
61			* Map Type.
62			*
63			* @var int
64			*/
65			private $_mapType = 0;
66
67			/**
68			* Number of characters in the stream.
69			*
70			* @var int
71			*/
72			private $_charCount = 0;
73
74			/**
75			* Position in the stream.
76			*
77			* @var int
78			*/
79			private $_currentPos = 0;
80
81			/**
82			* Constructor.
83			*
84			* @param Swift_CharacterReaderFactory $factory
85			* @param string $charset
86			*/
87	137		public function __construct(Swift_CharacterReaderFactory $factory, $charset)
88			{
89	137		$this->setCharacterReaderFactory($factory);
90	137		$this->setCharacterSet($charset);
91	137		}
92
93			/* -- Changing parameters of the stream -- */
94
95			/**
96			* Set the character set used in this CharacterStream.
97			*
98			* @param string $charset
99			*/
100	137		public function setCharacterSet($charset)
101			{
102	137		$this->_charset = $charset;
103	137		$this->_charReader = null;
104	137		$this->_mapType = 0;
105	137		}
106
107			/**
108			* Set the CharacterReaderFactory for multi charset support.
109			*
110			* @param Swift_CharacterReaderFactory $factory
111			*/
112	137		public function setCharacterReaderFactory(Swift_CharacterReaderFactory $factory)
113			{
114	137		$this->_charReaderFactory = $factory;
115	137		}
116
117			/**
118			* @see Swift_CharacterStream::flushContents()
119			*/
120	63		public function flushContents()
121			{
122	63		$this->_datas = null;
123	63		$this->_map = null;
124	63		$this->_charCount = 0;
125	63		$this->_currentPos = 0;
126	63		$this->_datasSize = 0;
127	63		}
128
129			/**
130			* @see Swift_CharacterStream::importByteStream()
131			*
132			* @param Swift_OutputByteStream $os
133			*/
134	4		public function importByteStream(Swift_OutputByteStream $os)
135			{
136	4		$this->flushContents();
137	4		$blocks = 512;
138	4		$os->setReadPointer(0);
139	4		while (false !== ($read = $os->read($blocks))) {
140	4		$this->write($read);
			0 ignored issues – show Bug introduced 2015-11-22 11:38 UTC by Report Bug Copy Issue Report It seems like `$read` defined by `$os->read($blocks)` on line `139` can also be of type `boolean`; however, `Swift_CharacterStream_NgCharacterStream::write()` does only seem to accept `string`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
141	4		}
142	4		}
143
144			/**
145			* @see Swift_CharacterStream::importString()
146			*
147			* @param string $string
148			*/
149	59		public function importString($string)
150			{
151	59		$this->flushContents();
152	59		$this->write($string);
153	59		}
154
155			/**
156			* @see Swift_CharacterStream::read()
157			*
158			* @param int $length
159			*
160			* @return string\|false false on error
161			*/
162	63		public function read($length)
163			{
164	63		if ($this->_currentPos >= $this->_charCount) {
165	61		return false;
			0 ignored issues – show Bug Best Practice introduced 2015-11-22 11:38 UTC by Report Bug Copy Issue Report The return type of `return false;` (`false`) is incompatible with the return type declared by the interface `Swift_CharacterStream::read` of type `string`. If you return a value from a function or method, it should be a sub-type of the type that is given by the parent type f.e. an interface, or abstract method. This is more formally defined by the Lizkov substitution principle, and guarantees that classes that depend on the parent type can use any instance of a child type interchangably. This principle also belongs to the SOLID principles for object oriented design. Let’s take a look at an example: class Author { private $name; public function __construct($name) { $this->name = $name; } public function getName() { return $this->name; } } abstract class Post { public function getAuthor() { return 'Johannes'; } } class BlogPost extends Post { public function getAuthor() { return new Author('Johannes'); } } class ForumPost extends Post { /* ... */ } function my_function(Post $post) { echo strtoupper($post->getAuthor()); } Our function `my_function` expects a `Post` object, and outputs the author of the post. The base class `Post` returns a simple string and outputting a simple string will work just fine. However, the child class `BlogPost` which is a sub-type of `Post` instead decided to return an `object`, and is therefore violating the SOLID principles. If a `BlogPost` were passed to `my_function`, PHP would not complain, but ultimately fail when executing the `strtoupper` call in its body. Loading history...
166			}
167
168	63		$ret = false;
			0 ignored issues – show Comprehensibility Best Practice introduced 2015-11-22 11:38 UTC by Report Bug Copy Issue Report The expression `false;` of type `false` adds `false` to the return on line 212 which is incompatible with the return type declared by the interface `Swift_CharacterStream::read` of type `string`. It seems like you forgot to handle an error condition. Loading history...
169	63		$length = $this->_currentPos + $length > $this->_charCount ? $this->_charCount - $this->_currentPos : $length;
170	63		switch ($this->_mapType) {
171	63		case Swift_CharacterReader::MAP_TYPE_FIXED_LEN:
172	35		$len = $length * $this->_map;
173	35		$ret = substr($this->_datas, $this->_currentPos * $this->_map, $len);
174	35		$this->_currentPos += $length;
175	35		break;
176
177	29		case Swift_CharacterReader::MAP_TYPE_INVALID:
178			$ret = '';
179			for (; $this->_currentPos < $length; ++$this->_currentPos) {
180			if (isset($this->_map[$this->_currentPos])) {
181			$ret .= '?';
182			} else {
183			$ret .= $this->_datas[$this->_currentPos];
184			}
185			}
186			break;
187
188	29		case Swift_CharacterReader::MAP_TYPE_POSITIONS:
189	29		$end = $this->_currentPos + $length;
190	29		$end = $end > $this->_charCount ? $this->_charCount : $end;
191	29		$ret = '';
192
193	29		$start = 0;
194	29		if ($this->_currentPos > 0) {
195	2		$start = $this->_map['p'][$this->_currentPos - 1];
196	2		}
197
198	29		$to = $start;
199	29		for (; $this->_currentPos < $end; ++$this->_currentPos) {
200	29		if (isset($this->_map['i'][$this->_currentPos])) {
201			$ret .= substr($this->_datas, $start, $to - $start) . '?';
202			$start = $this->_map['p'][$this->_currentPos];
203			} else {
204	29		$to = $this->_map['p'][$this->_currentPos];
205			}
206	29		}
207
208	29		$ret .= substr($this->_datas, $start, $to - $start);
209	29		break;
210	63		}
211
212	63		return $ret;
213			}
214
215			/**
216			* @see Swift_CharacterStream::readBytes()
217			*
218			* @param int $length
219			*
220			* @return integer[]\|false false on error
221			*/
222	63	View Code Duplication	public function readBytes($length)
			0 ignored issues – show Duplication introduced 2015-11-22 11:38 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...
223			{
224	63		$read = $this->read($length);
225	63		if ($read !== false) {
226	63		$ret = array_map('ord', str_split($read, 1));
227
228	63		return $ret;
229			}
230
231	63		return false;
			0 ignored issues – show Bug Best Practice introduced 2015-11-22 11:38 UTC by Report Bug Copy Issue Report The return type of `return false;` (`false`) is incompatible with the return type declared by the interface `Swift_CharacterStream::readBytes` of type `integer[]`. If you return a value from a function or method, it should be a sub-type of the type that is given by the parent type f.e. an interface, or abstract method. This is more formally defined by the Lizkov substitution principle, and guarantees that classes that depend on the parent type can use any instance of a child type interchangably. This principle also belongs to the SOLID principles for object oriented design. Let’s take a look at an example: class Author { private $name; public function __construct($name) { $this->name = $name; } public function getName() { return $this->name; } } abstract class Post { public function getAuthor() { return 'Johannes'; } } class BlogPost extends Post { public function getAuthor() { return new Author('Johannes'); } } class ForumPost extends Post { /* ... */ } function my_function(Post $post) { echo strtoupper($post->getAuthor()); } Our function `my_function` expects a `Post` object, and outputs the author of the post. The base class `Post` returns a simple string and outputting a simple string will work just fine. However, the child class `BlogPost` which is a sub-type of `Post` instead decided to return an `object`, and is therefore violating the SOLID principles. If a `BlogPost` were passed to `my_function`, PHP would not complain, but ultimately fail when executing the `strtoupper` call in its body. Loading history...
232			}
233
234			/**
235			* @see Swift_CharacterStream::setPointer()
236			*
237			* @param int $charOffset
238			*/
239			public function setPointer($charOffset)
240			{
241			if ($this->_charCount < $charOffset) {
242			$charOffset = $this->_charCount;
243			}
244			$this->_currentPos = $charOffset;
245			}
246
247			/**
248			* @see Swift_CharacterStream::write()
249			*
250			* @param string $chars
251			*/
252	63		public function write($chars)
253			{
254	63		if (!isset($this->_charReader)) {
255	63		$this->_charReader = $this->_charReaderFactory->getReaderFor($this->_charset);
256	63		$this->_map = array();
257	63		$this->_mapType = $this->_charReader->getMapType();
258	63		}
259
260	63		$ignored = '';
261	63		$this->_datas .= $chars;
262	63		$this->_charCount += $this->_charReader->getCharPositions(
263	63		substr($this->_datas, $this->_datasSize), $this->_datasSize, $this->_map, $ignored
264	63		);
265
266	63		if ($ignored !== false) {
267	63		$this->_datasSize = strlen($this->_datas) - strlen($ignored);
268	63		} else {
269			$this->_datasSize = strlen($this->_datas);
270			}
271	63		}
272			}
273

voku / swiftmailer

Push — 5.x ( 0b26a1...a498b0 )

Swift_CharacterStream_NgCharacterStream A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

10 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like