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 | private function setCreerFichier($new_page) { |
||
| 131 | $page_type = ROOT."config/page_type/page_type.html"; |
||
| 132 | |||
| 133 | if ((!file_exists($new_page)) && (copy($page_type, $new_page))) { |
||
| 134 | return true; |
||
| 135 | } |
||
| 141 | |||
| 142 | /** |
||
| 143 | * fonction qui permet de créer un page |
||
| 144 | * @param $balise_title |
||
| 145 | * @param $url |
||
| 146 | * @param $meta_description |
||
| 147 | * @param $titre_page |
||
| 148 | * @param $parent |
||
| 149 | */ |
||
| 150 | public function setCreerPage($balise_title, $url, $meta_description, $titre_page, $parent, $affiche = 1) { |
||
| 182 | |||
| 183 | /** |
||
| 184 | * function that will create a redirection on an other site |
||
| 185 | * @param $balise_title |
||
| 186 | * @param $url |
||
| 187 | * @param $titre_page |
||
| 188 | * @param $parent |
||
| 189 | */ |
||
| 190 | public function setCreerPageRedirect($balise_title, $url, $titre_page, $parent, $affiche = 1) { |
||
| 214 | |||
| 215 | /** |
||
| 216 | * fonction qui permet de modifier une page en fonction de son id |
||
| 217 | * @param $id_page |
||
| 218 | * @param $balise_title |
||
| 219 | * @param $url |
||
| 220 | * @param $meta_description |
||
| 221 | * @param $titre_page |
||
| 222 | * @param $parent |
||
| 223 | * @param $contenu |
||
| 224 | */ |
||
| 225 | public function setModifierPage($id_page, $balise_title, $url, $meta_description, $titre_page, $parent, $affiche = 1) { |
||
| 261 | |||
| 262 | /** |
||
| 263 | * @param $id_page |
||
| 264 | * @param $contenu |
||
| 265 | */ |
||
| 266 | public function setModifierContenu($id_page, $contenu) { |
||
| 271 | |||
| 272 | /** |
||
| 273 | * fonction qui permet de supprimer une page, test si fichier exist, si oui on delete |
||
| 274 | * @param $id_page |
||
| 275 | */ |
||
| 276 | public function setSupprimerPage() { |
||
| 295 | |||
| 296 | /** |
||
| 297 | * @param string $id |
||
| 298 | * @param string $value_id |
||
| 299 | * @param integer $affiche |
||
| 300 | */ |
||
| 301 | private function setAjoutLienNavigation($id, $value_id, $affiche) { |
||
| 306 | |||
| 307 | /** |
||
| 308 | * @param string $id |
||
| 309 | * @param integer $affiche |
||
| 310 | */ |
||
| 311 | private function setModifierLienNavigation($id, $id_page, $parent, $affiche) { |
||
| 319 | |||
| 320 | /** |
||
| 321 | * delete link in nav and delete page in table |
||
| 322 | */ |
||
| 323 | private function setSupprimerLienNavigation() { |
||
| 330 | //-------------------------- FIN SETTER ----------------------------------------------------------------------------// |
||
| 331 | } |
