Discussion :

[Nalfeim]

Bonjour

Je me trouve actuellement devant une petite problématique de conception qui est je pense assez courante. Bien que je puisse facilement bricoler quelquechose, j'aimerai avoir des avis éclairés sur la question, histoire de pas avoir à reprendre ça dans un mois ou deux

Le portail que je développe a un point d'entrée unique. Les liens ou les formulaires passent donc en paramètre un identifiant d'action, plus d'éventuels informations complémentaires.
Suivant du mieux possible le modèle MVC, j'ai donc un ActionsControler qui dispatche ces requetes en fonction de l'identifiant d'action, que je définis comme étant le hachage md5 de deux trois mots explicites comme "ConfigurationList" ou "ServerDetailView".

Techiquement, j'ai commencé par un gros if / elseif / ... Je trouve cela un peu moche car très répétitif, surtout que mes actions sont assez standard (lister, voir le détail, éditer, ... des environnements / serveurs / configurations / ...).
Du coup je me dis qu'il doit y avoir moyen de faire ça intelligemment, comme une table associative aActionMap[cible][action] = md5, ou un truc dans ce gout qui inclue les rangs utilisateurs (qui a accès à quoi) voir la disponibilité (en travaux / pas encore codé / ...)

Voilà où j'en suis. Merci d'avance pour tout conseil, toutes pistes de recherche voir le moindre exemple !

[Yoshio]

Du coup je me dis qu'il doit y avoir moyen de faire ça intelligemment,

Prendre un vrai framework tel que le Zend Framework ou symfony me semble la meilleur solution vu que le MVC est d'office d'application.

[Nalfeim]

Je sais. Mais ça n'a pas été commencé comme ça, et maintenant je dois faire avec ce qu'il y a, car l'horloge tourne.

[mistertbo]

Et pourquoi pas un switch case ? Ca sera déjà plus propre qu'un if/then/else, non ?

Edit. Autre idée qui me vient à l'esprit... avoir une classe par cible, avec des méthodes nommées selon les actions possible par cible... ainsi tu pourrais appeler tes actions dynamiquement selon les variables get, et gérer les erreurs si la classe/méthode n'existe pas.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
$cible = $_GET['cible'];
$action = $_GET['action'];
 
try {
    $cObj = new $cible();
    $cObj->$action($args);
}
catch (Exception $e)
{
    // ....
}

Qu'en penses-tu ?