Simplifier un code répétitif

Bonjour,
J'ai un code qui utilise une classe de ma création pour créer des listes d'options.
Ici j'ai 5 listes d'options à créer de façon identique et je ne sais pas comment ne pas répéter 5 fois le même code. J'ai trop de paramètres pour créer une fonction qui serait trop lourde. J'ai donc pensé utiliser des variables de variables mais je ne sais pas comment faire vu le nombre de variables variables.
Le code suivant montre deux listes.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25

$Authors = getParams('lst_authors'); $lstAuthors = new OptListSelect('author'); $lstAuthors->addOption( new OptListOption('') ); $lstAuthors->addOption( new OptListOption('0', '<nouveau>') ); if($Authors) { foreach($Authors as $key=>$value) { $lstAuthors->addOption( new OptListOption( (string) $value['id'], $value['text']) ); } }   $Publishers = getParams('lst_publishers'); $lstPublishers = new OptListSelect('publisher'); $lstPublishers->addOption( new OptListOption('') ); $lstPublishers->addOption( new OptListOption('0', '<nouveau>') ); if($Publishers) {   foreach($Publishers as $value) { $lstPublishers->addOption( new OptListOption( (string) $value['id'], $value['text']) ); }   }

---

Première chose, je retirerais les if($Authors) et if($Publishers) car si $Authors ou $Publishers sont des tableaux vides (mieux vaut les initialiser de cette manière plutôt que de les initaliser à null), les boucles foreach ne tourneront pas de toute façon.

En résumé, tu disposes d'une classe qui crée des selects et d'une autre qui crée les options. Il te manque une classe en surplomb qui se charge de créer les deux: on lui passe les paramètres du select et les paramètres des options (sous forme de tableau par exemple) et elle va créer les objets nécessaires. Il s'agit du design pattern builder. De cette manière tu éviteras les addOption(new OptListOption...) en l'utilisant:

Code:

---

$selectAuthors = new SelectBuilder($select, $options);

---

Ici, $select et $options ne sont pas des instances de tes deux classes, mais sont des tableaux (ou autres structures) de paramètres que la classe SelectBuilder fournira aux constructeurs des classes Select et Option pour créer les objects nécessaires. Le job de SelectBuilder est de demander aux autres classes ce dont elle a besoin. C'est un peu un chef de chantier. Bien entendu, cette classe doit être en mesure au final de renvoyer une instance de la classe Select munie de toutes ces options.

Pour l'instant, j'ai résolu le problème avec une fonction. Par contre la solution que tu préconises semble séduisante et pourrait me servir dans d'autres cas. Par contre, il faut que je l'étudie plus en détail. A voir. Pour l'instant, je mets le sujet comme résolu. Je reviendrai ici si j'ai des difficultés avec la solution que tu préconises.

Il faut spécifier les éléments qui changent d'un <select> à l'autre dans un tableau PHP ($models dans l'exemple).

Ensuite tu boucles sur ton tableau.

Avec ton exemple de 2 <select> cela donne :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30

$models = [ [ 'params' => 'lst_authors', 'select_name' => 'author', ], [ 'params' => 'lst_publishers', 'select_name' => 'publisher', ], // ... ];   $selects = []; // Tableau de résultat // Les <select> seront accessibles selon le select_name paramétré dans le modèle   // Construction dynamique du tablau de résultat // à partir de $models foreach ($models as $model) { $params = getParams($model['params']) ?? []; // À charge de getParams() de retourner un tableau vide $select = new OptListSelect($model['select_name']); $select->addOption(new OptListOption('')); $select->addOption(new OptListOption('0', '<Nouveau>')); // Caractères spéciaux à échapper à l'affichage foreach ($params as $value) { $select->addOption(new OptListOption((string)$value['id'], $value['text'])); } $selects[$model['select_name']] = $select; // Ajout au tableau de résultat }   print_r($selects);

---