StringUtil::parseValue() - Code Metrics - puli/cli - Measure and Improve Code Quality continuously with Scrutinizer

StringUtil::parseValue() B
last analyzed 2016-08-15 16:33 UTC

↳ Parent: StringUtil

Complexity

Conditions	7
Paths	7

Size

Total Lines	21
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	7.049

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	21
ccs	9
cts	10
cp	0.9
rs	7.551
cc	7
eloc	11
nc	7
nop	1
crap	7.049

<?php

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

namespace Puli\Cli\Util;

/**
 * @since  1.0
 *
 * @author Bernhard Schussek <[email protected]>
 */
class StringUtil

{
    /**
     * Returns the short class name for a fully-qualified class name.
     *
     * @param string $className The fully-qualified class name.
     *
     * @return string The short class name.
     */
    public static function getShortClassName($className)
    {
        if (false !== ($pos = strrpos($className, '\\'))) {
            return substr($className, $pos + 1);
        }

        return $className;
    }

    public static function parseValue($string)

    {
        switch ($string) {
            case '': return null;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'null': return null;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'true': return true;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'false': return false;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
        }

        if ($string === (string) ($int = (int) $string)) {
            return $int;
        }

        // Check for " or ' delimiters
        // https://3v4l.org/5u0AU
        if (preg_match('/^(["\']).*\1$/m', $string)) {
            return substr($string, 1, -1);
        }

        return $string;
    }

    public static function formatValue($value, $quote = true)
    {
        if (null === $value) {
            return 'null';
        }

        if (is_bool($value)) {
            return $value ? 'true' : 'false';
        }

        if (is_string($value)) {
            $q = $quote ? '"' : '';

            return $q.$value.$q;
        }

        return (string) $value;
    }

    private function __construct()
    {
    }
}


1		<?php
2
3		/*
4		* This file is part of the puli/cli package.
5		*
6		* (c) Bernhard Schussek <[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 Puli\Cli\Util;
13
14		/**
15		* @since 1.0
16		*
17		* @author Bernhard Schussek <[email protected]>
18		*/
19		class StringUtil
		0 ignored issues – show Coding Style introduced 2015-12-06 22:45 UTC by Report Bug Copy Issue Report Since you have declared the constructor as private, maybe you should also declare the class as final. Loading history...
20		{
21		/**
22		* Returns the short class name for a fully-qualified class name.
23		*
24		* @param string $className The fully-qualified class name.
25		*
26		* @return string The short class name.
27		*/
28	56	public static function getShortClassName($className)
29		{
30	56	if (false !== ($pos = strrpos($className, '\\'))) {
31	51	return substr($className, $pos + 1);
32		}
33
34	5	return $className;
35		}
36
37	15	public static function parseValue($string)
		0 ignored issues – show Documentation introduced 2015-12-06 22:45 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
38		{
39		switch ($string) {
40	15	case '': return null;
		0 ignored issues – show Coding Style introduced 2016-06-29 08:55 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history... Coding Style introduced 2016-06-29 08:55 UTC by Report Bug Copy Issue Report Terminating statement must be on a line by itself As per the PSR-2 coding standard, the `break` (or other terminating) statement must be on a line of its own. switch ($expr) { case "A": doSomething(); break; //wrong case "B": doSomething(); break; //right case "C:": doSomething(); return true; //right } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
41	15	case 'null': return null;
		0 ignored issues – show Coding Style introduced 2016-06-29 08:55 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history... Coding Style introduced 2016-06-29 08:55 UTC by Report Bug Copy Issue Report Terminating statement must be on a line by itself As per the PSR-2 coding standard, the `break` (or other terminating) statement must be on a line of its own. switch ($expr) { case "A": doSomething(); break; //wrong case "B": doSomething(); break; //right case "C:": doSomething(); return true; //right } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
42	14	case 'true': return true;
		0 ignored issues – show Coding Style introduced 2016-06-29 08:55 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history... Coding Style introduced 2016-06-29 08:55 UTC by Report Bug Copy Issue Report Terminating statement must be on a line by itself As per the PSR-2 coding standard, the `break` (or other terminating) statement must be on a line of its own. switch ($expr) { case "A": doSomething(); break; //wrong case "B": doSomething(); break; //right case "C:": doSomething(); return true; //right } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
43	12	case 'false': return false;
		0 ignored issues – show Coding Style introduced 2016-06-29 08:55 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history... Coding Style introduced 2016-06-29 08:55 UTC by Report Bug Copy Issue Report Terminating statement must be on a line by itself As per the PSR-2 coding standard, the `break` (or other terminating) statement must be on a line of its own. switch ($expr) { case "A": doSomething(); break; //wrong case "B": doSomething(); break; //right case "C:": doSomething(); return true; //right } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
44		}
45
46	11	if ($string === (string) ($int = (int) $string)) {
47	2	return $int;
48		}
49
50		// Check for " or ' delimiters
51		// https://3v4l.org/5u0AU
52	9	if (preg_match('/^(["\']).*\1$/m', $string)) {
53		return substr($string, 1, -1);
54		}
55
56	9	return $string;
57		}
58
59	21	public static function formatValue($value, $quote = true)
60		{
61	21	if (null === $value) {
62	2	return 'null';
63		}
64
65	20	if (is_bool($value)) {
66	7	return $value ? 'true' : 'false';
67		}
68
69	15	if (is_string($value)) {
70	11	$q = $quote ? '"' : '';
71
72	11	return $q.$value.$q;
73		}
74
75	4	return (string) $value;
76		}
77
78		private function __construct()
79		{
80		}
81		}
82

puli / cli

StringUtil::parseValue() B last analyzed 2016-08-15 16:33 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

StringUtil::parseValue() B
last analyzed 2016-08-15 16:33 UTC