bzh a écrit :
C'est pas très clair tout ça. Pourquoi tu ne fais pas un return de ton objet uniquement, tu n'es pas obligé de renvoyer tout le tableau. Note qu'il n y a même pas de return dans le code que tu as soumis...
require_once(ROOT . DIR_PLUGINS . $create_object[$i]['name'] . '/class.php');
Ceci n'a pas l'air très propre, à quoi cela sert-il ?
----------------------
Ok, je viens de comprendre ^^ encore désolée du coup !
C'est au final la seul solution que j'ai trouvé le coup du return.
Mais je voulait voir si il n'y avait pas un moyen plus simple.
La method dynamicObject sert a recuperer dans la BDD la listes des plugins qui sont activer et a créer dynamiquement l'objet.
Explications :
ROOT = au chemin jusqu'a la racine du site : /
DIR_PLUGINS = au chemin de la racine du site jusqu'au dossier plugins : /core/plugins/
Du coup la ligne que tu ne 'comprend' pas :
require_once(ROOT . DIR_PLUGINS . $create_object[$i]['name'] . '/class.php');
On inclut le fichier class.php du plugins user.
${'S_'. strtoupper($create_object[$i]['name'])} = new $create_object[$i]['name']
Cette ligne sert a créer dynamiquement l'objet! au final c'est comme si c'étais équivalent a :
Pour terminer on pourrait traduire les deux lignes suivante pour un plugins user comme cela :
require('/var/www/html/Lab/SimpleFrameworks/public/../core/plugins/user/class.php');
$S_USER = new user();
J'ai modifier la suite pour la retourner sous forme de tableau si il y a plusieurs objet en faisait notamment un return.
----------------------
niuxe a écrit :
+1
Le problème est que tu ne décris pas le tenant et l'aboutissant (une méthode et quelques instances de class). Tu ne commentes pas ton code et il faut déchiffrer. Tu parles d'un fichiers de config.php. Or un tel fichier doit avoir seulement des données statiques. Tu vas retrouver dans ce fichier des constantes comme la connexion à la base, le mode de l'application (dev ou prod), le timezone, le niveau de sécurité, le sel pour le hash, la méthode de hash/encrypt, le mail du serveur, etc. etc.
En générale, dans ce fichier tu n'as pas de création de class. Par contre, ce fichier peut être une class à part entière. Je dirai même une class statique.
À vue de nez, je t'invite à regarder :
- le PSR4 de php (!!)
- la création d'un factory (design pattern) --> bien connaître l'avantage ET l'inconvénient (Si tu commences la poo, oublie !)
Je suis désoler de ne pas commenter mon code :s
il faudrait, je le sais... mais je préfére le faire une fois le projet finis.
Pour ce qui est de PSR4 ect... j'imagine que tu fait référence a
composer ? et l'autoloader qui y est associé ? Je n'aime pas trop
.
Dans la dernières partie de ton message tu parle de
variable dynamique!
C'est exactement cela! Je n'arrivait pas a avoir n'importe quelle objet sur n'importe quelle page.
J'ai finalement réussi en mettant les objet dans un tableau et en le passant a la method render(main_controller.php) qui rend la vue.
J'ai procéder ainsi :
- Dans le fichiers config :
- Création de toutes les class dont on a besoin en les regroupant sous la forme d'un tableau.
Par exemple pour le plugins user nous avons
-> user_model.php
-> user_controller.php
Mon tableau correspond alors :
$S_CLASS['user'][user_model]->exempledemethode();
Je peut aussi appeler une autre classe dont j'ai besoin :
$S_CLASS['plugins']->isActive();
-------------------
Mon fichier config =>
ICI
-------------------
Il est entier, je ferais par la suite un autoload a la place des 6 dernieres lignes.
Du coup a part les 6 dernières lignes tu trouve que cela ne va pas pour un fichier config ?
-------------------
Ma "librairie" qui gérent les plugins =>
ICI
-------------------
D’où ma question, pour la method dynamicObject() je pensais qu'il y avais moyen de récupérer l'objet que sort cette méthod sans avoir a utiliser de return.
Mais visiblement il n'y a pas moyen de faire autrement!
Je vous remercie quand même!
Je tiens a préciser qu'il ne s'agit nullement d'un projet pro.
Je fait de la POO depuis quelques semaines (~2mois).
Avant je faisait surtout du procédural.
Je cherche a faire quelques choses le plus sérieux possible.
J’espère avoir été plus clair. Si ce n'est pas le cas c'est que mon projet est surement mal réfléchis...
Modifié par Eito79 (25 Sep 2017 - 17:21)
