CodePointString - Code Metrics - Inspection of "Add new config options: {{what}} and {{where}}" - deployphp/deployer - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 5cefd1...492078 )

by Anton

created 2024-10-22 06:54 UTC

CodePointString F

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	244
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	119
c	1
b	0
f	0
dl	0
loc	244
rs	2.64
wmc	72

15 Methods

Rating	Name	Size	Complexity
A	append()	10	3
A	__construct()	7	3
A	codePointsAt()	5	3
A	chunk()	26	5
B	equalsTo()	15	7
A	replace()	19	6
A	length()	3	1
A	prepend()	10	3
B	split()	29	7
A	splice()	12	4
B	startsWith()	19	7
B	indexOf()	17	7
B	endsWith()	19	8
B	indexOfLast()	17	7
A	slice()	6	1

How to fix Complexity

<?php

/*
 * This file is part of the Symfony package.
 *
 * (c) Fabien Potencier <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Symfony\Component\String;

use Symfony\Component\String\Exception\ExceptionInterface;
use Symfony\Component\String\Exception\InvalidArgumentException;

/**
 * Represents a string of Unicode code points encoded as UTF-8.
 *
 * @author Nicolas Grekas <[email protected]>
 * @author Hugo Hamon <[email protected]>
 *
 * @throws ExceptionInterface
 */
class CodePointString extends AbstractUnicodeString
{
    public function __construct(string $string = '')
    {
        if ('' !== $string && !preg_match('//u', $string)) {
            throw new InvalidArgumentException('Invalid UTF-8 string.');
        }

        $this->string = $string;
    }

    public function append(string ...$suffix): AbstractString
    {
        $str = clone $this;
        $str->string .= 1 >= \count($suffix) ? ($suffix[0] ?? '') : implode('', $suffix);

        if (!preg_match('//u', $str->string)) {
            throw new InvalidArgumentException('Invalid UTF-8 string.');
        }

        return $str;
    }

    public function chunk(int $length = 1): array
    {
        if (1 > $length) {
            throw new InvalidArgumentException('The chunk length must be greater than zero.');
        }

        if ('' === $this->string) {
            return [];
        }

        $rx = '/(';
        while (65535 < $length) {
            $rx .= '.{65535}';
            $length -= 65535;
        }
        $rx .= '.{'.$length.'})/us';

        $str = clone $this;
        $chunks = [];

        foreach (preg_split($rx, $this->string, -1, \PREG_SPLIT_DELIM_CAPTURE | \PREG_SPLIT_NO_EMPTY) as $chunk) {
            $str->string = $chunk;
            $chunks[] = clone $str;
        }

        return $chunks;
    }

    public function codePointsAt(int $offset): array
    {
        $str = $offset ? $this->slice($offset, 1) : $this;

        return '' === $str->string ? [] : [mb_ord($str->string, 'UTF-8')];
    }

    public function endsWith($suffix): bool
    {
        if ($suffix instanceof AbstractString) {
            $suffix = $suffix->string;
        } elseif (\is_array($suffix) || $suffix instanceof \Traversable) {
            return parent::endsWith($suffix);
        } else {
            $suffix = (string) $suffix;
        }

        if ('' === $suffix || !preg_match('//u', $suffix)) {
            return false;
        }

        if ($this->ignoreCase) {
            return preg_match('{'.preg_quote($suffix).'$}iuD', $this->string);

        }

        return \strlen($this->string) >= \strlen($suffix) && 0 === substr_compare($this->string, $suffix, -\strlen($suffix));
    }

    public function equalsTo($string): bool
    {
        if ($string instanceof AbstractString) {
            $string = $string->string;
        } elseif (\is_array($string) || $string instanceof \Traversable) {
            return parent::equalsTo($string);
        } else {
            $string = (string) $string;
        }

        if ('' !== $string && $this->ignoreCase) {
            return \strlen($string) === \strlen($this->string) && 0 === mb_stripos($this->string, $string, 0, 'UTF-8');
        }

        return $string === $this->string;
    }

    public function indexOf($needle, int $offset = 0): ?int
    {
        if ($needle instanceof AbstractString) {
            $needle = $needle->string;
        } elseif (\is_array($needle) || $needle instanceof \Traversable) {
            return parent::indexOf($needle, $offset);
        } else {
            $needle = (string) $needle;
        }

        if ('' === $needle) {
            return null;
        }

        $i = $this->ignoreCase ? mb_stripos($this->string, $needle, $offset, 'UTF-8') : mb_strpos($this->string, $needle, $offset, 'UTF-8');

        return false === $i ? null : $i;

    }

    public function indexOfLast($needle, int $offset = 0): ?int
    {
        if ($needle instanceof AbstractString) {
            $needle = $needle->string;
        } elseif (\is_array($needle) || $needle instanceof \Traversable) {
            return parent::indexOfLast($needle, $offset);
        } else {
            $needle = (string) $needle;
        }

        if ('' === $needle) {
            return null;
        }

        $i = $this->ignoreCase ? mb_strripos($this->string, $needle, $offset, 'UTF-8') : mb_strrpos($this->string, $needle, $offset, 'UTF-8');

        return false === $i ? null : $i;

    }

    public function length(): int
    {
        return mb_strlen($this->string, 'UTF-8');
    }

    public function prepend(string ...$prefix): AbstractString
    {
        $str = clone $this;
        $str->string = (1 >= \count($prefix) ? ($prefix[0] ?? '') : implode('', $prefix)).$this->string;

        if (!preg_match('//u', $str->string)) {
            throw new InvalidArgumentException('Invalid UTF-8 string.');
        }

        return $str;
    }

    public function replace(string $from, string $to): AbstractString
    {
        $str = clone $this;

        if ('' === $from || !preg_match('//u', $from)) {
            return $str;
        }

        if ('' !== $to && !preg_match('//u', $to)) {
            throw new InvalidArgumentException('Invalid UTF-8 string.');
        }

        if ($this->ignoreCase) {
            $str->string = implode($to, preg_split('{'.preg_quote($from).'}iuD', $this->string));
        } else {
            $str->string = str_replace($from, $to, $this->string);
        }

        return $str;
    }

    public function slice(int $start = 0, int $length = null): AbstractString
    {
        $str = clone $this;
        $str->string = mb_substr($this->string, $start, $length, 'UTF-8');

        return $str;
    }

    public function splice(string $replacement, int $start = 0, int $length = null): AbstractString
    {
        if (!preg_match('//u', $replacement)) {
            throw new InvalidArgumentException('Invalid UTF-8 string.');
        }

        $str = clone $this;
        $start = $start ? \strlen(mb_substr($this->string, 0, $start, 'UTF-8')) : 0;
        $length = $length ? \strlen(mb_substr($this->string, $start, $length, 'UTF-8')) : $length;
        $str->string = substr_replace($this->string, $replacement, $start, $length ?? \PHP_INT_MAX);

        return $str;
    }

    public function split(string $delimiter, int $limit = null, int $flags = null): array
    {
        if (1 > $limit = $limit ?? \PHP_INT_MAX) {
            throw new InvalidArgumentException('Split limit must be a positive integer.');
        }

        if ('' === $delimiter) {
            throw new InvalidArgumentException('Split delimiter is empty.');
        }

        if (null !== $flags) {
            return parent::split($delimiter.'u', $limit, $flags);
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
        }

        if (!preg_match('//u', $delimiter)) {
            throw new InvalidArgumentException('Split delimiter is not a valid UTF-8 string.');
        }

        $str = clone $this;
        $chunks = $this->ignoreCase
            ? preg_split('{'.preg_quote($delimiter).'}iuD', $this->string, $limit)
            : explode($delimiter, $this->string, $limit);

        foreach ($chunks as &$chunk) {
            $str->string = $chunk;
            $chunk = clone $str;
        }

        return $chunks;
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
    }

    public function startsWith($prefix): bool
    {
        if ($prefix instanceof AbstractString) {
            $prefix = $prefix->string;
        } elseif (\is_array($prefix) || $prefix instanceof \Traversable) {
            return parent::startsWith($prefix);
        } else {
            $prefix = (string) $prefix;
        }

        if ('' === $prefix || !preg_match('//u', $prefix)) {
            return false;
        }

        if ($this->ignoreCase) {
            return 0 === mb_stripos($this->string, $prefix, 0, 'UTF-8');
        }

        return 0 === strncmp($this->string, $prefix, \strlen($prefix));
    }
}


GitHub Access Token became invalid

Push — master ( 5cefd1...492078 )

CodePointString F

Complexity

Size/Duplication

Importance

15 Methods

How to fix Complexity

Complex Class

1			<?php
2
3			/*
4			* This file is part of the Symfony package.
5			*
6			* (c) Fabien Potencier <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Symfony\Component\String;
13
14			use Symfony\Component\String\Exception\ExceptionInterface;
15			use Symfony\Component\String\Exception\InvalidArgumentException;
16
17			/**
18			* Represents a string of Unicode code points encoded as UTF-8.
19			*
20			* @author Nicolas Grekas <[email protected]>
21			* @author Hugo Hamon <[email protected]>
22			*
23			* @throws ExceptionInterface
24			*/
25			class CodePointString extends AbstractUnicodeString
26			{
27			public function __construct(string $string = '')
28			{
29			if ('' !== $string && !preg_match('//u', $string)) {
30			throw new InvalidArgumentException('Invalid UTF-8 string.');
31			}
32
33			$this->string = $string;
34			}
35
36			public function append(string ...$suffix): AbstractString
37			{
38			$str = clone $this;
39			$str->string .= 1 >= \count($suffix) ? ($suffix[0] ?? '') : implode('', $suffix);
40
41			if (!preg_match('//u', $str->string)) {
42			throw new InvalidArgumentException('Invalid UTF-8 string.');
43			}
44
45			return $str;
46			}
47
48			public function chunk(int $length = 1): array
49			{
50			if (1 > $length) {
51			throw new InvalidArgumentException('The chunk length must be greater than zero.');
52			}
53
54			if ('' === $this->string) {
55			return [];
56			}
57
58			$rx = '/(';
59			while (65535 < $length) {
60			$rx .= '.{65535}';
61			$length -= 65535;
62			}
63			$rx .= '.{'.$length.'})/us';
64
65			$str = clone $this;
66			$chunks = [];
67
68			foreach (preg_split($rx, $this->string, -1, \PREG_SPLIT_DELIM_CAPTURE \| \PREG_SPLIT_NO_EMPTY) as $chunk) {
69			$str->string = $chunk;
70			$chunks[] = clone $str;
71			}
72
73			return $chunks;
74			}
75
76			public function codePointsAt(int $offset): array
77			{
78			$str = $offset ? $this->slice($offset, 1) : $this;
79
80			return '' === $str->string ? [] : [mb_ord($str->string, 'UTF-8')];
81			}
82
83			public function endsWith($suffix): bool
84			{
85			if ($suffix instanceof AbstractString) {
86			$suffix = $suffix->string;
87			} elseif (\is_array($suffix) \|\| $suffix instanceof \Traversable) {
88			return parent::endsWith($suffix);
89			} else {
90			$suffix = (string) $suffix;
91			}
92
93			if ('' === $suffix \|\| !preg_match('//u', $suffix)) {
94			return false;
95			}
96
97			if ($this->ignoreCase) {
98			return preg_match('{'.preg_quote($suffix).'$}iuD', $this->string);
			0 ignored issues – show Bug Best Practice introduced 2024-10-12 21:52 UTC by Report Bug Copy Issue Report The expression `return preg_match('{' . ...'$}iuD', $this->string)` returns the type `integer` which is incompatible with the type-hinted return `boolean`. Loading history...
99			}
100
101			return \strlen($this->string) >= \strlen($suffix) && 0 === substr_compare($this->string, $suffix, -\strlen($suffix));
102			}
103
104			public function equalsTo($string): bool
105			{
106			if ($string instanceof AbstractString) {
107			$string = $string->string;
108			} elseif (\is_array($string) \|\| $string instanceof \Traversable) {
109			return parent::equalsTo($string);
110			} else {
111			$string = (string) $string;
112			}
113
114			if ('' !== $string && $this->ignoreCase) {
115			return \strlen($string) === \strlen($this->string) && 0 === mb_stripos($this->string, $string, 0, 'UTF-8');
116			}
117
118			return $string === $this->string;
119			}
120
121			public function indexOf($needle, int $offset = 0): ?int
122			{
123			if ($needle instanceof AbstractString) {
124			$needle = $needle->string;
125			} elseif (\is_array($needle) \|\| $needle instanceof \Traversable) {
126			return parent::indexOf($needle, $offset);
127			} else {
128			$needle = (string) $needle;
129			}
130
131			if ('' === $needle) {
132			return null;
133			}
134
135			$i = $this->ignoreCase ? mb_stripos($this->string, $needle, $offset, 'UTF-8') : mb_strpos($this->string, $needle, $offset, 'UTF-8');
136
137			return false === $i ? null : $i;
			0 ignored issues – show introduced 2024-10-12 21:52 UTC by Report Bug Copy Issue Report The condition `false === $i` is always `false`. Loading history...
138			}
139
140			public function indexOfLast($needle, int $offset = 0): ?int
141			{
142			if ($needle instanceof AbstractString) {
143			$needle = $needle->string;
144			} elseif (\is_array($needle) \|\| $needle instanceof \Traversable) {
145			return parent::indexOfLast($needle, $offset);
146			} else {
147			$needle = (string) $needle;
148			}
149
150			if ('' === $needle) {
151			return null;
152			}
153
154			$i = $this->ignoreCase ? mb_strripos($this->string, $needle, $offset, 'UTF-8') : mb_strrpos($this->string, $needle, $offset, 'UTF-8');
155
156			return false === $i ? null : $i;
			0 ignored issues – show introduced 2024-10-12 21:52 UTC by Report Bug Copy Issue Report The condition `false === $i` is always `false`. Loading history...
157			}
158
159			public function length(): int
160			{
161			return mb_strlen($this->string, 'UTF-8');
162			}
163
164			public function prepend(string ...$prefix): AbstractString
165			{
166			$str = clone $this;
167			$str->string = (1 >= \count($prefix) ? ($prefix[0] ?? '') : implode('', $prefix)).$this->string;
168
169			if (!preg_match('//u', $str->string)) {
170			throw new InvalidArgumentException('Invalid UTF-8 string.');
171			}
172
173			return $str;
174			}
175
176			public function replace(string $from, string $to): AbstractString
177			{
178			$str = clone $this;
179
180			if ('' === $from \|\| !preg_match('//u', $from)) {
181			return $str;
182			}
183
184			if ('' !== $to && !preg_match('//u', $to)) {
185			throw new InvalidArgumentException('Invalid UTF-8 string.');
186			}
187
188			if ($this->ignoreCase) {
189			$str->string = implode($to, preg_split('{'.preg_quote($from).'}iuD', $this->string));
190			} else {
191			$str->string = str_replace($from, $to, $this->string);
192			}
193
194			return $str;
195			}
196
197			public function slice(int $start = 0, int $length = null): AbstractString
198			{
199			$str = clone $this;
200			$str->string = mb_substr($this->string, $start, $length, 'UTF-8');
201
202			return $str;
203			}
204
205			public function splice(string $replacement, int $start = 0, int $length = null): AbstractString
206			{
207			if (!preg_match('//u', $replacement)) {
208			throw new InvalidArgumentException('Invalid UTF-8 string.');
209			}
210
211			$str = clone $this;
212			$start = $start ? \strlen(mb_substr($this->string, 0, $start, 'UTF-8')) : 0;
213			$length = $length ? \strlen(mb_substr($this->string, $start, $length, 'UTF-8')) : $length;
214			$str->string = substr_replace($this->string, $replacement, $start, $length ?? \PHP_INT_MAX);
215
216			return $str;
217			}
218
219			public function split(string $delimiter, int $limit = null, int $flags = null): array
220			{
221			if (1 > $limit = $limit ?? \PHP_INT_MAX) {
222			throw new InvalidArgumentException('Split limit must be a positive integer.');
223			}
224
225			if ('' === $delimiter) {
226			throw new InvalidArgumentException('Split delimiter is empty.');
227			}
228
229			if (null !== $flags) {
230			return parent::split($delimiter.'u', $limit, $flags);
			0 ignored issues – show Bug Best Practice introduced 2024-10-12 21:52 UTC by Report Bug Copy Issue Report The expression `return parent::split($de... . 'u', $limit, $flags)` returns the type `array<mixed,array\|string>` which is incompatible with the return type mandated by `Symfony\Component\String\AbstractString::split()` of `array<mixed,Symfony\Comp...\String\AbstractString>`. In the issue above, the returned value is violating the contract defined by the mentioned interface. Let's take a look at an example: interface HasName { /** @return string / public function getName(); } class Name { public $name; } class User implements HasName { /* @return string\|Name */ public function getName() { return new Name('foo'); // This is a violation of the ``HasName`` interface // which only allows a string value to be returned. } } Loading history...
231			}
232
233			if (!preg_match('//u', $delimiter)) {
234			throw new InvalidArgumentException('Split delimiter is not a valid UTF-8 string.');
235			}
236
237			$str = clone $this;
238			$chunks = $this->ignoreCase
239			? preg_split('{'.preg_quote($delimiter).'}iuD', $this->string, $limit)
240			: explode($delimiter, $this->string, $limit);
241
242			foreach ($chunks as &$chunk) {
243			$str->string = $chunk;
244			$chunk = clone $str;
245			}
246
247			return $chunks;
			0 ignored issues – show Bug Best Practice introduced 2024-10-12 21:52 UTC by Report Bug Copy Issue Report The expression `return $chunks` returns the type `string[]` which is incompatible with the return type mandated by `Symfony\Component\String\AbstractString::split()` of `array<mixed,Symfony\Comp...\String\AbstractString>`. In the issue above, the returned value is violating the contract defined by the mentioned interface. Let's take a look at an example: interface HasName { /** @return string / public function getName(); } class Name { public $name; } class User implements HasName { /* @return string\|Name */ public function getName() { return new Name('foo'); // This is a violation of the ``HasName`` interface // which only allows a string value to be returned. } } Loading history...
248			}
249
250			public function startsWith($prefix): bool
251			{
252			if ($prefix instanceof AbstractString) {
253			$prefix = $prefix->string;
254			} elseif (\is_array($prefix) \|\| $prefix instanceof \Traversable) {
255			return parent::startsWith($prefix);
256			} else {
257			$prefix = (string) $prefix;
258			}
259
260			if ('' === $prefix \|\| !preg_match('//u', $prefix)) {
261			return false;
262			}
263
264			if ($this->ignoreCase) {
265			return 0 === mb_stripos($this->string, $prefix, 0, 'UTF-8');
266			}
267
268			return 0 === strncmp($this->string, $prefix, \strlen($prefix));
269			}
270			}
271

deployphp / deployer

GitHub Access Token became invalid

Push — master ( 5cefd1...492078 )

CodePointString F

Complexity

Size/Duplication

Importance

15 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like