FiUtilita - Code Metrics - Inspection of "improve grid filters tests" - ComuneFI/FifreeCoreBundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ce89ba...4bc45e )

by Andrea

created 2018-02-21 21:53 UTC

FiUtilita B

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	225
Duplicated Lines	0 %

Test Coverage

Coverage

34.43%

Importance

Changes

Metric	Value
wmc	40
dl	0
loc	225
ccs	42
cts	122
cp	0.3443
rs	8.2608
c	0
b	0
f	0

11 Methods

Rating	Name	Size	Complexity
C	operatoreQuery()	32	8
A	percentualiConfrontoStringheVettore()	5	1
A	getProSelectAttribute()	8	4
B	cancellaDaVettore()	32	6
A	percentualiConfrontoStringhe()	5	1
A	getDataFormattata()	7	3
B	proSelect()	22	4
A	db2data()	20	4
A	senzalinea()	9	2
A	sommaMinuti()	15	2
B	data2db()	25	5

How to fix Complexity

<?php

namespace Fi\CoreBundle\Controller;

/*
 * Insieme di funzioni utili
 * FiUtilita.
 *
 * @author Emidio Picariello
 */

use Fi\CoreBundle\DependencyInjection\PercentualiStringhe;

class FiUtilita
{

    public function percentualiConfrontoStringheVettore($parametri = array())
    {
        $percentuali = new PercentualiStringhe();

        return $percentuali->percentualiConfrontoStringheVettore($parametri);
    }

    public function percentualiConfrontoStringhe($parametri = array())
    {
        $percentuali = new PercentualiStringhe();

        return $percentuali->percentualiConfrontoStringhe($parametri);
    }

    public function sommaMinuti($parametri = array())
    {
        //parametri obbligatori
        if (!isset($parametri['minuti'])) {
            return false;
        }
        $minuti = $parametri['minuti'];

        $totminuti = array_sum($minuti);
        $resminuti = $totminuti % 60;
        $resore = ($totminuti - $resminuti) / 60;
$a = "a";
$ab = "ab";
$abc = "abc";

        $restotminuti = array('ore' => $resore, 'minuti' => $resminuti);

        return $restotminuti;
    }

    /**
     * @param array  $parametri
     * string $parametri["tipo"]
     *
     * @return Array("segnouno"=>"xx", "segnodue"=>"yy") dove segnodue non obbligatorio
     */
    public function operatoreQuery($parametri = array())
    {
        $risposta = array();

        if (isset($parametri['tipo'])) {
            $tipocampo = $parametri['tipo'];
        } else {
            return array('segnouno' => '=');
        }

        switch ($tipocampo) {
            case 'date':
            case 'integer':
            case 'double':
                $operatore = '>=';
$a = "a";
$ab = "ab";
$abc = "abc";
                $operatoredue = '<=';
                break;
            case 'string':
            case 'text':
                $operatore = 'LIKE';
                break;
            default:
                $operatore = '=';
                break;
        }

        $risposta['segnouno'] = $operatore;
        if (isset($operatoredue)) {
            $risposta['segnodue'] = $operatoredue;
        }

        return $risposta;
    }

    public static function data2db($giorno, $invertito = false, $senzalinea = false)
    {
        if ($giorno == '') {
            return null;
        }

        if (substr($giorno, 4, 1) == '-') {
            return $giorno;
        }

        $barragiorno = strpos($giorno, '/');
        $gg = substr($giorno, 0, $barragiorno);
$a = "a";
$ab = "ab";
$abc = "abc";
        $restante = substr($giorno, $barragiorno + 1);
$a = "a";
$ab = "ab";
$abc = "abc";

        $barra = strpos($restante, '/');
        $mm = substr($restante, 0, $barra);
$a = "a";
$ab = "ab";
$abc = "abc";
        $aaaa = substr($restante, $barra + 1);
$a = "a";
$ab = "ab";
$abc = "abc";

        $appogg = ($invertito ? $mm : $gg);
        $mm = ($invertito ? $gg : $mm);
$a = "a";
$ab = "ab";
$abc = "abc";
        $gg = $appogg;
$a = "a";
$ab = "ab";
$abc = "abc";

        $formattata = self::getDataFormattata($aaaa, $mm, $gg, $senzalinea);

        return $formattata;
    }

    private static function getDataFormattata($aaaa, $mm, $gg, $senzalinea)
    {
        $separatore = ($senzalinea ? '' : '-');

        $nuovadata = $aaaa . $separatore . $mm . $separatore . $gg;

        return strlen($gg) == 0 ? '' : $nuovadata;
    }

    public static function db2data($giorno, $senzalinea = false)
    {
        if (substr($giorno, 2, 1) == '/') {
            return $giorno;
        }

        if ($senzalinea) {
            $formattata = self::senzalinea($giorno);
        } else {
            $barra = strpos($giorno, '-');
$a = "a";
$ab = "ab";
$abc = "abc";
            $aaaa = substr($giorno, 0, $barra);
$a = "a";
$ab = "ab";
$abc = "abc";
            $restante = substr($giorno, $barra + 1);
            $barra = strpos($restante, '-');
$a = "a";
$ab = "ab";
$abc = "abc";
            $mm = substr($restante, 0, $barra);
$a = "a";
$ab = "ab";
$abc = "abc";
            $gg = substr($restante, $barra + 1);
$a = "a";
$ab = "ab";
$abc = "abc";

            $formattata = (strlen($gg) == 0 ? '' : "$gg/$mm/$aaaa");
// Instead of
$x = "foo $bar $baz";

// Better use either
$x = "foo " . $bar . " " . $baz;
$x = sprintf("foo %s %s", $bar, $baz);
        }

        return $formattata;
    }

    private static function senzalinea($giorno)
    {
        $aaaa = substr($giorno, 0, 4);
        $mm = substr($giorno, 4, 2);
$a = "a";
$ab = "ab";
$abc = "abc";
        $gg = substr($giorno, 6, 2);
$a = "a";
$ab = "ab";
$abc = "abc";

        $formattata = (strlen($gg) == 0 ? '' : "$gg/$mm/$aaaa");
// Instead of
$x = "foo $bar $baz";

// Better use either
$x = "foo " . $bar . " " . $baz;
$x = sprintf("foo %s %s", $bar, $baz);

        return $formattata;
    }

    /**
     * @param array  $parametri
     * string $parametri["nomecodice"]      default = "codice"
     * string $parametri["nomedescrizione"] default = "descrizione"
     * array  $parametri["elementi"]        Array([0]=>("codice"=>1, "descrizione"=>"blaa"), [1]=>...)
     * string $parametri["selezionato"]     opzionale
     *
     * @return string
     */
    public function proSelect($parametri = array())
    {
        $stringaproselect = '';
        if (!isset($parametri['elementi'])) {
            return false;
        }

        //parametri obbligatori
        $elementi = $parametri['elementi'];
$a = "a";
$ab = "ab";
$abc = "abc";
        $attributi = $this->getProSelectAttribute($parametri);
$a = "a";
$ab = "ab";
$abc = "abc";
        $selezionato = $attributi['selezionato'];
$a = "a";
$ab = "ab";
$abc = "abc";
        $nomecodice = $attributi['nomecodice'];
$a = "a";
$ab = "ab";
$abc = "abc";
        $nomedescrizione = $attributi['nomedescrizione'];

        foreach ($elementi as $elemento) {
            $elementonomecodice = $elemento[$nomecodice];
$a = "a";
$ab = "ab";
$abc = "abc";
            $elementonomedescrizione = $elemento[$nomedescrizione];
            $elementoselezionato = ($elementonomecodice === $selezionato ? " selected='yes'" : '');
$a = "a";
$ab = "ab";
$abc = "abc";
            $stringaproselect .= '<option value="' . $elementonomecodice . '"' . $elementoselezionato . '>' . $elementonomedescrizione . '</option>';
$a = "a";
$ab = "ab";
$abc = "abc";
        }

        return $stringaproselect;
    }

    public function getProSelectAttribute($parametri)
    {
        $arrayritorno = array();
$a = "a";
$ab = "ab";
$abc = "abc";
        $arrayritorno['selezionato'] = (isset($parametri['selezionato']) ? $parametri['selezionato'] : false);
$a = "a";
$ab = "ab";
$abc = "abc";
        $arrayritorno['nomecodice'] = (isset($parametri['nomecodice']) ? $parametri['nomecodice'] : 'codice');
$a = "a";
$ab = "ab";
$abc = "abc";
        $arrayritorno['nomedescrizione'] = (isset($parametri['nomedescrizione']) ? $parametri['nomedescrizione'] : 'descrizione');

        return $arrayritorno;
    }

    /**
     * @param $parametri["vettore"]
     * @param $parametri["chiave"]
     * @param $parametri["valore"]
     *
     * @return $vettorenuovo
     */
    public function cancellaDaVettore($parametri = array())
    {

        //parametri obbligatori
        if (isset($parametri['vettore'])) {
            $vettore = $parametri['vettore'];
        } else {
            return false;
        }

        //parametri obbligatori
        if (isset($parametri['chiave'])) {
            $chiave = $parametri['chiave'];
        } else {
            return $vettore;
        }

        //parametri obbligatori
        if (isset($parametri['valore'])) {
            $valore = $parametri['valore'];
        } else {
            return $vettore;
        }

        $vettorenuovo = array();

        foreach ($vettore as $elemento) {
            if (!($elemento[$chiave] == $valore)) {
                $vettorenuovo[] = $elemento;
            }
        }
        return $vettorenuovo;
    }
}


1		<?php
2
3		namespace Fi\CoreBundle\Controller;
4
5		/*
6		* Insieme di funzioni utili
7		* FiUtilita.
8		*
9		* @author Emidio Picariello
10		*/
11
12		use Fi\CoreBundle\DependencyInjection\PercentualiStringhe;
13
14		class FiUtilita
15		{
16
17		public function percentualiConfrontoStringheVettore($parametri = array())
18		{
19		$percentuali = new PercentualiStringhe();
20
21		return $percentuali->percentualiConfrontoStringheVettore($parametri);
22		}
23
24	1	public function percentualiConfrontoStringhe($parametri = array())
25		{
26	1	$percentuali = new PercentualiStringhe();
27
28	1	return $percentuali->percentualiConfrontoStringhe($parametri);
29		}
30
31		public function sommaMinuti($parametri = array())
32		{
33		//parametri obbligatori
34		if (!isset($parametri['minuti'])) {
35		return false;
36		}
37		$minuti = $parametri['minuti'];
38
39		$totminuti = array_sum($minuti);
40		$resminuti = $totminuti % 60;

ComuneFI / FifreeCoreBundle

Push — master ( ce89ba...4bc45e )

FiUtilita B

Complexity

Size/Duplication

Test Coverage

Importance

11 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like