Query

Complexity

Total Complexity

9

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	addConnection()	0	4	1
A	setConnection()	0	3	1
A	query()	0	4	1
A	queryInsert()	0	4	1
A	queryFetch()	0	4	1
A	queryArray()	0	4	1
A	stdQuery()	0	14	2

<?php
	class Query extends Module {

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

		public $connection;
		public $currentConnection;
		public function __construct() {
			parent::__construct();
			$this->connection = [];
			$this->currentConnection = null;
		}
		public function addConnection( $_name, $_connection ) {
			$this->connection["$_name"] = $_connection;
			$this->currentConnection = $_connection;
		}
		public function setConnection( $_name ) {
			$this->currentConnection = $this->connection["$_name"];
		}
		public function query( $_query ) {
			$this->stdQuery($_query);
			return true;
		}
		public function queryInsert( $_query ) {
			$temp = $this->stdQuery($_query);

$temp is not used, you could remove the assignment.

			return $this->currentConnection->database->lastInsertId();
		}

		public function queryFetch( $_query ) {
			$temp = $this->stdQuery($_query);
			return $temp->fetchAll(PDO::FETCH_ASSOC);
		}

		public function queryArray( $_query ) {
			$temp = $this->stdQuery($_query);
			return $temp->fetchAll(PDO::FETCH_COLUMN, 0);
		}

		private function stdQuery($_query) {
			$database = $this->currentConnection->database;
			$error = "Errore query [$_query]";
			$query = $database->prepare($_query);
			$_result = $query->execute();
			if(!$_result) {
				echo "$_query<br>";
				echo "Something wrong: ".$error;
				var_dump($query->errorInfo());

var_dump($query->errorInfo()); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

				var_dump($database->errorInfo());
				exit();

The method stdQuery() contains an exit expression.

			}
			return $query;
		}
	}
?>

It is not recommended to use PHP's closing tag ?> in files other than templates.