EVO Automatisation des models et des views dans le Controller

[rbcreation]

SUGGESTION : Automatiser les includes des models et des views ainsi que leur instanciation respective dans sl_controller et sla_controller.

BUT : Ne plus inclure et instancier à chaque fois les models et les views des modules.

Pour faire cela, il faut :

• Nommer les fichiers de models et de views en commençant par le nom du module.

Exemple avec un module "sl_secure" : models/sl_secure_form.model.php views/default/sl_secure.view.php views/default/sl_secure_form.view.php sl_secure.controller.php

• Modifier les fichiers sl_controller et sla_controller
  • Créer les attributs $models et $views.
  • Ajouter $this->load_models() et $this->load_views() dans le __construct() avant le $this->sl_construct().
  • Créer les méthodes load_models() et load_views().
  • Créer une méthode include_files() que l'on peut placer dans une classe extérieure. Dans mon système, j'ai créé une classe sl_module dans lequel je regroupe les fonctions liées aux modules.

public function load_models() {

$models = sl_module::include_files($this->module["url"]."models/");

foreach ($models as $value) { $model_name = explode(".", $value); $model = $model_name[0]."_model"; $this->models[$model_name[0]] = new $model($this); }

}

public function load_views() {

$views= sl_module::include_files($this->module["url"]."views/".$this->config["template_name"]."/");

foreach ($views as $value) { $view_name = explode(".", $value); $view = $view_name[0]."_view"; $this->views[$view_name[0]] = new $view($this); }

}

public function include_files($dir) { $filenames = array();

if (is_dir($dir)) { if ($dir_handle = opendir($dir)) { while (($file = readdir($dir_handle)) !== false) { if (is_file($dir.$file)) { include ($dir.$file); array_push($filenames, $file); } } closedir($dir_handle); } } return $filenames; }

• APPELER les models et les views dans les modules : Exemple avec un module sl_secure : $this->views["sl_secure"]->show_module(); $this->views["sl_secure_form]->show_form(); $this->models["sl_secure"]->list_users();

Et pour la totale automatisation on remplace par exemple $this->views["sl_secure_form"] par $this->views[$this->module["name"]."_form"]

[wakdev]

J'aime beaucoup la finalité, qui peux même s'écrire par exemple : $this->views["form"]->render(); pour l'affichage du formulaire et $this->views["list"]->render(); pour le listing principal

Par contre :

• Je n'aime pas du tout le fait d'inclure les fichiers de l'ensemble d'une répertoire, c'est hyper dangereux.
• $this->config["template_name"] n'est pas valable, les vues des modules sont indépendant du template.
• Autant, pour les vues pourquoi pas, mais je ne pense pas qu'automatiser l'inclusion des modèles de données soit pertinent.

D'autres avis sur le sujet ?

[rbcreation]

• Je suis d'accord pour l'inclusion de l'ensemble des fichiers d'un répertoire. Ca peut être dangereux. C'est c.. parce que c'est hyper pratique. Après est-ce qu'un pirate peut inclure des fichiers dans ces dossiers ? Quelles-sont les réels danger ? Ya pas moyen de mettre des sécurités ?
• Pour le $this->['template_name'] : dans mon système, le nom de template du site est le même que pour les modules. Par exemple, si dans la base je défini template_v1 pour le site. Dans mes modules, j'aurais dans le dossier : views > template_v1 > module.view.php. Les vues des modules étant adapté au template. Après, je sais pas, c'est peut être pas une bonne idée, mais ça me parait plus cohérent.
• Pour les models, je trouvais ca plus harmonieux d'avoir la même logique que pour les views.

[wakdev]

Je vais réfléchir à un système pratique et sécurisé sur la base de ta réflexion ;-)

[rbcreation]

Cool! J'ai hâte de voir ca. Pour les classes inluses dans le sl_functions, j'ai fait pareil. Un ptit script qui récupère les fichiers et qui les inclus automatiquement.

[wakdev]

Je pense voir pour inclure les classes avec l'autoload si tu connais. C'est pratique, et ca permet de n'inclure que les classes utilisées ;-)

[wakdev]

Tiens, je viens de le mettre en place : https://github.com/wakdev/slash-cms/commit/dfd7c683fde05313aff504ccfa05e44ff2253217

[rbcreation]

De souvenir, je crois que dans le tuto sur la POO dans le site du zero, ils en parlent.

[rbcreation]

C'est bien ce que tu as fait. Ce sera la même manip pour les views et les models ?

[wakdev]

Oui pourquoi pas, je ne me suis pas penché encore sur le problème ^^