Piou-piou / Ruined-World

modules/bataille/app/controller/Bataille.php

Doc Comments +6 added lines, -6 removed lines

@@ -161,7 +161,7 @@ 
 
 		/**
 		 * @param null $id_base -> sert si definit a recuperer l'id identite de la abse en question
-		 * @return mixed
+		 * @return string|null
 		 * recupere la date de la derniere connexion
 		 */
 		public static function getLastConnexion($id_base = null) {
@@ -188,7 +188,7 @@ 
 		}
 
 		/**
-		 * @param $nom_ressource
+		 * @param string $nom_ressource
 		 * @param $ressource
 		 * @return array
 		 * fonction qui permet de renvyer la couleur rouge si pas assez de ressource pour construire le batiment
@@ -213,8 +213,8 @@ 
 		
 		/**
 		 * @param $id_base
-		 * @param null $vitesse = vitesse de l'unité en question
-		 * @return number
+		 * @param integer $vitesse = vitesse de l'unité en question
+		 * @return integer|null
 		 * fonction qui renvoi le temps de trajet entre la base du joueur et une autre base en secondes
 		 */
 		public static function getDureeTrajet($id_base, $vitesse = 1) {
@@ -295,8 +295,8 @@ 
 		}
 
 		/**
-		 * @param $param
-		 * @return mixed
+		 * @param string $param
+		 * @return integer
 		 * fonction qui sert à récupérer un parametre spécifique pour un batiment
 		 * par exemple la vitesse d'un marchand ou  le nombred'emplacment de la base
 		 */

Spacing +4 added lines, -4 removed lines

@@ -197,7 +197,7 @@ 
 		public static function getTestAssezRessourceBase($nom_ressource, $ressource) {
 			$f = "get".ucfirst($nom_ressource);
 
-			if ($ressource >  Bataille::getRessource()->$f()) {
+			if ($ressource > Bataille::getRessource()->$f()) {
 				/*echo("$nom_ressource $ressource ".Bataille::getRessource()->getEau()." ---");*/
 				return [
 					"ressource" => $ressource,
@@ -224,10 +224,10 @@ 
 			
 			//calcul des distances séparant les deux bases en x et y
 			//cette dstance sera multipliée par 15 sur x et y puis ajoutée pour avoir le temps du trajte en seconde
-			$calc_x = abs($base_joueur['posx']-$base_autre['posx']);
-			$calc_y = abs($base_joueur['posy']-$base_autre['posy']);
+			$calc_x = abs($base_joueur['posx'] - $base_autre['posx']);
+			$calc_y = abs($base_joueur['posy'] - $base_autre['posy']);
 			
-			$temps_voyage = (($calc_x*70)+($calc_y*70))/$vitesse;
+			$temps_voyage = (($calc_x * 70) + ($calc_y * 70)) / $vitesse;
 			
 			return $temps_voyage;
 		}

Braces +17 added lines, -9 removed lines

@@ -124,7 +124,9 @@ 
 				->get();
 
 			if ((is_array($query)) && (count($query) == 1)) {
-				foreach ($query as $obj) return $obj->ID_base;
+				foreach ($query as $obj) {
+					return $obj->ID_base;
+				}
 			}
 		}
 
@@ -169,8 +171,7 @@ 
 
 			if ($id_base === null) {
 				$query = $dbc->select()->from("_bataille_last_connexion")->where("ID_identite", "=", self::getIdIdentite())->get();
-			}
-			else {
+			} else {
 				$query = $dbc->select("_bataille_last_connexion.last_connexion")->from("_bataille_base")
 					->from("_bataille_last_connexion")
 					->from("identite")
@@ -272,7 +273,9 @@ 
 				->get();
 
 			if ((is_array($query)) && (count($query) == 1)) {
-				foreach ($query as $obj) return $obj->ID_base;
+				foreach ($query as $obj) {
+					return $obj->ID_base;
+				}
 			}
 
 			return 0;
@@ -288,7 +291,9 @@ 
 			$query = $dbc->select("nombre_joueur")->from("_bataille_nombre_joueur")->where("ID_nombre_joueur", "=", 1)->get();
 
 			if ((is_array($query)) && (count($query) == 1)) {
-				foreach ($query as $obj) return $obj->nombre_joueur;
+				foreach ($query as $obj) {
+					return $obj->nombre_joueur;
+				}
 			}
 
 			return 0;
@@ -306,7 +311,9 @@ 
 			$query = $dbc->select($param)->from("configuration")->where("ID_configuration", "=", 1)->get();
 
 			if ((is_array($query)) && (count($query) == 1)) {
-				foreach ($query as $obj) return $obj->$param;
+				foreach ($query as $obj) {
+					return $obj->$param;
+				}
 			}
 		}
 		//-------------------------- END GETTER ----------------------------------------------------------------------------//
@@ -322,11 +329,12 @@ 
 
 			if ($id_base === null) {
 				$id_identite = self::getIdIdentite();
-			}
-			else {
+			} else {
 				$query = $dbc->select("ID_identite")->from("_bataille_base")->where("ID_base", "=", $id_base)->get();
 
-				foreach ($query as $obj) $id_identite = $obj->ID_identite;
+				foreach ($query as $obj) {
+					$id_identite = $obj->ID_identite;
+				}
 			}
 
 			$dbc->update("last_connexion", date("Y-m-d H:i:s"))

modules/bataille/app/controller/Marche.php

Spacing +10 added lines, -10 removed lines

@@ -82,15 +82,15 @@ 
 			$today = Bataille::getToday();
 
 			//on test si déja arrivé à destination
-			if (($this->aller == 1) && (($this->date_arrivee-$today) <= 0)) {
+			if (($this->aller == 1) && (($this->date_arrivee - $today) <= 0)) {
 				$this->setLivrerRessource();
 
 				//on calcul la date d'arrivée du retour
 				if ($this->id_base_dest == Bataille::getIdBase()) {
-					$date_retour = Bataille::getDureeTrajet($this->id_base, Bataille::getParam("vitesse_marchand"))+$this->date_arrivee;
+					$date_retour = Bataille::getDureeTrajet($this->id_base, Bataille::getParam("vitesse_marchand")) + $this->date_arrivee;
 				}
 				else {
-					$date_retour = Bataille::getDureeTrajet($this->id_base_dest, Bataille::getParam("vitesse_marchand"))+$this->date_arrivee;
+					$date_retour = Bataille::getDureeTrajet($this->id_base_dest, Bataille::getParam("vitesse_marchand")) + $this->date_arrivee;
 				}
 
 				//si le retour du trajet est également arrivé on finit le transport sinon on le place sur le retour
@@ -99,15 +99,15 @@ 
 				}
 				else {
 					$this->setTrajetRetour($date_retour);
-					$this->duree_restante_trajet = $date_retour-$today;
+					$this->duree_restante_trajet = $date_retour - $today;
 					$set_array = true;
 				}
 			}
-			else if (($this->aller == 0) && (($this->date_arrivee-$today) <= 0)) {
+			else if (($this->aller == 0) && (($this->date_arrivee - $today) <= 0)) {
 				$this->setTerminerTransport();
 			}
 			else {
-				$this->duree_restante_trajet = $this->date_arrivee-$today;
+				$this->duree_restante_trajet = $this->date_arrivee - $today;
 				$set_array = true;
 			}
 
@@ -159,11 +159,11 @@ 
 			}
 
 			//on a le nombre de marchand dispo dans la base
-			$nombre_marchand_dispo = $nombre_max_marchand-$marchand_transport;
+			$nombre_marchand_dispo = $nombre_max_marchand - $marchand_transport;
 
 			//on calcul savoir si on en a assez pour transport toutes les ressoures
 			//il faut 1 marchand pour 1000 ressource
-			$nombre_marchand_trajet = ceil($all_ressource/1000);
+			$nombre_marchand_trajet = ceil($all_ressource / 1000);
 
 			//si on a assez de marchand on revoi true sinon false
 			if ($nombre_marchand_dispo >= $nombre_marchand_trajet) {
@@ -244,7 +244,7 @@ 
 				}
 
 				//on check si assez marchand dans la base, si pas assez on return false
-				$nb_marchand = $this->getAssezMarchand($eau+$electricite+$fer+$fuel+$nourriture);
+				$nb_marchand = $this->getAssezMarchand($eau + $electricite + $fer + $fuel + $nourriture);
 
 				if ($nb_marchand === false) {
 					FlashMessage::setFlash("Vous n'avez pas assez de marchans disponibles pour effectuer ce trajet");
@@ -253,7 +253,7 @@ 
 
 				//sinon initialise le transport
 				//on recup la date d'arrivee dans la base de destintation
-				$date_arrivee = Bataille::getDureeTrajet($id_base_dest, Bataille::getParam("vitesse_marchand"))+Bataille::getToday();
+				$date_arrivee = Bataille::getDureeTrajet($id_base_dest, Bataille::getParam("vitesse_marchand")) + Bataille::getToday();
 
 				$ressource = [
 					"eau" => $eau,