Database::query() - Code Metrics - Inspection of "Separaing QueryBuilder class of Database class" - Piou-piou/ribs-framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 535a29...e8ae10 )

by Anthony

created 2016-03-15 16:28 UTC

Database::query() A

↳ Parent: Database

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	7

Duplication

Lines	11
Ratio	100 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	11
loc	11
rs	9.4285
cc	2
eloc	7
nc	2
nop	1

<?php
	namespace core;

	use core\HTML\flashmessage\FlashMessage;
	use PDO;

	class Database {
		private $db_type;
		private $db_name;
		private $db_user;
		private $db_pass;
		private $db_host;
		private $dbc;

		//pour le query builder
		private $req_beginning;
		private $champs = [];
		private $value = [];
		private $conditions = [];
		private $table = [];



		//-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
		public function __construct($db_type, $db_name, $db_user, $db_pass, $db_host) {

			$this->db_type = $db_type;
			$this->db_name = $db_name;
			$this->db_user = $db_user;
			$this->db_pass = $db_pass;
			$this->db_host = $db_host;
		}
		//-------------------------- FIN CONSTRUCTEUR ----------------------------------------------------------------------------//



		//-------------------------- GETTER ----------------------------------------------------------------------------//
		/**
		 * function qui fait la connexion a la bdd ne peu etre appelee que dans la classe
		 * @return PDO
		 */
		private function getPdo() {

			if ($this->dbc === null) {
				$dbc = new PDO($this->db_type.':host='.$this->db_host.';dbname='.$this->db_name, $this->db_user, $this->db_pass);
				$dbc->exec("set names utf8");
				$this->dbc = $dbc;
			}
			return $this->dbc;
		}
		//-------------------------- FIN GETTER ----------------------------------------------------------------------------//

		//-------------------------- FUNCTION QUI FONT DES REQUETES SUR LA BDD --------------------------------------------//
		/**
		 * effectue une requete en selectr dans la BDD, si ok on renvoit les donnees sinon on renvoi une erreur
		 * @param $req
		 * @return array
		 */
		public function query($req) {

			$query = $this->getPdo()->query($req);

			if ($query) {
				$obj = $query->fetchAll(PDO::FETCH_OBJ);
				return $obj;
			}
			else {
				FlashMessage::setFlash("Une erreur est survenue en executant cette requette : ".$req);
			}
		}

		/**
		 * fonction qui prepare une requete et qui l'envoi, marche pour insert et update et delete
		 * @param $req -> la req a executer
		 * @param $value -> le ou les tableaux de valeurs
		 */
		public function prepare($req, $value) {

			$query = $this->getPdo()->prepare($req);
			//si on a plusieurs tableaux
			if (array_key_exists(0, $value)) {
				foreach ($value as $val) {
					if (!$query->execute($val)) {
						$err = true;
					}
				}
			}
			else {
				if (!$query->execute($value)) {
					$err = true;
				}
			}

			//si on a une erreur on renvoi un message
			if (isset($err)) {
				FlashMessage::setFlash("Une erreur est survenue en executant cette requette : ".$req);
			}
		}

		/**
		 * pour savoir si une valeur sur un champ précis existe deja en bdd, renvoi true si vrai
		 * @param $table
		 * @param $champ
		 * @param $value
		 * @return boolean|null
		 */
		public function rechercherEgalite($table, $champ, $value, $id_table = null, $id = null) {

			if ($id == null) {
				$query = $this->getPdo()->query("SELECT COUNT($champ) as nb FROM $table WHERE $champ LIKE '$value'");
			}
			else {
				$query = $this->getPdo()->query("SELECT COUNT($champ) as nb FROM $table WHERE $champ LIKE '$value' AND $id_table != $id");
			}

			if (count($query) > 0) {
				foreach ($query as $obj) {
					$nb = $obj["nb"];
				}

				if ((isset($nb)) && ($nb != 0)) return true;
			}
			else {
				return false;
			}
		}
		//-------------------------- FIN FUNCTION QUI FONT DES REQUETES SUR LA BDD --------------------------------------------//


		/**
		 * tester si une table dans la base donnee existe
		 * @param string $table definit la table pour laquelle on doit tester l'existance
		 * @return boolean
		 */
		public function TestTableExist($table) {

			$query = $this->getPdo()->query("SHOW TABLES LIKE '$table'");

			if ($query->rowCount() > 0) {
				return true;
			}
			else {
				return false;
			}
		}

		public function quote($quote) {
			return $this->getPdo()->quote($quote);
		}

		public function lastInsertId() {
			return $this->getPdo()->lastInsertId();
		}



		//-------------------------- QUERY BUILDER in construction no test have been done --------------------------------------------//
		/**
		 * @param string $champs
		 * @return $this
		 *
		 * pour initialisé une requete avec un select
		 */
		public function select($champs = "*") {
			$this->req_beginning = "SELECT ";
			$this->champs[] = $champs;

			return $this;
		}

		/**
		 * @param $champ
		 * @param $value
		 * @return $this
		 *
		 * fonction qui permet de préparer les champs et la valeur qui y sera associée
		 */
		public function insert($champ, $value) {
			$this->add($champ, $value);

			$this->req_beginning = "INSERT INTO ";

			return $this;
		}

		/**
		 * @param $champ
		 * @param $value
		 * @return $this
		 */
		public function update($champ, $value) {
			$this->add($champ, $value);

			$this->req_beginning = "UPDATE ";

			return $this;
		}

		/**
		 * @return $this
		 *
		 * fonction qui initialise un delete en base de donnée
		 */
		public function delete() {
			$this->req_beginning = "DELETE FROM ";

			return $this;
		}

		/**
		 * @param $champ
		 * @param $value
		 *
		 * fonction qui se cahrge d'ajouter les valeurs et les champs si non null dans leurs
		 * tableaux respectifs (appellée dans this->insert et this->update
		 */
		private function add($champ, $value) {
			if (($champ !== null) && ($value !== null)) {

				$this->champs[] = $champ;
				$this->value[] = $value;
			}
		}

		/**
		 * @param $table
		 * @return $this
		 *
		 * pour initialiser la les listes des tables ou il faudra aler chercher les données
		 */
		public function from($table) {
			$this->table[] = $table;

			return $this;
		}

		/**
		 * @param $table
		 *
		 * pour initialiser la table dans laquelle on va insérer les données
		 */
		public function into($table) {
			$this->table[] = $table;

			return $this;
		}

		/**
		 * @param $champ
		 * @param $cond
		 * @param $champ_test
		 * @param null $closure
		 * @return $this
		 *
		 * pour intialiser la ou les clauses where d'une requete
		 */
		public function where($champ, $cond, $champ_test, $closure = null) {

			if ( $closure === null) {
				$this->conditions[] = $champ.$cond.$champ_test;
			}
			else {
				$this->conditions[] = $champ.$cond.$champ_test." ".$closure;
			}

			return $this;
		}

		/**
		 * @return array
		 *
		 * fonction qui permet de récupérer un select fait sur une table
		 */
		public function get() {

			$requete = $this->req_beginning . implode(",", $this->champs) . " FROM " . implode(",", $this->table);

			if (!empty($this->conditions)) {
				$requete .= " WHERE ". implode(" ", $this->conditions);
			}

			$this->unsetQueryBuilder();
			return $this->query($requete);
		}

		/**
		 * fonction utlisée pour terminer un insert ou un update dans la base de données
		 */
		public function set() {

			$values = array_combine($this->champs, $this->value);

			$datas = [];
			$count = count($this->champs);
			for ($i=0 ; $i<$count ; $i++) {
				$datas[] = $this->champs[$i]."=:".$this->champs[$i];
			}

			//si on a des conditions alors on sera dans un insert
			$requete = $this->req_beginning . implode(",", $this->table) . " SET " . implode(", ", $datas);

			if (!empty($this->conditions)) {
				$requete .= " WHERE " . implode(" ", $this->conditions);
			}

			$this->prepare($requete, $values);
			$this->unsetQueryBuilder();
		}

		/**
		 * fonction utilisée pour finir un delete
		 */
		public function del() {

			$requete = $this->req_beginning . implode(",", $this->table);

			if (!empty($this->conditions)) {
				$requete .= " WHERE " . implode(" ", $this->conditions);
			}

			$this->query($requete);
			$this->unsetQueryBuilder();
		}

		/**
		 * fonction qui détruit toutes les variables utilisées.
		 */
		private function unsetQueryBuilder() {

			$this->req_beginning;
			$this->champs = [];
			$this->value = [];
			$this->conditions = [];
			$this->table = [];
		}
	}

1		<?php
2		namespace core;
3
4		use core\HTML\flashmessage\FlashMessage;
5		use PDO;
6
7		class Database {
8		private $db_type;
9		private $db_name;
10		private $db_user;
11		private $db_pass;
12		private $db_host;
13		private $dbc;
14
15		//pour le query builder
16		private $req_beginning;
17		private $champs = [];
18		private $value = [];
19		private $conditions = [];
20		private $table = [];
21
22
23
24		//-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
25	View Code Duplication	public function __construct($db_type, $db_name, $db_user, $db_pass, $db_host) {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
26		$this->db_type = $db_type;
27		$this->db_name = $db_name;
28		$this->db_user = $db_user;
29		$this->db_pass = $db_pass;
30		$this->db_host = $db_host;
31		}
32		//-------------------------- FIN CONSTRUCTEUR ----------------------------------------------------------------------------//
33
34
35
36		//-------------------------- GETTER ----------------------------------------------------------------------------//
37		/**
38		* function qui fait la connexion a la bdd ne peu etre appelee que dans la classe
39		* @return PDO
40		*/
41	View Code Duplication	private function getPdo() {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
42		if ($this->dbc === null) {
43		$dbc = new PDO($this->db_type.':host='.$this->db_host.';dbname='.$this->db_name, $this->db_user, $this->db_pass);
44		$dbc->exec("set names utf8");
45		$this->dbc = $dbc;
46		}
47		return $this->dbc;
48		}
49		//-------------------------- FIN GETTER ----------------------------------------------------------------------------//
50
51		//-------------------------- FUNCTION QUI FONT DES REQUETES SUR LA BDD --------------------------------------------//
52		/**
53		* effectue une requete en selectr dans la BDD, si ok on renvoit les donnees sinon on renvoi une erreur
54		* @param $req
55		* @return array
56		*/
57	View Code Duplication	public function query($req) {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
58		$query = $this->getPdo()->query($req);
59
60		if ($query) {
61		$obj = $query->fetchAll(PDO::FETCH_OBJ);
62		return $obj;
63		}
64		else {
65		FlashMessage::setFlash("Une erreur est survenue en executant cette requette : ".$req);
66		}
67		}
68
69		/**
70		* fonction qui prepare une requete et qui l'envoi, marche pour insert et update et delete
71		* @param $req -> la req a executer
72		* @param $value -> le ou les tableaux de valeurs
73		*/
74	View Code Duplication	public function prepare($req, $value) {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
75		$query = $this->getPdo()->prepare($req);
76		//si on a plusieurs tableaux
77		if (array_key_exists(0, $value)) {
78		foreach ($value as $val) {
79		if (!$query->execute($val)) {
80		$err = true;
81		}
82		}
83		}
84		else {
85		if (!$query->execute($value)) {
86		$err = true;
87		}
88		}
89
90		//si on a une erreur on renvoi un message
91		if (isset($err)) {
92		FlashMessage::setFlash("Une erreur est survenue en executant cette requette : ".$req);
93		}
94		}
95
96		/**
97		* pour savoir si une valeur sur un champ précis existe deja en bdd, renvoi true si vrai
98		* @param $table
99		* @param $champ
100		* @param $value
101		* @return boolean\|null
102		*/
103	View Code Duplication	public function rechercherEgalite($table, $champ, $value, $id_table = null, $id = null) {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
104		if ($id == null) {
105		$query = $this->getPdo()->query("SELECT COUNT($champ) as nb FROM $table WHERE $champ LIKE '$value'");
106		}
107		else {
108		$query = $this->getPdo()->query("SELECT COUNT($champ) as nb FROM $table WHERE $champ LIKE '$value' AND $id_table != $id");
109		}
110
111		if (count($query) > 0) {
112		foreach ($query as $obj) {
113		$nb = $obj["nb"];
114		}
115
116		if ((isset($nb)) && ($nb != 0)) return true;
117		}
118		else {
119		return false;
120		}
121		}
122		//-------------------------- FIN FUNCTION QUI FONT DES REQUETES SUR LA BDD --------------------------------------------//
123
124
125		/**
126		* tester si une table dans la base donnee existe
127		* @param string $table definit la table pour laquelle on doit tester l'existance
128		* @return boolean
129		*/
130	View Code Duplication	public function TestTableExist($table) {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
131		$query = $this->getPdo()->query("SHOW TABLES LIKE '$table'");
132
133		if ($query->rowCount() > 0) {
134		return true;
135		}
136		else {
137		return false;
138		}
139		}
140
141		public function quote($quote) {
142		return $this->getPdo()->quote($quote);
143		}
144
145		public function lastInsertId() {
146		return $this->getPdo()->lastInsertId();
147		}
148
149
150
151		//-------------------------- QUERY BUILDER in construction no test have been done --------------------------------------------//
152		/**
153		* @param string $champs
154		* @return $this
155		*
156		* pour initialisé une requete avec un select
157		*/
158		public function select($champs = "*") {
159		$this->req_beginning = "SELECT ";
160		$this->champs[] = $champs;
161
162		return $this;
163		}
164
165		/**
166		* @param $champ
167		* @param $value
168		* @return $this
169		*
170		* fonction qui permet de préparer les champs et la valeur qui y sera associée
171		*/
172		public function insert($champ, $value) {
173		$this->add($champ, $value);
174
175		$this->req_beginning = "INSERT INTO ";
176
177		return $this;
178		}
179
180		/**
181		* @param $champ
182		* @param $value
183		* @return $this
184		*/
185		public function update($champ, $value) {
186		$this->add($champ, $value);
187
188		$this->req_beginning = "UPDATE ";
189
190		return $this;
191		}
192
193		/**
194		* @return $this
195		*
196		* fonction qui initialise un delete en base de donnée
197		*/
198		public function delete() {
199		$this->req_beginning = "DELETE FROM ";
200
201		return $this;
202		}
203
204		/**
205		* @param $champ
206		* @param $value
207		*
208		* fonction qui se cahrge d'ajouter les valeurs et les champs si non null dans leurs
209		* tableaux respectifs (appellée dans this->insert et this->update
210		*/
211		private function add($champ, $value) {
212	View Code Duplication	if (($champ !== null) && ($value !== null)) {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
213		$this->champs[] = $champ;
214		$this->value[] = $value;
215		}
216		}
217
218		/**
219		* @param $table
220		* @return $this
221		*
222		* pour initialiser la les listes des tables ou il faudra aler chercher les données
223		*/
224		public function from($table) {
225		$this->table[] = $table;
226
227		return $this;
228		}
229
230		/**
231		* @param $table
232		*
233		* pour initialiser la table dans laquelle on va insérer les données
234		*/
235		public function into($table) {
236		$this->table[] = $table;
237
238		return $this;
239		}
240
241		/**
242		* @param $champ
243		* @param $cond
244		* @param $champ_test
245		* @param null $closure
246		* @return $this
247		*
248		* pour intialiser la ou les clauses where d'une requete
249		*/
250	View Code Duplication	public function where($champ, $cond, $champ_test, $closure = null) {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
251		if ( $closure === null) {
252		$this->conditions[] = $champ.$cond.$champ_test;
253		}
254		else {
255		$this->conditions[] = $champ.$cond.$champ_test." ".$closure;
256		}
257
258		return $this;
259		}
260
261		/**
262		* @return array
263		*
264		* fonction qui permet de récupérer un select fait sur une table
265		*/
266	View Code Duplication	public function get() {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
267		$requete = $this->req_beginning . implode(",", $this->champs) . " FROM " . implode(",", $this->table);
268
269		if (!empty($this->conditions)) {
270		$requete .= " WHERE ". implode(" ", $this->conditions);
271		}
272
273		$this->unsetQueryBuilder();
274		return $this->query($requete);
275		}
276
277		/**
278		* fonction utlisée pour terminer un insert ou un update dans la base de données
279		*/
280	View Code Duplication	public function set() {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
281		$values = array_combine($this->champs, $this->value);
282
283		$datas = [];
284		$count = count($this->champs);
285		for ($i=0 ; $i<$count ; $i++) {
286		$datas[] = $this->champs[$i]."=:".$this->champs[$i];
287		}
288
289		//si on a des conditions alors on sera dans un insert
290		$requete = $this->req_beginning . implode(",", $this->table) . " SET " . implode(", ", $datas);
291
292		if (!empty($this->conditions)) {
293		$requete .= " WHERE " . implode(" ", $this->conditions);
294		}
295
296		$this->prepare($requete, $values);
297		$this->unsetQueryBuilder();
298		}
299
300		/**
301		* fonction utilisée pour finir un delete
302		*/
303	View Code Duplication	public function del() {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
304		$requete = $this->req_beginning . implode(",", $this->table);
305
306		if (!empty($this->conditions)) {
307		$requete .= " WHERE " . implode(" ", $this->conditions);
308		}
309
310		$this->query($requete);
311		$this->unsetQueryBuilder();
312		}
313
314		/**
315		* fonction qui détruit toutes les variables utilisées.
316		*/
317	View Code Duplication	private function unsetQueryBuilder() {
		0 ignored issues – show Duplication introduced 2016-03-15 16:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
318		$this->req_beginning;
319		$this->champs = [];
320		$this->value = [];
321		$this->conditions = [];
322		$this->table = [];
323		}
324		}

Piou-piou / ribs-framework

Push — master ( 535a29...e8ae10 )

Database::query() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like