Design Patern Politique et types.

Bonjour,

J'ai une classe template qui écoute un port et qui transmet les informations à traiter.

J'utilise alors un design pattern politique pour grossièrement faire :

Code:

---

1 2 3 4 5

template <typename Mode> class Listener { Listener(Type cible); };

---

En fonction de Mode, le mode de transmission des information sera différent.
Dans le premier cas j'ai besoin d'une liste d'instruction (ListeInstruction pour Mode = Liste) et dans le second j'ai besoin d'une classe de traitement d'instruction (Traitement pour Mode = Param).

J'aimerais donc que la classe passée en paramètre template (Mode) détermine aussi le type "Type".

Le moyen le plus simple que je verrais serait de faire :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

template <typename Mode> class Listener { Listener(Mode::Type cible); };   class Liste { public : class Type : ListeInstruction { }; } Listener<Liste>( liste );

---

Mais je me demande s'il n'existe pas une meilleurs solution.
Le cas idéal serait de faire un typedef dans les classes Mode (donc dans Liste et Param) qu'on puisse réutiliser dans le constructeur de Listener mais je ne pense pas que ce soit possible...

Qu'en pensez-vous?

pourquoi ce ne serait pas possible ?
ta sous classe Type n'a pas besoin d'etre une classe, ca peut etre une typedef.

petit edit:
"le plus simple" -> phrase intéressante. relativise l'utilité de ce pattern ici, si tu n'as que deux politiques "le plus simple" reste un 'if / else'.

Citation:

---

Envoyé par Lightness1024

pourquoi ce ne serait pas possible ?
ta sous classe Type n'a pas besoin d'etre une classe, ca peut etre une typedef.

---

J'y avais pensé, mais après avoir testé j'ai remarqué que d'utiliser des sous-classes ou des typedef, on a le même message d'erreur :

Citation:

---

Mode::Cible is not a type

---

Citation:

---

Envoyé par Lightness1024

petit edit:
"le plus simple" -> phrase intéressante. relativise l'utilité de ce pattern ici, si tu n'as que deux politiques "le plus simple" reste un 'if / else'.

---

L'utilisation d'un tel partern a été bien réfléchit. C'est la solution la plus optimisée et la plus pratique pour cette utilisation.

De plus, Listener a une méthode qui tourne en boucle, une bonne quinzaines de if/else ou d'appel de méthodes en moins n'est donc pas si négligeable.

Autre problème avec les if_else, c'est que si l'on ajoute un troisième mode, il faut modifier du code existant (le code la classe Listner), ce qui contredit le principe ouvert-fermé

Perso, dans cette situation, je pars très rapidement sur des classes de traits/polices + spécialisation (en fait, c'est même le premier design auquel je pense par habitude)

Code:

---

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

template <typename T> struct Listener_traits;   struct Mode1; template <> struct Listener_traits<Mode1> { typedef ListeInstruction Type; typedef Mode1 Mode; };   struct Mode2; template <> struct Listener_traits<Mode2> { typedef ParamInstruction Type; typedef Mode1 Mode; };   template <typename Mode> class Listener { Listener(typename Listener_traits<Mode>::Type cible); };   Listener<Mode1> listener1; Listener<Mode2> listener2;

---

Ce code permet d'ajouter des spécialisations sans modifier le code existant et permet beaucoup de souplesse (vive le principe "tous les problèmes peuvent être réglés par l'ajout d'une indirection")

EDIT :

Citation:

---

Envoyé par Neckara

J'y avais pensé, mais après avoir testé j'ai remarqué que d'utiliser des sous-classes ou des typedef, on a le même message d'erreur :

---

hum, tu as oublié le typename : http://www.developpez.net/forums/d12...5-cle-typename

EDIT2 : pour rappel, la FAQ (Qu'est-ce qu'une classe de trait ? Qu'est-ce qu'une classe de politique ?) et le tutoriel de Alp : Présentation des classes de Traits et de Politiques en C++

Une alternative est la factory, mais cela provoque une variabilité à l'exécution plutot qu'à la compilation.

J'aurais plutôt dit un DP stratégie (qui est l'équivalent au runtime des classes de police finalement)

Citation:

---

hum, tu as oublié le typename : http://www.developpez.net/forums/d12...5-cle-typename

---

C'est exactement ceci qui me manquait, j'avais vite survolé le sujet il y a peu de temps en plus :aie:

J'ai donc mis le typedef dans le Mode et ton fonctionne parfaitement, merci beaucoup.