Piou-piou /
ribs-framework
Complex classes like GestionContenus 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 GestionContenus, and based on these observations, apply Extract Interface, too.
| 1 | <?php |
||
| 9 | class GestionContenus extends Contenus { |
||
| 10 | use ParentTexte; |
||
| 11 | |||
| 12 | private $erreur; |
||
| 13 | |||
| 14 | |||
| 15 | //-------------------------- GETTER ----------------------------------------------------------------------------// |
||
| 16 | public function getErreur() { |
||
| 19 | |||
| 20 | /** |
||
| 21 | * @param integer $parent |
||
| 22 | * @return int |
||
| 23 | */ |
||
| 24 | private function getOrdrePage($parent) { |
||
| 25 | if ($parent != "") { |
||
| 26 | $dbc = \core\App::getDb(); |
||
| 27 | $ordre = 1; |
||
| 28 | |||
| 29 | $query = $dbc->select("ordre")->from("page")->orderBy("ordre", "DESC")->limit(0, 1)->get(); |
||
| 30 | if (count($query) > 0) { |
||
| 31 | foreach ($query as $obj) { |
||
| 32 | $ordre = $obj->ordre; |
||
| 33 | } |
||
| 34 | } |
||
| 35 | |||
| 36 | return $ordre; |
||
| 37 | } |
||
| 38 | } |
||
| 39 | |||
| 40 | private function getParentId($parent) { |
||
| 41 | $dbc = \core\App::getDb(); |
||
| 42 | |||
| 43 | if ($parent == "") { |
||
| 44 | return 0; |
||
| 45 | } |
||
| 46 | |||
| 47 | $query = $dbc->select("ID_page")->from("page")->where("titre", " LIKE ", '"%'.$parent.'%"', "", true)->get(); |
||
| 48 | |||
| 49 | if (count($query) == 1) { |
||
| 50 | foreach ($query as $obj) { |
||
| 51 | return $obj->ID_page; |
||
| 52 | } |
||
| 53 | } |
||
| 54 | |||
| 55 | return 0; |
||
| 56 | } |
||
| 57 | |||
| 58 | /** |
||
| 59 | * @param string $nom_table |
||
| 60 | * @param string $nom_id_table |
||
| 61 | * @param string $champ |
||
| 62 | * @param $value |
||
| 63 | * @param integer $limit_char |
||
| 64 | * @param string $err_char |
||
| 65 | * @param string $err_egalite |
||
| 66 | * @param null $value_id_table |
||
| 67 | * @return string |
||
| 68 | * fonction qui permet de vérifier qu'il n'y ait pas d'erreur dans le champ spécifié ni de doublons |
||
| 69 | */ |
||
| 70 | private function getVerifChamp($nom_table, $nom_id_table, $champ, $value, $limit_char, $err_char, $err_egalite, $value_id_table = null) { |
||
| 82 | |||
| 83 | private function getTestBaliseTitle($balise_title, $id_page = null) { |
||
| 84 | $err_balise_title_char = "Le titre pour le navigateur ne doit pas dépasser 70 caractères"; |
||
| 85 | $err_balise_title_egalite = "Ce titre est déjà présent en base de données, merci d'en choisir un autre pour optimiser le référencement de votre site"; |
||
| 86 | return $this->getVerifChamp("page", "ID_page", "balise_title", $balise_title, 70, $err_balise_title_char, $err_balise_title_egalite, $id_page); |
||
| 87 | } |
||
| 88 | |||
| 89 | private function getTestUrl($url, $id_page = null) { |
||
| 90 | $err_url_char = "L'url ne doit pas dépasser 92 caractères"; |
||
| 91 | $err_url_egalite = "Cette url est déjà présent en base de données, merci d'en choisir une autre pour ne pas avoir de conflit entre vos pages"; |
||
| 92 | return $this->getVerifChamp("page", "ID_page", "url", $url, 92, $err_url_char, $err_url_egalite, $id_page); |
||
| 93 | } |
||
| 94 | |||
| 95 | private function getTestMetaDescription($meta_description, $id_page = null) { |
||
| 96 | $err_meta_description_char = "La description de cette page ne doit pas dépasser 158 caractères"; |
||
| 97 | $err_meta_description_egalite = "Cette description est déjà présent en base de données, merci d'en choisir une autre pour optimiser le référencement de votre site"; |
||
| 98 | return $this->getVerifChamp("page", "ID_page", "meta_description", $meta_description, 158, $err_meta_description_char, $err_meta_description_egalite, $id_page); |
||
| 99 | } |
||
| 100 | |||
| 101 | private function getTestTitrePage($titre_page, $id_page = null) { |
||
| 102 | $err_titre_page_char = "Le titre de cette page ne doit pas dépasser 50 caractères"; |
||
| 103 | $err_titre_page_egalite = "Cette titre de page est déjà présent en base de données, merci d'en choisir un autre pour ne pas avoir de conflit dans votre navigation"; |
||
| 104 | return $this->getVerifChamp("page", "ID_page", "titre", $titre_page, 50, $err_titre_page_char, $err_titre_page_egalite, $id_page); |
||
| 105 | } |
||
| 106 | //-------------------------- FIN GETTER ----------------------------------------------------------------------------// |
||
| 107 | |||
| 108 | |||
| 109 | |||
| 110 | //-------------------------- SETTER ----------------------------------------------------------------------------// |
||
| 111 | /** |
||
| 112 | * @param string $url |
||
| 113 | * @param string $err_balise_title |
||
| 114 | * @param string $err_url |
||
| 115 | * @param string $err_meta_description |
||
| 116 | * @param string $err_titre_page |
||
| 117 | */ |
||
| 118 | private function setErreurContenus($balise_title, $url, $meta_description, $titre_page, $parent, $err_balise_title, $err_url, $err_meta_description, $err_titre_page) { |
||
| 129 | |||
| 130 | /** |
||
| 131 | * fonction qui permet de créer un page |
||
| 132 | * @param $balise_title |
||
| 133 | * @param $url |
||
| 134 | * @param $meta_description |
||
| 135 | * @param $titre_page |
||
| 136 | * @param $parent |
||
| 137 | */ |
||
| 138 | public function setCreerPage($balise_title, $url, $meta_description, $titre_page, $parent, $affiche = 1) { |
||
| 183 | |||
| 184 | /** |
||
| 185 | * function that will create a redirection on an other site |
||
| 186 | * @param $balise_title |
||
| 187 | * @param $url |
||
| 188 | * @param $titre_page |
||
| 189 | * @param $parent |
||
| 190 | */ |
||
| 191 | public function setCreerPageRedirect($balise_title, $url, $titre_page, $parent, $affiche = 1) { |
||
| 222 | |||
| 223 | /** |
||
| 224 | * fonction qui permet de modifier une page en fonction de son id |
||
| 225 | * @param $id_page |
||
| 226 | * @param $balise_title |
||
| 227 | * @param $url |
||
| 228 | * @param $meta_description |
||
| 229 | * @param $titre_page |
||
| 230 | * @param $parent |
||
| 231 | * @param $contenu |
||
| 232 | */ |
||
| 233 | public function setModifierPage($id_page, $balise_title, $url, $meta_description, $titre_page, $parent, $affiche = 1) { |
||
| 280 | |||
| 281 | /** |
||
| 282 | * @param $id_page |
||
| 283 | * @param $contenu |
||
| 284 | */ |
||
| 285 | public function setModifierContenu($id_page, $contenu) { |
||
| 290 | |||
| 291 | /** |
||
| 292 | * fonction qui permet de supprimer une page, test si fichier exist, si oui on delete |
||
| 293 | * @param $id_page |
||
| 294 | */ |
||
| 295 | public function setSupprimerPage() { |
||
| 314 | |||
| 315 | /** |
||
| 316 | * @param string $id |
||
| 317 | * @param string $value_id |
||
| 318 | * @param integer $affiche |
||
| 319 | */ |
||
| 320 | private function setAjoutLienNavigation($id, $value_id, $affiche) { |
||
| 325 | |||
| 326 | /** |
||
| 327 | * @param string $id |
||
| 328 | * @param integer $affiche |
||
| 329 | */ |
||
| 330 | private function setModifierLienNavigation($id, $id_page, $parent, $affiche) { |
||
| 338 | |||
| 339 | /** |
||
| 340 | * delete link in nav and delete page in table |
||
| 341 | */ |
||
| 342 | private function setSupprimerLienNavigation() { |
||
| 349 | //-------------------------- FIN SETTER ----------------------------------------------------------------------------// |
||
| 350 | } |
