cli.php ➔ colorForShell() - Code Metrics - Inspection of "Fix load module at "modulesLoadTime_Module" time" - bfw-systems/bfw - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#68)

by Vermeulen

created 2016-06-25 13:08 UTC

cli.php ➔ colorForShell() C

↳ Parent: Project

Complexity

Conditions	19
Paths	19

Size

Total Lines	76
Code Lines	36

Duplication

Lines	67
Ratio	88.16 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	19
eloc	36
c	1
b	0
f	1
nc	19
nop	2
dl	67
loc	76
rs	5.222

How to fix Long Method Complexity

<?php
/**
 * Toutes les fonctions de base utilisé par le système cli
 * @author Vermeulen Maxime <[email protected]>
 * @package bfw
 */

/**
 * Permet de récupérer les arguments de la console
 * Permet surtout de toujours avoir les arguments obligatoire par le système.
 * 
 * @link http://php.net/manual/fr/function.getopt.php
 * 
 * @param string $options  : Chaque caractère dans cette chaîne sera utilisé en tant que caractères optionnels et 
 *                           devra correspondre aux options passées, commençant par un tiret simple (-). 
 *                           Par exemple, une chaîne optionnelle "x" correspondra à l'option -x. 
 *                           Seuls a-z, A-Z et 0-9 sont autorisés.
 * @param array  $longopts : Un tableau d'options. Chaque élément de ce tableau sera utilisé comme option et devra 
 *                           correspondre aux options passées, commençant par un tiret double (--). 
 *                           Par exemple, un élément longopts "opt" correspondra à l'option --opt.
 *                           Le paramètre options peut contenir les éléments suivants :
 *                              * Caractères individuels (n'accepte pas de valeur)
 *                              * Caractères suivis par un deux-points (le paramètre nécessite une valeur)
 *                              * Caractères suivis par deux deux-points (valeur optionnelle)
 *                           Les valeurs optionnelles sont les premiers arguments après la chaîne. 
 *                           Si une valeur est requise, peu importe que la valeur soit suivi d'un espace ou non.
 * 
 * @return array
 */
function getCliParams($options, $longopts = array())
{
    $longopts = array_merge($longopts, array('type_site::'));
    $opt      = getopt('f:'.$options, $longopts);
    unset($opt['f']);

    return $opt;
}

/**
 * Permet de facilement afficher un message dans la console
 * 
 * @param string $msg : Le message à afficher
 */
function displayMsg($msg, $colorTxt = null, $colorBg = null, $style = 'normal')
{
    if($colorTxt == null)
    {
        echo $msg."\n";
        return;
    }

    //Gestion cas avec couleur

    $colorStart = '';
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
    $colorEnd   = '';
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

    $styleNum = styleForShell($style);
    if($styleNum === false)
    {
        $styleNum = styleForShell('normal');
    }

    $colorTxtNum = colorForShell('white', 'txt');
    if($colorTxt !== null)
    {
        $colorTxtNumArg = colorForShell($colorTxt, 'txt');
        if($colorTxtNumArg !== false)
        {
            $colorTxtNum = $colorTxtNumArg;
        }
    }

    $colorBgNum = colorForShell('black', 'bg');
    if($colorBg !== null)
    {
        $colorBgNumArg = colorForShell($colorBg, 'bg');
        if($colorBgNumArg !== false)
        {
            $colorBgNum = $colorBgNumArg;
        }
    }

    echo "\033[".$styleNum.";".$colorBgNum.";".$colorTxtNum."m".$msg."\033[0m\n";
}

/**
 * Converti le texte d'une couleur vers son code shell
 * 
 * @param string $color : Le nom de la couleur en anglais
 * @param string $type  : (txt|bg) Si c'est pour le texte (txt) ou pour la couleur de fond (bg)
 * 
 * @return string
 */
function colorForShell($color, $type)
{
    //Possibilité d'améliorer la compléxité du script via des boucles etc...
    if($type == 'txt')
    {
        if($color == 'black')

        {
            return 30;
        }
        elseif($color == 'red')
        {
            return 31;
        }
        elseif($color == 'green')
        {
            return 32;
        }
        elseif($color == 'yellow')
        {
            return 33;
        }
        elseif($color == 'blue')
        {
            return 34;
        }
        elseif($color == 'magenta')
        {
            return 35;
        }
        elseif($color == 'cyan')
        {
            return 36;
        }
        elseif($color == 'white')
        {
            return 37;
        }
    }
    elseif($type == 'bg')

    {
        if($color == 'black')
        {
            return 40;
        }
        elseif($color == 'red')
        {
            return 41;
        }
        elseif($color == 'green')
        {
            return 42;
        }
        elseif($color == 'yellow')
        {
            return 43;
        }
        elseif($color == 'blue')
        {
            return 44;
        }
        elseif($color == 'magenta')
        {
            return 45;
        }
        elseif($color == 'cyan')
        {
            return 46;
        }
        elseif($color == 'white')
        {
            return 47;
        }
    }

    return false;
}

/**
 * Permet de convertir un style définie en anglais pour le shell
 */
function styleForShell($style)
{
    if($style == 'normal')
    {
        return 0;
    }
    elseif($style == 'bold')
    {
        return 1;
    }
    elseif($style == 'not-bold')
    {
        return 21;
    }
    elseif($style == 'underline')
    {
        return 4;
    }
    elseif($style == 'not-underline')
    {
        return 24;
    }
    elseif($style == 'blink')
    {
        return 5;
    }
    elseif($style == 'not-blink')
    {
        return 25;
    }
    elseif($style == 'reverse')
    {
        return 7;
    }
    elseif($style == 'not-reverse')
    {
        return 27;
    }

    return false;
}


1		<?php
2		/**
3		* Toutes les fonctions de base utilisé par le système cli
4		* @author Vermeulen Maxime <[email protected]>
5		* @package bfw
6		*/
7
8		/**
9		* Permet de récupérer les arguments de la console
10		* Permet surtout de toujours avoir les arguments obligatoire par le système.
11		*
12		* @link http://php.net/manual/fr/function.getopt.php
13		*
14		* @param string $options : Chaque caractère dans cette chaîne sera utilisé en tant que caractères optionnels et
15		* devra correspondre aux options passées, commençant par un tiret simple (-).
16		* Par exemple, une chaîne optionnelle "x" correspondra à l'option -x.
17		* Seuls a-z, A-Z et 0-9 sont autorisés.
18		* @param array $longopts : Un tableau d'options. Chaque élément de ce tableau sera utilisé comme option et devra
19		* correspondre aux options passées, commençant par un tiret double (--).
20		* Par exemple, un élément longopts "opt" correspondra à l'option --opt.
21		* Le paramètre options peut contenir les éléments suivants :
22		* * Caractères individuels (n'accepte pas de valeur)
23		* * Caractères suivis par un deux-points (le paramètre nécessite une valeur)
24		* * Caractères suivis par deux deux-points (valeur optionnelle)
25		* Les valeurs optionnelles sont les premiers arguments après la chaîne.
26		* Si une valeur est requise, peu importe que la valeur soit suivi d'un espace ou non.
27		*
28		* @return array
29		*/
30		function getCliParams($options, $longopts = array())
31		{
32		$longopts = array_merge($longopts, array('type_site::'));
33		$opt = getopt('f:'.$options, $longopts);
34		unset($opt['f']);
35
36		return $opt;
37		}
38
39		/**
40		* Permet de facilement afficher un message dans la console
41		*
42		* @param string $msg : Le message à afficher
43		*/
44		function displayMsg($msg, $colorTxt = null, $colorBg = null, $style = 'normal')
45		{
46		if($colorTxt == null)
47		{
48		echo $msg."\n";
49		return;
50		}
51
52		//Gestion cas avec couleur
53
54		$colorStart = '';
		0 ignored issues – show Unused Code introduced 2016-01-16 16:22 UTC by Report Bug Copy Issue Report `$colorStart` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
55		$colorEnd = '';
		0 ignored issues – show Unused Code introduced 2016-01-16 16:22 UTC by Report Bug Copy Issue Report `$colorEnd` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
56
57		$styleNum = styleForShell($style);
58		if($styleNum === false)
59		{
60		$styleNum = styleForShell('normal');
61		}
62
63		$colorTxtNum = colorForShell('white', 'txt');
64		if($colorTxt !== null)
65		{
66		$colorTxtNumArg = colorForShell($colorTxt, 'txt');
67		if($colorTxtNumArg !== false)
68		{
69		$colorTxtNum = $colorTxtNumArg;
70		}
71		}
72
73		$colorBgNum = colorForShell('black', 'bg');
74		if($colorBg !== null)
75		{
76		$colorBgNumArg = colorForShell($colorBg, 'bg');
77		if($colorBgNumArg !== false)
78		{
79		$colorBgNum = $colorBgNumArg;
80		}
81		}
82
83		echo "\033[".$styleNum.";".$colorBgNum.";".$colorTxtNum."m".$msg."\033[0m\n";
84		}
85
86		/**
87		* Converti le texte d'une couleur vers son code shell
88		*
89		* @param string $color : Le nom de la couleur en anglais
90		* @param string $type : (txt\|bg) Si c'est pour le texte (txt) ou pour la couleur de fond (bg)
91		*
92		* @return string
93		*/
94		function colorForShell($color, $type)
95		{
96		//Possibilité d'améliorer la compléxité du script via des boucles etc...
97		if($type == 'txt')
98		{
99	View Code Duplication	if($color == 'black')
		0 ignored issues – show Duplication introduced 2016-01-16 16:22 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...
100		{
101		return 30;
102		}
103		elseif($color == 'red')
104		{
105		return 31;
106		}
107		elseif($color == 'green')
108		{
109		return 32;
110		}
111		elseif($color == 'yellow')
112		{
113		return 33;
114		}
115		elseif($color == 'blue')
116		{
117		return 34;
118		}
119		elseif($color == 'magenta')
120		{
121		return 35;
122		}
123		elseif($color == 'cyan')
124		{
125		return 36;
126		}
127		elseif($color == 'white')
128		{
129		return 37;
130		}
131		}
132	View Code Duplication	elseif($type == 'bg')
		0 ignored issues – show Duplication introduced 2016-01-16 16:22 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...
133		{
134		if($color == 'black')
135		{
136		return 40;
137		}
138		elseif($color == 'red')
139		{
140		return 41;
141		}
142		elseif($color == 'green')
143		{
144		return 42;
145		}
146		elseif($color == 'yellow')
147		{
148		return 43;
149		}
150		elseif($color == 'blue')
151		{
152		return 44;
153		}
154		elseif($color == 'magenta')
155		{
156		return 45;
157		}
158		elseif($color == 'cyan')
159		{
160		return 46;
161		}
162		elseif($color == 'white')
163		{
164		return 47;
165		}
166		}
167
168		return false;
169		}
170
171		/**
172		* Permet de convertir un style définie en anglais pour le shell
173		*/
174		function styleForShell($style)
175		{
176		if($style == 'normal')
177		{
178		return 0;
179		}
180		elseif($style == 'bold')
181		{
182		return 1;
183		}
184		elseif($style == 'not-bold')
185		{
186		return 21;
187		}
188		elseif($style == 'underline')
189		{
190		return 4;
191		}
192		elseif($style == 'not-underline')
193		{
194		return 24;
195		}
196		elseif($style == 'blink')
197		{
198		return 5;
199		}
200		elseif($style == 'not-blink')
201		{
202		return 25;
203		}
204		elseif($style == 'reverse')
205		{
206		return 7;
207		}
208		elseif($style == 'not-reverse')
209		{
210		return 27;
211		}
212
213		return false;
214		}
215

bfw-systems / bfw

GitHub Access Token became invalid

Pull Request — master (#68)

cli.php ➔ colorForShell() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like