ntentan\atiaa\DbContext

Complexity

Total Complexity

8

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	initialize()	0	5	1
A	getInstance()	0	7	2
A	getDriver()	0	8	2
A	query()	0	4	1
A	destroy()	0	5	1

<?php

namespace ntentan\atiaa;

use ntentan\panie\Container;
use ntentan\utils\Text;

class DbContext

Since you have declared the constructor as private, maybe you should also declare the class as final.

{

    private $driver;
    private $driverFactory;
    private static $instance;

    private function __construct(DriverFactory $driverFactory)
    {
        $this->driverFactory = $driverFactory;
        self::$instance = $this;
    }

    public static function initialize(DriverFactory $driverFactory)
    {
        self::$instance = new self($driverFactory);
        return self::$instance;
    }

    public static function getInstance(): DbContext
    {
        if(!self::$instance) {
            throw new \Exception("The database context has not been initialized");  
        }
        return self::$instance;
    }

    /**
     *
     * @return Driver
     */
    public function getDriver() : Driver
    {
        if (is_null($this->driver)) {
            $this->driver = $this->driverFactory->createDriver();
            $this->driver->connect();
        }
        return $this->driver;
    }

    public function query($query, $bindData = false)
    {
        return $this->getDriver()->query($query, $bindData);

$bindData is of type boolean, but the function expects a false|array<integer,*>.

    }

    public static function destroy()
    {
        self::$instance->getDriver()->disconnect();
        self::$instance = null;
    }

}