Drone\Db\Driver\DriverFactory::create()

Complexity

Conditions	3
Paths	3

Size

Total Lines	17
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	3.0175

Importance

Changes

0

<?php
    public static function create($connection_options)
    {
        $drivers = [
            "Oci8"   => "Drone\Db\Driver\Oracle",
            "Mysqli" => "Drone\Db\Driver\MySQL",
            "Sqlsrv" => "Drone\Db\Driver\SQLServer",
        ];

        if (!array_key_exists('driver', $connection_options))

It is generally a best practice to always use braces with control structures.

            throw new \RuntimeException("The database driver key has not been declared");

        $drv = $connection_options["driver"];

        if (array_key_exists($drv, $drivers))

It is generally a best practice to always use braces with control structures.

            return new $drivers[$drv]($connection_options);
        else

Expected 1 space after ELSE keyword; newline found

It is generally a best practice to always use braces with control structures.

            throw new \RuntimeException("The database driver does not exists");
    }
}