Piou-piou /
Ruined-World
Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
Complex classes like Unite often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use Unite, and based on these observations, apply Extract Interface, too.
| 1 | <?php |
||
| 10 | class Unite { |
||
| 11 | private $coef_unite; |
||
| 12 | private $pour_recruter; |
||
| 13 | private $temps_recrutement; |
||
| 14 | private $ameliorations; |
||
| 15 | |||
| 16 | |||
| 17 | //-------------------------- BUILDER ----------------------------------------------------------------------------// |
||
| 18 | public function __construct() { |
||
| 21 | //-------------------------- END BUILDER ----------------------------------------------------------------------------// |
||
| 22 | |||
| 23 | |||
| 24 | //-------------------------- GETTER ----------------------------------------------------------------------------// |
||
| 25 | /** |
||
| 26 | * @param $caracteristique |
||
| 27 | * @param $force |
||
| 28 | * @param $niveau |
||
| 29 | * @return float |
||
| 30 | * fonction qui permet de renvoyer la puissance d'une unité dans un élément spécifique |
||
| 31 | */ |
||
| 32 | private function getAmeliorationUnite($caracteristique, $force, $niveau) { |
||
| 42 | |||
| 43 | |||
| 44 | /** |
||
| 45 | * @param $unite |
||
| 46 | * @param $niveau |
||
| 47 | * @param $type |
||
| 48 | * @return array |
||
| 49 | * récupère les caractéristiques de l'unité en fonction de son niveau |
||
| 50 | */ |
||
| 51 | public function getCaracteristiqueUnite($unite, $niveau, $type) { |
||
| 93 | |||
| 94 | /** |
||
| 95 | * @return array |
||
| 96 | * fonction qui renvoit tous les types d'unités qu'il est possible de recruter |
||
| 97 | */ |
||
| 98 | private function getAllType() { |
||
| 101 | |||
| 102 | /** |
||
| 103 | * @param $type |
||
| 104 | * fonction qui permet de récupérer les unités qu'i est possible de recruter en fonction |
||
| 105 | * du type (batiment sur lequel on a cliqué) |
||
| 106 | */ |
||
| 107 | public function getUnitePossibleRecruter($type) { |
||
| 119 | |||
| 120 | /** |
||
| 121 | * fonction qui renvoi les unité en cours de recrutement |
||
| 122 | */ |
||
| 123 | public function getRecrutement() { |
||
| 149 | |||
| 150 | /** |
||
| 151 | * @param null $id_base |
||
| 152 | * @param null $id_groupe |
||
| 153 | * fonction qui récupère toutes les unités qui sont dans la base |
||
| 154 | */ |
||
| 155 | public function getAllUnites($id_base = null, $id_groupe = null) { |
||
| 179 | |||
| 180 | /** |
||
| 181 | * @param $type |
||
| 182 | * @param $id_base |
||
| 183 | * @param null $id_groupe |
||
| 184 | * @return array |
||
| 185 | * fonction qui récupère toutes les unités en fonction d'un type précis |
||
| 186 | */ |
||
| 187 | private function getAllUniteType($type, $id_base, $id_groupe = null) { |
||
| 223 | |||
| 224 | /** |
||
| 225 | * @param $type |
||
| 226 | * @param $nom |
||
| 227 | * @return int |
||
| 228 | * renvoi le nombre d'unite en fonction d'un type et d'un nom qui ne sont ni dans un groupe ni |
||
| 229 | * en mission |
||
| 230 | */ |
||
| 231 | protected function getNombreUniteNom($type, $nom) { |
||
| 245 | |||
| 246 | /** |
||
| 247 | * @return int |
||
| 248 | * fonction qui renvoi le nombre d'unité vivante dans la base qui consomme de la nourriture |
||
| 249 | */ |
||
| 250 | View Code Duplication | public function getNombreUniteHumaine() { |
|
| 260 | |||
| 261 | /** |
||
| 262 | * @param $id_mission |
||
| 263 | * @return int |
||
| 264 | * fonction qui renvoi le nombre d'unités envoyées sur une mission en particulier |
||
| 265 | */ |
||
| 266 | View Code Duplication | public function getUnitesMission($id_mission) { |
|
| 276 | |||
| 277 | /** |
||
| 278 | * @param $type |
||
| 279 | * @param $nom |
||
| 280 | * récupération du temmp de recrutement + les ressources nécéssaires |
||
| 281 | */ |
||
| 282 | private function getInfosRecrutementUnite($type, $nom) { |
||
| 299 | //-------------------------- END GETTER ----------------------------------------------------------------------------// |
||
| 300 | |||
| 301 | |||
| 302 | //-------------------------- SETTER ----------------------------------------------------------------------------// |
||
| 303 | /** |
||
| 304 | * @param $nom -> nom de l'unité à recruter |
||
| 305 | * @param $type -> type de l'unité à recruter |
||
| 306 | * @param $nombre -> nombre d'unité à recruter |
||
| 307 | * fonction qui permet d'initialiser le début du recrutement d'unités |
||
| 308 | */ |
||
| 309 | public function setCommencerRecruter($nom, $type, $nombre) { |
||
| 347 | |||
| 348 | /** |
||
| 349 | * @param $id_recrutement |
||
| 350 | * fonction appellée dans celle qui récupère les recrutement uniquement quand celui ci est finit |
||
| 351 | * fonction qui sert à terminer un rcrutement et ajouter les unités dans la base |
||
| 352 | */ |
||
| 353 | private function setTerminerRecrutement($id_recrutement) { |
||
| 376 | |||
| 377 | /** |
||
| 378 | * @param $nombre_unite |
||
| 379 | * @param $nom_unite |
||
| 380 | * @param $type_unite |
||
| 381 | * @param $id_mission |
||
| 382 | * @return bool |
||
| 383 | * permet de lancer des unites en expédition en ajoutant à chaque unité un id_mission |
||
| 384 | */ |
||
| 385 | public function setCommencerExpedition($nombre_unite, $nom_unite, $type_unite, $id_mission) { |
||
| 407 | |||
| 408 | /** |
||
| 409 | * @param $id_mission |
||
| 410 | * @param $pourcentage_perte |
||
| 411 | * @return int |
||
| 412 | * fonction qui termine une expdedition au niveau des troupes, cette fonction s'occupe d'en |
||
| 413 | * supprimer de la bdd en fonction du nombre de troupe envoyé et du cpourcentage de perte |
||
| 414 | */ |
||
| 415 | public function setTerminerExpedition($id_mission, $pourcentage_perte) { |
||
| 448 | |||
| 449 | /** |
||
| 450 | * @param $nombre |
||
| 451 | * fonction qui permet de tuer des unites |
||
| 452 | */ |
||
| 453 | public function setTuerUnites($nombre) { |
||
| 466 | //-------------------------- END SETTER ----------------------------------------------------------------------------// |
||
| 467 | } |
Loading history...
If you define a variable conditionally, it can happen that it is not defined for all execution paths.
Let’s take a look at an example:
In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.
Available Fixes
Check for existence of the variable explicitly:
Define a default value for the variable:
Add a value for the missing path: