Config::factoryDriver() - Code Metrics - Inspection of "fix style code" - pedro151/orm-generator - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 5502fa...062bdf )

by Pedro

created 2016-04-27 20:10 UTC

Config::factoryDriver() B

↳ Parent: Config

Complexity

Conditions	8
Paths	8

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	5
Bugs	1	Features	0

Metric	Value
c	5
b	1
f	0
dl	0
loc	14
rs	7.7777
cc	8
eloc	9
nc	8
nop	0

<?php


namespace Classes;

use Classes\AdapterConfig\None;
use Classes\AdapterConfig\ZendFrameworkOne;
use Classes\AdaptersDriver\Dblib;
use Classes\AdaptersDriver\Mssql;
use Classes\AdaptersDriver\Mysql;
use Classes\AdaptersDriver\Pgsql;
use Classes\AdaptersDriver\Sqlsrv;

require_once 'AdapterConfig/None.php';
require_once 'AdapterConfig/ZendFrameworkOne.php';
require_once 'AdaptersDriver/Dblib.php';
require_once 'AdaptersDriver/Mssql.php';
require_once 'AdaptersDriver/Mysql.php';
require_once 'AdaptersDriver/Pgsql.php';
require_once 'AdaptersDriver/Sqlsrv.php';

/**
 * @author Pedro Alarcao <[email protected]>
 * @link   https://github.com/pedro151/ORM-Generator
 */
class Config
{

    /**
     * @var string
     */
    public static $version = "1.2.0";

    /**
     * String that separates the parent section name
     *
     * @var string
     */
    protected $sectionSeparator = ':';

    /**
     * @var string
     */
    private $configIniDefault = '/configs/config.ini';

    /**
     * @var string
     */
    public $_basePath;
    /**
     * @var array
     */
    private $argv = array ();

    /**
     * @var \Classes\AdapterConfig\AbstractAdapter
     */
    private $adapterConfig;

    /**
     * @var \Classes\AdaptersDriver\AbsractAdapter
     */
    private $adapterDriver;

    public function __construct ( $argv , $basePath )
    {
        if ( array_key_exists ( 'help' , $argv ) )
        {
            die ( $this->getUsage () );

        }
        if ( array_key_exists ( 'status' , $argv ) )
        {
            $argv[ 'status' ] = true;
        }

        $this->argv = $this->parseConfig ( $basePath , $argv );
    }

    /**
     * Lista de ajuda quando digita 'help'
     *
     * @return string
     */
    public function getUsage ()
    {
        $version = static::$version;

        return <<<EOF
parameters:

    --init                : Creates the necessary configuration file to start using the ORM-Generator.
    --config-ini          : reference to another .ini file configuration (relative path).
    --config-env          : ORM-Generator configuration environment.
    --framework           : name framework used, which has the contents of the database configurations and framework template.
    --driver              : database driver name (Ex.: pgsql).
    --database            : database name.
 *  --schema              : database schema name (one or more than one).
    --status              : show status of implementation carried out after completing the process.
    --version             : shows the version of ORM-Generator.
    --help                : help command explaining all the options and manner of use.
    --path                  specify where to create the files (default is current directory).

 example: php generate.php --framework=zend_framework --database=foo --table=foobar --status

ORM-Generator By: Pedro Alarcao Version: $version
EOF;
    }

    public function getVersion ()
    {
        $version = static::$version;

        return "ORM-Generator By: Pedro Alarcao Version: $version";
    }

    /**
     * Analisa e estrutura a Configuracao do generate
     *
     * @param  string $basePath
     * @param array   $argv
     *
     * @return array
     * @throws \Exception
     */
    private function parseConfig ( $basePath , $argv )
    {
        $this->_basePath = dirname ( $basePath );

        $configIni = isset( $argv[ 'config-ini' ] ) ? $argv[ 'config-ini' ]
            : $this->_basePath . $this->configIniDefault;

        $configTemp = $this->loadIniFile ( realpath ( $configIni ) );
        $configCurrent = self::parseConfigEnv ( $configTemp , $argv );

        if ( ! isset( $configCurrent[ 'framework' ] ) )
        {
            throw new \Exception( "configure which framework you want to use! \n" );
        }

        return $argv + array_filter ( $configCurrent );
    }

    /**
     *
     * @param $configTemp
     * @param $argv
     *
     * @return string
     */
    private static function parseConfigEnv ( $configTemp , $argv )
    {
        $thisSection = isset( $configTemp[ key ( $configTemp ) ][ 'config-env' ] ) ?
            $configTemp[ key ( $configTemp ) ][ 'config-env' ] : null;

        $thisSection = isset( $argv[ 'config-env' ] ) ? $argv[ 'config-env' ]
            : $thisSection;

        if ( isset( $configTemp[ $thisSection ][ 'extends' ] ) )
        {
            #faz marge da config principal com a config extendida
            return $configTemp[ $thisSection ]
                   + $configTemp[ $configTemp[ $thisSection ][ 'extends' ] ];
        }

        return $configTemp[ key ( $configTemp ) ];
    }

    /**
     * Carregar o arquivo ini e pré-processa o separador de seção ':'
     * no nome da seção (que é usado para a extensão seção) de modo a que a
     * matriz resultante tem os nomes de seção corretos e as informações de
     * extensão é armazenado em uma sub-ch ve
     *
     * @param string $filename
     *
     * @throws \Exception
     * @return array
     */
    protected function loadIniFile ( $filename )
    {
        if ( ! is_file ( $filename ) )
        {
            throw new \Exception( "configuration file does not exist! \n" );
        }

        $loaded = parse_ini_file ( $filename , true );
        $iniArray = array ();
        foreach ( $loaded as $key => $data )
        {
            $pieces = explode ( $this->sectionSeparator , $key );
            $thisSection = trim ( $pieces[ 0 ] );
            switch ( count ( $pieces ) )
            {
                case 1:
                    $iniArray[ $thisSection ] = $data;
                    break;

                case 2:
                    $extendedSection = trim ( $pieces[ 1 ] );
                    $iniArray[ $thisSection ] = array_merge ( array ( 'extends' => $extendedSection ) , $data );
                    break;

                default:
                    throw new \Exception( "Section '$thisSection' may not extend multiple sections in $filename" );
            }
        }

        return $iniArray;
    }

    /**
     * analisa a opção e cria a instancia do Atapter do determinado framework
     *
     */
    private function factoryConfig ()
    {
        switch ( strtolower ( $this->argv[ 'framework' ] ) )
        {
            case 'none':
                $this->adapterConfig = new None( $this->argv );
                break;
            case 'zend_framework':
                $this->adapterConfig = new ZendFrameworkOne( $this->argv );
                break;
        }

    }

    /**
     * Analisa a opção e instancia o determinado banco de dados
     *
     */
    private function factoryDriver ()
    {
        switch ( $this->argv[ 'driver' ] )
        {
            case 'pgsql':
            case 'pdo_pgsql': $this->adapterDriver = new Pgsql( $this->getAdapterConfig () ); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'mysql':
            case 'pdo_mysql': $this->adapterDriver = new Mysql( $this->getAdapterConfig () ); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'mssql': $this->adapterDriver = new Mssql( $this->getAdapterConfig () ); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'dblib':$this->adapterDriver = new Dblib( $this->getAdapterConfig () ); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'sqlsrv': $this->adapterDriver = new Sqlsrv( $this->getAdapterConfig () ); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
        }

    }

    /**
     * @return AdapterConfig\AbstractAdapter
     */
    public function getAdapterConfig ()
    {
        if ( ! $this->adapterConfig )
        {
            $this->factoryConfig ();
        }

        return $this->adapterConfig;
    }

    /**
     * @return AdaptersDriver\AbsractAdapter
     */
    public function getAdapterDriver ()
    {
        if ( ! $this->adapterDriver )
        {
            $this->factoryDriver ();
        }

        return $this->adapterDriver;
    }

}


1			<?php
			0 ignored issues – show Coding Style Compatibility introduced 2016-04-25 13:41 UTC by Report Bug Copy Issue Report For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 25 and the first side effect is on line 13. The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file. The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place. To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1. Loading history...
2
3			namespace Classes;
4
5			use Classes\AdapterConfig\None;
6			use Classes\AdapterConfig\ZendFrameworkOne;
7			use Classes\AdaptersDriver\Dblib;
8			use Classes\AdaptersDriver\Mssql;
9			use Classes\AdaptersDriver\Mysql;
10			use Classes\AdaptersDriver\Pgsql;
11			use Classes\AdaptersDriver\Sqlsrv;
12
13			require_once 'AdapterConfig/None.php';
14			require_once 'AdapterConfig/ZendFrameworkOne.php';
15			require_once 'AdaptersDriver/Dblib.php';
16			require_once 'AdaptersDriver/Mssql.php';
17			require_once 'AdaptersDriver/Mysql.php';
18			require_once 'AdaptersDriver/Pgsql.php';
19			require_once 'AdaptersDriver/Sqlsrv.php';
20
21			/**
22			* @author Pedro Alarcao <[email protected]>
23			* @link https://github.com/pedro151/ORM-Generator
24			*/
25			class Config
26			{
27
28			/**
29			* @var string
30			*/
31			public static $version = "1.2.0";
32
33			/**
34			* String that separates the parent section name
35			*
36			* @var string
37			*/
38			protected $sectionSeparator = ':';
39
40			/**
41			* @var string
42			*/
43			private $configIniDefault = '/configs/config.ini';
44
45			/**
46			* @var string
47			*/
48			public $_basePath;
49			/**
50			* @var array
51			*/
52			private $argv = array ();
53
54			/**
55			* @var \Classes\AdapterConfig\AbstractAdapter
56			*/
57			private $adapterConfig;
58
59			/**
60			* @var \Classes\AdaptersDriver\AbsractAdapter
61			*/
62			private $adapterDriver;
63
64			public function __construct ( $argv , $basePath )
65			{
66			if ( array_key_exists ( 'help' , $argv ) )
67			{
68			die ( $this->getUsage () );
			0 ignored issues – show Coding Style Compatibility introduced 2016-04-25 13:41 UTC by Report Bug Copy Issue Report The method `__construct()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
69			}
70			if ( array_key_exists ( 'status' , $argv ) )
71			{
72			$argv[ 'status' ] = true;
73			}
74
75			$this->argv = $this->parseConfig ( $basePath , $argv );
76			}
77
78			/**
79			* Lista de ajuda quando digita 'help'
80			*
81			* @return string
82			*/
83			public function getUsage ()
84			{
85			$version = static::$version;
86
87			return <<<EOF
88			parameters:
89
90			--init : Creates the necessary configuration file to start using the ORM-Generator.
91			--config-ini : reference to another .ini file configuration (relative path).
92			--config-env : ORM-Generator configuration environment.
93			--framework : name framework used, which has the contents of the database configurations and framework template.
94			--driver : database driver name (Ex.: pgsql).
95			--database : database name.
96			* --schema : database schema name (one or more than one).
97			--status : show status of implementation carried out after completing the process.
98			--version : shows the version of ORM-Generator.
99			--help : help command explaining all the options and manner of use.
100			--path specify where to create the files (default is current directory).
101
102			example: php generate.php --framework=zend_framework --database=foo --table=foobar --status
103
104			ORM-Generator By: Pedro Alarcao Version: $version
105			EOF;
106			}
107
108			public function getVersion ()
109			{
110			$version = static::$version;
111
112			return "ORM-Generator By: Pedro Alarcao Version: $version";
113			}
114
115			/**
116			* Analisa e estrutura a Configuracao do generate
117			*
118			* @param string $basePath
119			* @param array $argv
120			*
121			* @return array
122			* @throws \Exception
123			*/
124			private function parseConfig ( $basePath , $argv )
125			{
126			$this->_basePath = dirname ( $basePath );
127
128			$configIni = isset( $argv[ 'config-ini' ] ) ? $argv[ 'config-ini' ]
129			: $this->_basePath . $this->configIniDefault;
130
131			$configTemp = $this->loadIniFile ( realpath ( $configIni ) );
132			$configCurrent = self::parseConfigEnv ( $configTemp , $argv );
133
134			if ( ! isset( $configCurrent[ 'framework' ] ) )
135			{
136			throw new \Exception( "configure which framework you want to use! \n" );
137			}
138
139			return $argv + array_filter ( $configCurrent );
140			}
141
142			/**
143			*
144			* @param $configTemp
145			* @param $argv
146			*
147			* @return string
148			*/
149			private static function parseConfigEnv ( $configTemp , $argv )
150			{
151			$thisSection = isset( $configTemp[ key ( $configTemp ) ][ 'config-env' ] ) ?
152			$configTemp[ key ( $configTemp ) ][ 'config-env' ] : null;
153
154			$thisSection = isset( $argv[ 'config-env' ] ) ? $argv[ 'config-env' ]
155			: $thisSection;
156
157			if ( isset( $configTemp[ $thisSection ][ 'extends' ] ) )
158			{
159			#faz marge da config principal com a config extendida
160			return $configTemp[ $thisSection ]
161			+ $configTemp[ $configTemp[ $thisSection ][ 'extends' ] ];
162			}
163
164			return $configTemp[ key ( $configTemp ) ];
165			}
166
167			/**
168			* Carregar o arquivo ini e pré-processa o separador de seção ':'
169			* no nome da seção (que é usado para a extensão seção) de modo a que a
170			* matriz resultante tem os nomes de seção corretos e as informações de
171			* extensão é armazenado em uma sub-ch ve
172			*
173			* @param string $filename
174			*
175			* @throws \Exception
176			* @return array
177			*/
178			protected function loadIniFile ( $filename )
179			{
180			if ( ! is_file ( $filename ) )
181			{
182			throw new \Exception( "configuration file does not exist! \n" );
183			}
184
185			$loaded = parse_ini_file ( $filename , true );
186			$iniArray = array ();
187			foreach ( $loaded as $key => $data )
188			{
189			$pieces = explode ( $this->sectionSeparator , $key );
190			$thisSection = trim ( $pieces[ 0 ] );
191			switch ( count ( $pieces ) )
192			{
193			case 1:
194			$iniArray[ $thisSection ] = $data;
195			break;
196
197			case 2:
198			$extendedSection = trim ( $pieces[ 1 ] );
199			$iniArray[ $thisSection ] = array_merge ( array ( 'extends' => $extendedSection ) , $data );
200			break;
201
202			default:
203			throw new \Exception( "Section '$thisSection' may not extend multiple sections in $filename" );
204			}
205			}
206
207			return $iniArray;
208			}
209
210			/**
211			* analisa a opção e cria a instancia do Atapter do determinado framework
212			*
213			*/
214			private function factoryConfig ()
215			{
216			switch ( strtolower ( $this->argv[ 'framework' ] ) )
217			{
218			case 'none':
219			$this->adapterConfig = new None( $this->argv );
220			break;
221			case 'zend_framework':
222			$this->adapterConfig = new ZendFrameworkOne( $this->argv );
223			break;
224			}
225
226			}
227
228			/**
229			* Analisa a opção e instancia o determinado banco de dados
230			*
231			*/
232			private function factoryDriver ()
233			{
234			switch ( $this->argv[ 'driver' ] )
235			{
236			case 'pgsql':
237			case 'pdo_pgsql': $this->adapterDriver = new Pgsql( $this->getAdapterConfig () ); break;
			0 ignored issues – show Coding Style introduced 2016-04-27 20:12 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-04-27 20:12 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...
238			case 'mysql':
239			case 'pdo_mysql': $this->adapterDriver = new Mysql( $this->getAdapterConfig () ); break;
			0 ignored issues – show Coding Style introduced 2016-04-27 20:12 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-04-27 20:12 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...
240			case 'mssql': $this->adapterDriver = new Mssql( $this->getAdapterConfig () ); break;
			0 ignored issues – show Coding Style introduced 2016-04-27 20:12 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-04-27 20:12 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...
241			case 'dblib':$this->adapterDriver = new Dblib( $this->getAdapterConfig () ); break;
			0 ignored issues – show Coding Style introduced 2016-04-27 20:12 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-04-27 20:12 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...
242			case 'sqlsrv': $this->adapterDriver = new Sqlsrv( $this->getAdapterConfig () ); break;
			0 ignored issues – show Coding Style introduced 2016-04-27 20:12 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-04-27 20:12 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...
243			}
244
245			}
246
247			/**
248			* @return AdapterConfig\AbstractAdapter
249			*/
250			public function getAdapterConfig ()
251			{
252			if ( ! $this->adapterConfig )
253			{
254			$this->factoryConfig ();
255			}
256
257			return $this->adapterConfig;
258			}
259
260			/**
261			* @return AdaptersDriver\AbsractAdapter
262			*/
263			public function getAdapterDriver ()
264			{
265			if ( ! $this->adapterDriver )
266			{
267			$this->factoryDriver ();
268			}
269
270			return $this->adapterDriver;
271			}
272
273			}
274

pedro151 / orm-generator

GitHub Access Token became invalid

Push — master ( 5502fa...062bdf )

Config::factoryDriver() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like