Utility::getCSV() - Code Metrics - Inspection of "Get country and municipality codes automatically" - lsolesen/edbbrugs-php-sdk - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#15)

by Lars

created 2018-05-29 11:55 UTC

Utility::getCSV() A

↳ Parent: Utility

Complexity

Conditions	3
Paths	3

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	3.0175

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	7
cts	8
cp	0.875
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	6
nc	3
nop	2
crap	3.0175

<?php
/**
 * Utility to get country and municipality codes with EDB-Brugs
 *
 * PHP Version 5
 *
 * @category EDBBrugs
 * @package  EDBBrugs
 * @author   Lars Olesen <[email protected]>
 * @license  MIT Open Source License https://opensource.org/licenses/MIT
 * @version  GIT: <git_id>
 */

namespace EDBBrugs;

use EDBBrugs\UtilityInterface;

/**
 * Utility to get country and municipality codes with EDB-Brugs
 *
 * @category EDBBrugs
 * @package  EDBBrugs
 * @author   Lars Olesen <[email protected]>
 * @license  MIT Open Source License https://opensource.org/licenses/MIT
 * @version  GIT: <git_id>
 */
class Utility implements UtilityInterface
{
    /**
     * Gets country code
     *
     * $return mixed integer / null
     */
    public function getCountryCode($country)
    {
        $rows = $this->getCSV(DIRNAME(__FILE__) . "/data/Landekoder.csv", $country);
        $stored_in_column = 2;
        return $rows[$stored_in_column];
    }

    /**
     * Gets municipality code
     *
     * $return mixed integer / null
     */
    public function getMunicipalityCode($municipality)
    {
        $rows = $this->getCSV(DIRNAME(__FILE__) . "/data/Kommuner.csv", $municipality);
        $stored_in_column = 2;
        return $rows[$stored_in_column];
    }

    /**
     * Loops through csv file and stops with search
     *
     * @param $file_name
     * @param $search
     * @return array
     */
    protected function getCSV($file_name, $search)
    {
        $ch = fopen($file_name, "r");
        $header_row = fgetcsv($ch);
$myVar = 'Value';
$higher = false;

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

        // This will loop through all the rows until it reaches the end
        while ($row = fgetcsv($ch)) {
            if (in_array($search, $row)) {
                return $row;
            }
        }
    }
}


1		<?php
2		/**
3		* Utility to get country and municipality codes with EDB-Brugs
4		*
5		* PHP Version 5
6		*
7		* @category EDBBrugs
8		* @package EDBBrugs
9		* @author Lars Olesen <[email protected]>
10		* @license MIT Open Source License https://opensource.org/licenses/MIT
11		* @version GIT: <git_id>
12		*/
13
14		namespace EDBBrugs;
15
16		use EDBBrugs\UtilityInterface;
17
18		/**
19		* Utility to get country and municipality codes with EDB-Brugs
20		*
21		* @category EDBBrugs
22		* @package EDBBrugs
23		* @author Lars Olesen <[email protected]>
24		* @license MIT Open Source License https://opensource.org/licenses/MIT
25		* @version GIT: <git_id>
26		*/
27		class Utility implements UtilityInterface
28		{
29		/**
30		* Gets country code
31		*
32		* $return mixed integer / null
33		*/
34	2	public function getCountryCode($country)
35		{
36	2	$rows = $this->getCSV(DIRNAME(__FILE__) . "/data/Landekoder.csv", $country);
37	2	$stored_in_column = 2;
38	2	return $rows[$stored_in_column];
39		}
40
41		/**
42		* Gets municipality code
43		*
44		* $return mixed integer / null
45		*/
46	2	public function getMunicipalityCode($municipality)
47		{
48	2	$rows = $this->getCSV(DIRNAME(__FILE__) . "/data/Kommuner.csv", $municipality);
49	2	$stored_in_column = 2;
50	2	return $rows[$stored_in_column];
51		}
52
53		/**
54		* Loops through csv file and stops with search
55		*
56		* @param $file_name
57		* @param $search
58		* @return array
59		*/
60	4	protected function getCSV($file_name, $search)
61		{
62	4	$ch = fopen($file_name, "r");
63	4	$header_row = fgetcsv($ch);
		0 ignored issues – show Unused Code introduced 2018-05-29 11:57 UTC by Report Bug Copy Issue Report `$header_row` 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...
64
65		// This will loop through all the rows until it reaches the end
66	4	while ($row = fgetcsv($ch)) {
67	4	if (in_array($search, $row)) {
68		return $row;
69		}
70	4	}
71	4	}
72		}
73

lsolesen / edbbrugs-php-sdk

Pull Request — master (#15)

Utility::getCSV() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like