Piou-piou / Ruined-World

modules/messagerie/app/controller/Messagerie.php

Indentation +3 added lines, -3 removed lines

@@ -76,10 +76,10 @@ 
 			return $this->message;
 		}
 		public function getDateMessage(){
-		    return $this->date_message;
+			return $this->date_message;
 		}
 		public function getUrl(){
-		    return $this->url;
+			return $this->url;
 		}
 		public function getIdExpediteur() {
 			return $this->id_expediteur;
@@ -94,7 +94,7 @@ 
 			return $this->pseudo_receveur;
 		}
 		public function getValues(){
-		    return ["messagerie" => $this->values];
+			return ["messagerie" => $this->values];
 		}
 
 		/**

Braces +3 added lines, -3 removed lines

@@ -76,10 +76,10 @@ 
 		public function getMessage() {
 			return $this->message;
 		}
-		public function getDateMessage(){
+		public function getDateMessage() {
 		    return $this->date_message;
 		}
-		public function getUrl(){
+		public function getUrl() {
 		    return $this->url;
 		}
 		public function getIdExpediteur() {
@@ -94,7 +94,7 @@ 
 		public function getPseudoReceveur() {
 			return $this->pseudo_receveur;
 		}
-		public function getValues(){
+		public function getValues() {
 		    return ["messagerie" => $this->values];
 		}
 

Spacing +4 added lines, -4 removed lines

@@ -54,10 +54,10 @@ 
 		public function getMessage() {
 			return $this->message;
 		}
-		public function getDateMessage(){
+		public function getDateMessage() {
 		    return $this->date_message;
 		}
-		public function getUrl(){
+		public function getUrl() {
 		    return $this->url;
 		}
 		public function getIdExpediteur() {
@@ -72,7 +72,7 @@ 
 		public function getPseudoReceveur() {
 			return $this->pseudo_receveur;
 		}
-		public function getValues(){
+		public function getValues() {
 		    return ["messagerie" => $this->values];
 		}
 
@@ -320,7 +320,7 @@ 
 				$destinataires = explode(",", $destinataire);
 				$c = count($destinataires);
 
-				for ($i=0 ; $i<$c ; $i++) {
+				for ($i = 0; $i < $c; $i++) {
 					if ($this->getIdIdentiteExist($destinataires[$i]) !== false) {
 						$destinataires[] = $this->getIdIdentiteExist($destinataires[$i]);
 						$expediteur = $_SESSION['idlogin'.CLEF_SITE];

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 	use core\App;
 	use core\functions\ChaineCaractere;
-	use modules\bataille\app\controller\Bataille;
 	
 	class Messagerie {
 		public static $url_message;

modules/bataille/app/controller/Marche.php

Doc Comments +1 added lines, -1 removed lines

@@ -189,7 +189,7 @@
 		}
 		
 		/**
-		 * @param $date_retour
+		 * @param double $date_retour
 		 * fonction qui place le trajet en retour
 		 */
 		private function setTrajetRetour($date_retour) {

modules/bataille/app/controller/Aide.php

Spacing +4 added lines, -4 removed lines

@@ -93,12 +93,12 @@ 
 		private function getPourConstruire($pour_construire) {
 			$count = count($pour_construire);
 			$batiment = [];
-			for ($i = 0 ; $i < $count ; $i++) {
+			for ($i = 0; $i < $count; $i++) {
 				//si plusieur batiment pour construire le batiment en question
 				$count = count($pour_construire[$i]);
 				if ($count > 1) {
-					for ($j = 0 ; $j < $count ; $j++) {
-						$batiment[] =  [
+					for ($j = 0; $j < $count; $j++) {
+						$batiment[] = [
 							"nom_batiment" => $pour_construire[$i][$j][0],
 							"niveau_batiment" => $pour_construire[$i][$j][2]
 						];
@@ -107,7 +107,7 @@ 
 					return ["batiments" => $batiment];
 				}
 				else {
-					$batiment[] =  [
+					$batiment[] = [
 						"nom_batiment" => $pour_construire[$i][0][0],
 						"niveau_batiment" => $pour_construire[$i][0][2]
 					];

modules/bataille/app/controller/Nourriture.php

Doc Comments +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
 		}
 		
 		/**
-		 * @param $nb_heure
+		 * @param double $nb_heure
 		 * @param $nb_unite
 		 * fonction qui calcule la nourriture consomee en 1h
 		 * si elle est inférieur à 0 on tue un nombre d'unite defini en fonction de combien le nombre de

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
 		 * fonction qui renvoi la durée écoulée depuis le dernier check de la nourriture
 		 */
 		private function getLastCheckNourriture() {
-			$dbc= App::getDb();
+			$dbc = App::getDb();
 			
 			$query = $dbc->select("last_check_nourriture")->from("_bataille_base")
 				->where("ID_identite", "=", Bataille::getIdIdentite(), "AND")
@@ -119,7 +119,7 @@ 
 			$nourriture_base = Bataille::getRessource()->getNourriture();
 			$nourriture_consommee = ($nb_unite*$this->getConsommationNourritureUnite())*$nb_heure;
 			
-			$nourriture_retirer = $nourriture_base - $nourriture_consommee;
+			$nourriture_retirer = $nourriture_base-$nourriture_consommee;
 			
 			if ($nourriture_retirer < 0) {
 				$unite_tuer = (abs(round($nourriture_retirer/100)))+1;

modules/bataille/app/controller/GenerationRapport.php

Doc Comments +5 added lines

@@ -19,6 +19,11 @@
 		
 		
 		//-------------------------- SETTER ----------------------------------------------------------------------------//
+
+		/**
+		 * @param string $titre
+		 * @param string $type
+		 */
 		public static function setGenererRapport($titre, $infos, $type) {
 			$chemin = MODULEROOT."bataille/app/controller/rapports/";
 			

modules/bataille/app/controller/Ressource.php

Doc Comments +5 added lines, -6 removed lines

@@ -71,7 +71,7 @@ 
 		}
 		
 		/**
-		 * @param $ressource
+		 * @param string $ressource
 		 * @return string
 		 * fonction qui sert à tester si on a atteint le stockage maximum pour une ressource en particulier
 		 */
@@ -89,8 +89,7 @@ 
 		}
 		
 		/**
-		 * @param null $id_base -> sert si definit a recuperer l'id identite de la abse en question
-		 * recupere la date de la derniere connexion
+		 * @return string|null
 		 */
 		private function getLastConnexion() {
 			$dbc = App::getDb();
@@ -147,9 +146,9 @@ 
 		}
 
 		/**
-		 * @param $nom_ressource
+		 * @param string $nom_ressource
 		 * @param $ressrouce
-		 * @param $diff_temps
+		 * @param integer $diff_temps
 		 * fonction qui ajoute les ressources qu'on a eu dans la base et qui reinitialise la last co a now
 		 */
 		private function setAddRessource($nom_ressource, $ressrouce, $diff_temps) {
@@ -182,7 +181,7 @@ 
 		 * @param $fer
 		 * @param $fuel
 		 * @param $nourriture
-		 * @param $signe -> contient + ou -
+		 * @param string $signe -> contient + ou -
 		 * fonction qui permet de retirer des ressources pour construire des batiment ou creer unités
 		 */
 		public function setUpdateRessource($eau, $electricite, $fer, $fuel, $nourriture, $signe) {

Spacing +5 added lines, -5 removed lines

@@ -264,11 +264,11 @@
 		 * fonction qui gere le calcul pour retirer de setUpdateRessource
 		 */
 		private function getCalcRetirerRessource($eau, $electricite, $fer, $fuel, $nourriture) {
-			$eau = $this->eau - $eau;
-			$electricite = $this->electricite - $electricite;
-			$fer = $this->fer - $fer;
-			$fuel = $this->fuel - $fuel;
-			$nourriture = $this->nourriture - $nourriture;
+			$eau = $this->eau-$eau;
+			$electricite = $this->electricite-$electricite;
+			$fer = $this->fer-$fer;
+			$fuel = $this->fuel-$fuel;
+			$nourriture = $this->nourriture-$nourriture;
 			
 			if ($eau < 0) $eau = 0;
 			if ($electricite < 0) $electricite = 0;

modules/bataille/app/controller/CentreRecherche.php

Spacing +7 added lines, -7 removed lines

@@ -54,16 +54,16 @@ 
 		 * fonction qui renvoi le cout d'une recherche
 		 */
 		private function getCoutRecherche($cout, $niveau_recherche) {
-			$cout_eau = $cout["eau"] * ($this->coef_centre * $niveau_recherche);
+			$cout_eau = $cout["eau"]*($this->coef_centre*$niveau_recherche);
 			$cout_eau = Bataille::getTestAssezRessourceBase("eau", $cout_eau);
 			
-			$cout_electricite = $cout["electricite"] * ($this->coef_centre * $niveau_recherche);
+			$cout_electricite = $cout["electricite"]*($this->coef_centre*$niveau_recherche);
 			$cout_electricite = Bataille::getTestAssezRessourceBase("electricite", $cout_electricite);
 			
-			$cout_fer = $cout["fer"] * ($this->coef_centre * $niveau_recherche);
+			$cout_fer = $cout["fer"]*($this->coef_centre*$niveau_recherche);
 			$cout_fer = Bataille::getTestAssezRessourceBase("fer", $cout_fer);
 			
-			$cout_fuel = $cout["fuel"] * ($this->coef_centre * $niveau_recherche);
+			$cout_fuel = $cout["fuel"]*($this->coef_centre*$niveau_recherche);
 			$cout_fuel = Bataille::getTestAssezRessourceBase("fuel", $cout_fuel);
 			
 			return [
@@ -83,10 +83,10 @@ 
 			$pourcent = ($temps*Bataille::getBatiment()->getNiveauBatiment("centre_recherche")/100);
 
 			if ($niveau == 0) {
-				return round($temps-$pourcent);;
+				return round($temps-$pourcent); ;
 			}
 
-			return round(($temps * ($this->coef_centre * $niveau))-$pourcent);
+			return round(($temps*($this->coef_centre*$niveau))-$pourcent);
 		}
 
 		/**
@@ -256,7 +256,7 @@ 
 			$fuel = Bataille::getTestAssezRessourceBase("fuel", $cout["fuel"]["ressource"]);
 
 
-			if (($eau["class"] || $electricite["class"] || $fer["class"] || $fuel["class"]) == "rouge" ) {
+			if (($eau["class"] || $electricite["class"] || $fer["class"] || $fuel["class"]) == "rouge") {
 				FlashMessage::setFlash("Pas assez de ressources pour effectuer cette recherche");
 				return false;
 			}

modules/bataille/app/controller/Batiment.php

Spacing +6 added lines, -6 removed lines

@@ -328,13 +328,13 @@ 
 			$batiment_construit = $this->getBatimentBase();
 
 			//recuperation de la liste complete des batiments
-			$c_all_batiment  = $this->getAllBatimentGame();
+			$c_all_batiment = $this->getAllBatimentGame();
 
 			//boucle qui recupere en tableau le champ pour_construire d'un batiment
 			//et compare la liste des batiments qu'il faut pour construire le batiment
 			//a ceux qui sont deja construit dans la base
 			//si tous les batments qu'il faut son batis on autorise la construction du batiment
-			for ($i = 0 ; $i < $c_all_batiment ; $i++) {
+			for ($i = 0; $i < $c_all_batiment; $i++) {
 				if (!in_array($this->all_batiment[$i], $batiment_construit)) {
 					$query = $dbc1->select("pour_construire")
 						->select("temps_construction")
@@ -358,7 +358,7 @@ 
 						$ok_construction = false;
 						//test si tous les batiments sont construits et on le niveau nécéssaire
 						$count = count($pour_construire);
-						for ($j = 0 ; $j < $count ; $j++) {
+						for ($j = 0; $j < $count; $j++) {
 							if (in_array($pour_construire[$j][1], $batiment_construit)) {
 								if ($pour_construire[$j][2] <= $this->getNiveauBatiment($pour_construire[$j][1])) {
 									$ok_construction = true;
@@ -443,8 +443,8 @@ 
 					$finx_batiment = $taille_batiment[0]+$posx_batiment;
 					$finy_batiment = $taille_batiment[1]+$posy_batiment;
 					
-					for ($i = $posy ; $i < $finy ; $i++) {
-						for ($j = $posx ; $j < $finx ; $j++) {
+					for ($i = $posy; $i < $finy; $i++) {
+						for ($j = $posx; $j < $finx; $j++) {
 							if ((($posx++ >= $posx_batiment) && ($posx++ <= $finx_batiment)) && (($posy++ >= $posy_batiment) && ($posy++ <= $finy_batiment))) {
 								FlashMessage::setFlash("Un batiment est déjà présent à cet emplacement, merci d'en choisir un autre");
 								return false;
@@ -630,7 +630,7 @@ 
 			$ressource = $this->getRessourceConstruireBatiment($this->nom_batiment_sql, $this->niveau_batiment);
 			
 			//si pas assez de ressource
-			if (($ressource["eau"]["class"] || $ressource["electricite"]["class"] ||$ressource["fer"]["class"] || $ressource["fuel"]["class"]) == "rouge") {
+			if (($ressource["eau"]["class"] || $ressource["electricite"]["class"] || $ressource["fer"]["class"] || $ressource["fuel"]["class"]) == "rouge") {
 				FlashMessage::setFlash("Pas assez de ressources pour construire ce batiment");
 				return false;
 			}

modules/bataille/app/controller/Unite.php

Spacing +5 added lines, -5 removed lines

@@ -88,7 +88,7 @@ 
 			$unites = Bataille::getCentreRecherche()->getAllRechercheType($type);
 
 			//recupérer les caractéristiques de l'unité en question
-			for ($i=0 ; $i<count($unites) ; $i++) {
+			for ($i = 0; $i < count($unites); $i++) {
 				$unites[$i] += $this->getCaracteristiqueUnite($unites[$i]["recherche"], $unites[$i]["niveau"], $type);
 				$unites[$i] += ["type" => $type];
 			}
@@ -139,7 +139,7 @@ 
 			$count_type = count($types);
 			$unites = [];
 
-			for ($i=0 ; $i<$count_type ; $i++) {
+			for ($i = 0; $i < $count_type; $i++) {
 				$type_unite = $this->getAllUniteType($types[$i], $id_base, $id_groupe);
 
 				$unites = array_merge($unites, $type_unite);
@@ -300,7 +300,7 @@ 
 			$fuel = Bataille::getTestAssezRessourceBase("fuel", $retirer_fuel);
 
 
-			if (($eau["class"] || $electricite["class"] || $fer["class"] || $fuel["class"]) == "rouge" ) {
+			if (($eau["class"] || $electricite["class"] || $fer["class"] || $fuel["class"]) == "rouge") {
 				FlashMessage::setFlash("Pas assez de ressources pour recruter autant d'unités");
 				return false;
 			}
@@ -308,7 +308,7 @@ 
 				//on retire les ressources
 				Bataille::getRessource()->setUpdateRessource($retirer_eau, $retirer_electricite, $retirer_fer, $retirer_fuel, 0, "-");
 
-				$date_fin = Bataille::getToday()+($this->temps_recrutement *$nombre);
+				$date_fin = Bataille::getToday()+($this->temps_recrutement*$nombre);
 
 				$dbc->insert("nom", $nom)
 					->insert("type", $type)
@@ -339,7 +339,7 @@ 
 					$type = $obj->type;
 				}
 
-				for ($i=0 ; $i<$nombre ; $i++) {
+				for ($i = 0; $i < $nombre; $i++) {
 					$dbc->insert("nom", $nom)
 						->insert("type", $type)
 						->insert("ID_base", Bataille::getIdBase())

Doc Comments +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 		
 		//-------------------------- GETTER ----------------------------------------------------------------------------//
 		/**
-		 * @param $caracteristique
+		 * @param string $caracteristique
 		 * @param $force
 		 * @param $niveau
 		 * @return float