modules\bataille\app\controller\GroupeUnite

Complexity

Total Complexity

17

Size/Duplication

Total Lines	178
Duplicated Lines	7.3 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	3
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getAllGroupeUnite()	13	21	4
A	getTestGroupExist()	0	15	2
A	testNomGroupe()	0	13	3
B	setCreerGroupe()	0	28	4
A	setAjouterUniteGroupe()	0	21	2
A	setCommencerExpedition()	0	17	1
A	setTerminerExpedition()	0	5	1

<?php
	namespace modules\bataille\app\controller;
	
	
	use core\App;
	use core\functions\ChaineCaractere;
	use core\HTML\flashmessage\FlashMessage;
	
	class GroupeUnite extends Unite {
		
		
		//-------------------------- BUILDER ----------------------------------------------------------------------------//
		//-------------------------- END BUILDER ----------------------------------------------------------------------------//
		
		
		//-------------------------- GETTER ----------------------------------------------------------------------------//
		/**
		 * fonction qui récupère tous les groupes de la bases
		 */
		public function getAllGroupeUnite() {
			$dbc = App::getDb();
			
			$query = $dbc->select()->from("_bataille_groupe_unite")
				->where("ID_base", "=", Bataille::getIdBase(), "AND")
				->where("(ID_mission IS NULL OR ID_mission = 0)", "", "", "", true)->get();
			
			if ((is_array($query)) && (count($query))) {

This code seems to be duplicated across your project.

				$groupe = [];
				
				foreach ($query as $obj) {
					$groupe[] = [
						"id_groupe" => $obj->ID_groupe_unite,
						"nom_groue" => $obj->nom_groupe,
						"unites" => $this->getAllUnites(Bataille::getIdBase(), $obj->ID_groupe_unite)
					];
				}
				
				Bataille::setValues(["groupe_unites" => $groupe]);
			}
		}
		
		/**
		 * @param $nom_groupe
		 * @return bool
		 * fonction qui test si un groupe existe déjà ou pas dans cette base
		 */
		private function getTestGroupExist($nom_groupe) {
			$dbc = App::getDb();
			
			$query = $dbc->select("ID_groupe_unite")
				->from("_bataille_groupe_unite")
				->where("nom_groupe", "=", $nom_groupe, "AND")
				->where("ID_base", "=", Bataille::getIdBase())
				->get();
			
			if (count($query) > 0) {
				return false;
			}
			
			return true;
		}
		//-------------------------- END GETTER ----------------------------------------------------------------------------//
		
		
		//-------------------------- SETTER ----------------------------------------------------------------------------//
		/**
		 * @param $nom_groupe
		 * @return bool
		 * permet de tester le nom d'un groupe si > a 0 caractere et qu'il soit libre
		 */
		private function testNomGroupe($nom_groupe) {
			if (ChaineCaractere::testMinLenght($nom_groupe) == false) {
				FlashMessage::setFlash("Le nom du groupe ne peut pas être vide !");
				return false;
			}
			
			if ($this->getTestGroupExist($nom_groupe) == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

				FlashMessage::setFlash("Un groupe portant ce nom existe déjà, merci d'en choisir un autre !");
				return false;
			}
			
			return true;
		}
		
		/**
		 * @param $nombre_unite
		 * @param $nom_unite
		 * @param $type_unite
		 * @param $nom_groupe
		 * @return bool
		 * fonction qui permet de créer un groupe
		 */
		public function setCreerGroupe($nombre_unite, $nom_unite, $type_unite, $nom_groupe) {
			$dbc = App::getDb();
			
			if ($this->testNomGroupe($nom_groupe) == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

				return false;
			}
			
			if (array_sum($nombre_unite) == 0) {
				FlashMessage::setFlash("Vous ne pouvez pas créer un groupe vide !");
				return false;
			}
			
			$dbc->insert("nom_groupe", $nom_groupe)
				->insert("niveau_groupe", 1)
				->insert("ID_base", Bataille::getIdBase())
				->into("_bataille_groupe_unite")
				->set();
			
			$id_groupe = $dbc->lastInsertId();
			
			$count = count($nombre_unite);
			
			for ($i=0 ; $i<$count ; $i++) {
				$this->setAjouterUniteGroupe($nombre_unite[$i], $nom_unite[$i], $type_unite[$i], $id_groupe);
			}
			
			return true;
		}
		
		/**
		 * @param $nombre_unite
		 * @param $nom_unite
		 * @param $type_unite
		 * @param $id_groupe
		 * @return bool
		 * fonction qui permet d'ajouter des unités a un groupe
		 */
		private function setAjouterUniteGroupe($nombre_unite, $nom_unite, $type_unite, $id_groupe) {
			$dbc = App::getDb();
			
			$nombre_unite_base = $this->getNombreUniteNom($type_unite, $nom_unite);
			
			if ($nombre_unite > $nombre_unite_base) {
				FlashMessage::setFlash("Pas assez d'unités ".$nom_unite." disponibles dans les ajouter à ce groupe");
				return false;
			}
			
			$dbc->update("ID_groupe", $id_groupe)
				->from("_bataille_unite")
				->where("type", "=", $type_unite, "AND")
				->where("nom", "=", $nom_unite, "AND")
				->where("ID_base", "=", Bataille::getIdBase(), "AND")
				->where("(ID_groupe IS NULL OR ID_groupe = 0)", "", "", "", true)
				->limit($nombre_unite, "no")
				->set();
			
			return true;
		}
		
		/**
		 * @param $id_groupe
		 * @param $id_mission
		 * @return bool
		 * fonction qui sert à envoyer toutes les troupes d'un groupe en mission
		 */
		public function setCommencerExpedition($id_groupe, $id_mission) {
			$dbc = App::getDb();
			
			$dbc->update("ID_mission", $id_mission)
				->from("_bataille_unite")
				->where("ID_groupe", "=", $id_groupe, "AND")
				->where("ID_base", "=", Bataille::getIdBase())
				->set();
			
			$dbc->update("ID_mission", $id_mission)
				->from("_bataille_groupe_unite")
				->where("ID_groupe_unite", "=", $id_groupe, "AND")
				->where("ID_base", "=", Bataille::getIdBase())
				->set();
			
			return true;
		}
		
		/**
		 * @param $id_mission
		 * fonction qui termine la mission pour tous les groupes qui y ont été assignés
		 */
		public function setTerminerExpedition($id_mission) {
			$dbc = App::getDb();
			
			$dbc->update("ID_mission", 0)->from("_bataille_groupe_unite")->where("ID_base", "=", Bataille::getIdBase())->set();
		}
		//-------------------------- END SETTER ----------------------------------------------------------------------------//    
	}