Win\Connection\Database

Complexity

Total Complexity

5

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
	connect()	0	1	?
A	getPDO()	0	3	1
A	instance()	0	3	1
A	__construct()	0	9	1
A	validateConnection()	0	7	2

<?php

namespace Win\Connection;

/**
 * Conexão com banco de dados
 *
 */
use PDO;
use Win\Mvc\Application;
use Win\Mvc\View;

abstract class Database {

	/** @var \static */
	static protected $instance;

	/** @var PDO */
	protected $pdo;

	/**
	 * Cria e retorna conexao PDO
	 * @param string[] $dbConfig
	 * @return PDO
	 */
	abstract function connect(&$dbConfig);

It is generally recommended to explicitly declare the visibility for methods.

	/** @return PDO */
	final public function getPDO() {
		return $this->pdo;
	}

	/** @return static */
	final static function instance() {

It is generally recommended to explicitly declare the visibility for methods.

		return self::$instance;
	}

	/**
	 * Cria uma conexão com um banco de dados
	 * @param string[] $dbConfig
	 */
	public function __construct($dbConfig) {
		self::$instance = $this;

It seems like $this of type this<Win\Connection\Database> is incompatible with the declared type object<static> of property $instance.

		$errorLevel = error_reporting(0);
		$this->pdo = $this->connect($dbConfig);
		error_reporting($errorLevel);

		$this->validateConnection();
	}

	protected function validateConnection() {
		if ($this->pdo->errorCode()) {
			$data['error'] = $this->pdo->errorInfo();

$data was never initialized. Although not strictly required by PHP, it is generally a good practice to add $data = array(); before regardless.

			Application::app()->view = new View('500', $data);
			Application::app()->setTitle('Problemas de Conexão');
		}
	}

}