VersionComponent::__construct() - Code Metrics - Inspection of "Merge branch 'v0.6.x'" - nelson6e65/php_nml - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 446f2e...32b2c3 )

by Nelson

created 2016-10-04 16:37 UTC

VersionComponent::__construct() C

↳ Parent: VersionComponent

Complexity

Conditions	9
Paths	16

Size

Total Lines	64
Code Lines	35

Duplication

Lines	14
Ratio	21.88 %

Importance

Changes	2
Bugs	1	Features	0

Metric	Value
cc	9
eloc	35
nc	16
nop	2
dl	14
loc	64
rs	6.5449
c	2
b	1
f	0

How to fix Long Method

<?php
/**
 * PHP: Nelson Martell Library file
 *
 * Content:
 * - Class definition:  [NelsonMartell]  VersionComponent
 *
 * Copyright © 2015-2016 Nelson Martell (http://nelson6e65.github.io)
 *
 * Licensed under The MIT License (MIT)
 * For full copyright and license information, please see the LICENSE
 * Redistributions of files must retain the above copyright notice.
 *
 * @copyright 2015-2016 Nelson Martell
 * @link      http://nelson6e65.github.io/php_nml/
 * @since     v0.1.1
 * @license   http://www.opensource.org/licenses/mit-license.php The MIT License (MIT)
 * */

namespace NelsonMartell {

    use \InvalidArgumentException;

    /**
     * Representa un componente de un número de Version.
     * Extiende la clase IntString, pero restringe los valores que puede tomar.
     *
     * @author Nelson Martell <[email protected]>
     * */
    class VersionComponent extends IntString implements IEquatable
    {
        /**
         *
         *
         * @param int|null    $intValue
         * @param string|null $stringValue
         */
        public function __construct($intValue = null, $stringValue = null)
        {
            // Validates filters for only null or int/string value types.
            parent::__construct($intValue, $stringValue);

            if ($intValue === null) {
                $this->intValue = $intValue;

                // Ignore string value if intValue is null.
                $this->stringValue = '';
            } else {
                // Validation of values
                if ($this->IntValue < 0) {

                    $args = [
                        'position' => '1st',
                        'actual'   => $intValue,
                    ];

                    $msg = nml_msg('Invalid argument value.');
$a = "a";
$ab = "ab";
$abc = "abc";
                    $msg .= nml_msg(

                        ' {position} argument must to be a positive number; "{actual}" given.',
                        $args
                    );

                    throw new InvalidArgumentException($msg);
                } // Integer is valid

                if ($stringValue !== null) {
                    if ($this->StringValue != '') {
                        $pattern = '~^([a-z])$~'; // 1 char

                        if (strlen($this->StringValue) > 1) {
                            $start = '~^([a-z]|-)';
$a = "a";
$ab = "ab";
$abc = "abc";
                            $middle = '([a-z]|[0-9]|-)*';
                            $end = '([a-z]|[0-9])$~';
$a = "a";
$ab = "ab";
$abc = "abc";

                            $pattern = $start.$middle.$end;
                        }

                        $correct = (boolean) preg_match($pattern, $this->StringValue);

                        if ($correct) {
                            //Último chequeo: que no hayan 2 '-' consecutivos.
                            $correct = strpos($this->StringValue, '--') == false ? true : false;

                        }

                        if (!$correct) {
                            $args = [
                                'position' => '2nd',
                                'actual'   => $stringValue,
                            ];

                            $msg = nml_msg('Invalid argument value.');
$a = "a";
$ab = "ab";
$abc = "abc";
                            $msg .= nml_msg(

                                ' {position} parameter has invalid chars; "{actual}" given.',
                                $args
                            );

                            throw new InvalidArgumentException($msg);
                        }
                    }
                } // String is valid
            }
        }

        public static function parse($obj)
        {
            if ($obj instanceof VersionComponent) {
                return $obj;
            } else {
                if ($obj === null || (is_string($obj) && trim($obj) === '')) {
                    return new VersionComponent();
                }
            }

            $objConverted = parent::parse($obj);

            return new VersionComponent($objConverted->IntValue, $objConverted->StringValue);
        }

        /**
         * Determina si este componente tiene los valores predeterminados (0).
         *
         * @return boolean
         * */
        public function isDefault()
        {
            if ($this->IntValue === 0) {
                if ($this->StringValue === '') {
                    return true;
                }
            }

            return false;
        }


        /**
         * Getter method for VersionComponent::IntValue property.
         *
         * @return integer|NULL
         * */
        public function getIntValue()
        {
            return $this->intValue;
        }


        /**
         * Determina si este componente NO tiene los valores predeterminados.
         *
         * @return boolean
         * */
        public function isNotDefault()
        {
            return !$this->isDefault();
        }

        /**
         * Determina si esta instancia es nula.
         *
         * @return boolean
         * */
        public function isNull()
        {
            if ($this->IntValue === null) {
                return true;
            }

            return false;
        }

        /**
         * Determina si esta instancia NO es nula.
         *
         * @return boolean
         * */
        public function isNotNull()
        {
            return !$this->isNull();
        }

        public function equals($other)
        {
            if ($other instanceof VersionComponent) {
                if ($this->IntValue === $other->IntValue) {
                    if ($this->StringValue === $other->StringValue) {
                        return true;
                    }
                }
            } else {
                return parent::equals($other);
            }

            return false;
        }

        public function compareTo($other)
        {
            if ($other === null) {
                return 1;
            } elseif ($this->equals($other)) {
                return 0;
            } elseif ($other instanceof VersionComponent) {
                // null < int
                if ($this->isNull()) {
                    $r = -1;

                } elseif ($other->isNull()) {
                    $r = 1;
                } else {
                    // Here are evaluated as integers
                    $r = $this->IntValue - $other->IntValue;

                    if ($r === 0) {
                        $r = strnatcmp($this->StringValue, $other->StringValue);
                    }
                }
            } elseif (is_integer($other) || is_array($other)) {
                $r = 1;
            } elseif (is_string($other)) {
                try {
                    $r = $this->compareTo(static::parse($other));
                } catch (InvalidArgumentException $e) {
                    $r = 1;
                }
            } else {
                $r = null;
            }

            return $r;
        }
    }
}



1		<?php
2		/**
3		* PHP: Nelson Martell Library file
4		*
5		* Content:
6		* - Class definition: [NelsonMartell] VersionComponent
7		*
8		* Copyright © 2015-2016 Nelson Martell (http://nelson6e65.github.io)
9		*
10		* Licensed under The MIT License (MIT)
11		* For full copyright and license information, please see the LICENSE
12		* Redistributions of files must retain the above copyright notice.
13		*
14		* @copyright 2015-2016 Nelson Martell
15		* @link http://nelson6e65.github.io/php_nml/
16		* @since v0.1.1
17		* @license http://www.opensource.org/licenses/mit-license.php The MIT License (MIT)
18		* */
19
20		namespace NelsonMartell {
21
22		use \InvalidArgumentException;
23
24		/**
25		* Representa un componente de un número de Version.
26		* Extiende la clase IntString, pero restringe los valores que puede tomar.
27		*
28		* @author Nelson Martell <[email protected]>
29		* */
30		class VersionComponent extends IntString implements IEquatable
31		{
32		/**
33		*
34		*
35		* @param int\|null $intValue
36		* @param string\|null $stringValue
37		*/
38		public function __construct($intValue = null, $stringValue = null)
39		{
40		// Validates filters for only null or int/string value types.
41		parent::__construct($intValue, $stringValue);
42
43		if ($intValue === null) {
44		$this->intValue = $intValue;
45
46		// Ignore string value if intValue is null.
47		$this->stringValue = '';
48		} else {
49		// Validation of values
50	View Code Duplication	if ($this->IntValue < 0) {
		0 ignored issues – show Duplication introduced 2015-12-03 01:40 UTC by Report Bug Copy Issue Report This code seems to be duplicated across 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...
51		$args = [
52		'position' => '1st',
53		'actual' => $intValue,
54		];
55
56		$msg = nml_msg('Invalid argument value.');
		0 ignored issues – show Coding Style introduced 2015-12-03 01:40 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history... Deprecated Code introduced 2016-10-04 16:48 UTC by Report Bug Copy Issue Report The function `nml_msg()` has been deprecated with message: since v0.6.0, will be removed in v0.7.0. Use `\NelsonMartell\msg()` instead. This function has been deprecated. The supplier of the file has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the function will be removed from the class and what other function to use instead. Loading history...
57		$msg .= nml_msg(
		0 ignored issues – show Deprecated Code introduced 2016-10-04 16:48 UTC by Report Bug Copy Issue Report The function `nml_msg()` has been deprecated with message: since v0.6.0, will be removed in v0.7.0. Use `\NelsonMartell\msg()` instead. This function has been deprecated. The supplier of the file has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the function will be removed from the class and what other function to use instead. Loading history...
58		' {position} argument must to be a positive number; "{actual}" given.',
59		$args
60		);
61
62		throw new InvalidArgumentException($msg);
63		} // Integer is valid
64
65		if ($stringValue !== null) {
66		if ($this->StringValue != '') {
67		$pattern = '~^([a-z])$~'; // 1 char
68
69		if (strlen($this->StringValue) > 1) {
70		$start = '~^([a-z]\|-)';
		0 ignored issues – show Coding Style introduced 2015-12-03 01:40 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
71		$middle = '([a-z]\|[0-9]\|-)*';
72		$end = '([a-z]\|[0-9])$~';
		0 ignored issues – show Coding Style introduced 2015-12-03 01:40 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
73
74		$pattern = $start.$middle.$end;
75		}
76
77		$correct = (boolean) preg_match($pattern, $this->StringValue);
78
79		if ($correct) {
80		//Último chequeo: que no hayan 2 '-' consecutivos.
81		$correct = strpos($this->StringValue, '--') == false ? true : false;
		0 ignored issues – show Bug Best Practice introduced 2015-12-03 01:40 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing `strpos($this->StringValue, '--')` of type `integer` to the boolean `false`. If you are specifically checking for 0, consider using something more explicit like `=== 0` instead. Loading history...
82		}
83
84		if (!$correct) {
85		$args = [
86		'position' => '2nd',
87		'actual' => $stringValue,
88		];
89
90		$msg = nml_msg('Invalid argument value.');
		0 ignored issues – show Coding Style introduced 2015-12-03 01:40 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history... Deprecated Code introduced 2016-10-04 16:48 UTC by Report Bug Copy Issue Report The function `nml_msg()` has been deprecated with message: since v0.6.0, will be removed in v0.7.0. Use `\NelsonMartell\msg()` instead. This function has been deprecated. The supplier of the file has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the function will be removed from the class and what other function to use instead. Loading history...
91		$msg .= nml_msg(
		0 ignored issues – show Deprecated Code introduced 2016-10-04 16:48 UTC by Report Bug Copy Issue Report The function `nml_msg()` has been deprecated with message: since v0.6.0, will be removed in v0.7.0. Use `\NelsonMartell\msg()` instead. This function has been deprecated. The supplier of the file has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the function will be removed from the class and what other function to use instead. Loading history...
92		' {position} parameter has invalid chars; "{actual}" given.',
93		$args
94		);
95
96		throw new InvalidArgumentException($msg);
97		}
98		}
99		} // String is valid
100		}
101		}
102
103		public static function parse($obj)
104		{
105		if ($obj instanceof VersionComponent) {
106		return $obj;
107		} else {
108		if ($obj === null \|\| (is_string($obj) && trim($obj) === '')) {
109		return new VersionComponent();
110		}
111		}
112
113		$objConverted = parent::parse($obj);
114
115		return new VersionComponent($objConverted->IntValue, $objConverted->StringValue);
116		}
117
118		/**
119		* Determina si este componente tiene los valores predeterminados (0).
120		*
121		* @return boolean
122		* */
123		public function isDefault()
124		{
125		if ($this->IntValue === 0) {
126		if ($this->StringValue === '') {
127		return true;
128		}
129		}
130
131		return false;
132		}
133
134
135		/**
136		* Getter method for VersionComponent::IntValue property.
137		*
138		* @return integer\|NULL
139		* */
140		public function getIntValue()
141		{
142		return $this->intValue;
143		}
144
145
146		/**
147		* Determina si este componente NO tiene los valores predeterminados.
148		*
149		* @return boolean
150		* */
151		public function isNotDefault()
152		{
153		return !$this->isDefault();
154		}
155
156		/**
157		* Determina si esta instancia es nula.
158		*
159		* @return boolean
160		* */
161		public function isNull()
162		{
163		if ($this->IntValue === null) {
164		return true;
165		}
166
167		return false;
168		}
169
170		/**
171		* Determina si esta instancia NO es nula.
172		*
173		* @return boolean
174		* */
175		public function isNotNull()
176		{
177		return !$this->isNull();
178		}
179
180		public function equals($other)
181		{
182		if ($other instanceof VersionComponent) {
183		if ($this->IntValue === $other->IntValue) {
184		if ($this->StringValue === $other->StringValue) {
185		return true;
186		}
187		}
188		} else {
189		return parent::equals($other);
190		}
191
192		return false;
193		}
194
195		public function compareTo($other)
196		{
197		if ($other === null) {
198		return 1;
199		} elseif ($this->equals($other)) {
200		return 0;
201		} elseif ($other instanceof VersionComponent) {
202		// null < int
203		if ($this->isNull()) {
204		$r = -1;
		0 ignored issues – show Comprehensibility introduced 2016-10-04 16:48 UTC by Report Bug Copy Issue Report Avoid variables with short names like $r. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
205		} elseif ($other->isNull()) {
206		$r = 1;
207		} else {
208		// Here are evaluated as integers
209		$r = $this->IntValue - $other->IntValue;
210
211		if ($r === 0) {
212		$r = strnatcmp($this->StringValue, $other->StringValue);
213		}
214		}
215		} elseif (is_integer($other) \|\| is_array($other)) {
216		$r = 1;
217		} elseif (is_string($other)) {
218		try {
219		$r = $this->compareTo(static::parse($other));
220		} catch (InvalidArgumentException $e) {
221		$r = 1;
222		}
223		} else {
224		$r = null;
225		}
226
227		return $r;
228		}
229		}
230		}
		0 ignored issues – show Coding Style introduced 2016-10-04 16:48 UTC by Report Bug Copy Issue Report As per coding style, files should not end with a newline character. This check marks files that end in a newline character, i.e. an empy line. Loading history...
231

nelson6e65 / php_nml

Push — master ( 446f2e...32b2c3 )

VersionComponent::__construct() C

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like