Discussion :

[bolhrak]

Bonjour,

Une petite question de conception. J'ai une classe DocumentManager qui peut être paramétrée par une classe de Stockage :

Code :

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

1
2
3
4
5
6
 
 
template <class DocumentHandler, template<class> StoragePolicy>
class DocumentManager : public StoragePolicy<DocumentHandler> {
//...
};

Quelle que soit la classe de stockage, elle aura besoin d'accéder à une donnée membre (current_document) et deux méthodes.

Ma question est vaut-il mieux créer une classe supplémentaire dans laquelle je vais mettre ces trois membres (2 fonctions + donnée) et en faire hériter chacune de mes classes de stockage, ou bien les inclure dans la classe DocumentManager et utiliser le CRTP :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
template <class Derived>
class DefaultStorage {
//...
};
 
template <class DocumentHandler, template <class> StoragePolicy>
class DocumentManager : public StoragePolicy<DocumentManager>
{
//...
};

Merci d'avance.

[Alp]

Pour ma part je pencherais pour le CRTP.

Cette conception factorise le code sans le rendre moins souple. (ou alors je n'ai pas bien compris l'énoncé de ta question)

[bolhrak]

Disons que dans tous les cas le code ne se trouvera qu'à un seul endroit, la classe de DocumentManager dans le cas du CRTP, la classe de base dont héritent toutes mes classes de stockage dans l'autre cas. Après c'est plus une question de lisibilité de code et de contraintes imposées par rapport au design des classes de stockage :

- dans le cas où je n'utilise pas le CRTP, j'impose aux designers des classes de stockage de dériver de la classe de base.
- l'enrichissement de ma classe DocumentManager passe par deux niveaux si je n'utilise pas le CRTP : les classes de stockage (inévitable quelle que soit la solution que je choisis) et la classe de base.

Disons que dans les deux cas il existe des contraintes sur le code des classes de stockage, c'est juste une manière différente de les exprimer. Dans le cas où je n'utilise pas le CRTP ces contraintes font partie d'une classe de base, mais qui enrichira la class DocumentManager, donc les utilisateurs de cette classe devront connaître toute la hiérarchie, dans le cas où j'utilise le CRTP les contratintes sont exprimées dans la classe dérivé (autrement dit DocumentManager) et les utilisateurs ne doivent se soucier que de l'enrichissement dû à la classe de stockage utilisée.

Reformulé de manière plus général, la question serait : faut-il utiliser le CRTP uniquement lorsque l'on a pas le choix, ou bien peut-on l'utiliser lorsque cela est plus pratique ?

[koala01]

Salut,

Reformulée ainsi, le principe général est d'utiliser ce qui sera le plus facile à utiliser pour les évolution futures et le plus simple à maintenir.

Si le but est, comme il le semble, de créer une bibliothèque que d'autres utiliseront, il me semble de bon ton de veiller à ce que ta bibliothèque reste le plus simple et le plus pratique possible à utiliser, quitte à perdre un peu de simplicité de ton côté

Quoi qu'il en soit, la question aurait sans doute dû être débattue dés l'analyse de base, et la solution décidée (envisagée) à ce moment là doit être respectée.

S'il s'agit c'est une question "subsidiaire" (AKA d'une question qui n'était pas apparue directement lors de l'analyse de base), il s'agirait normalement de reprendre l'analyse en vue d'y répondre afin de s'assurer que la réponse reste cohérente avec le reste (et les cas d'utilisation, entre autres)

[bolhrak]

Salut,

Reformulée ainsi, le principe général est d'utiliser ce qui sera le plus facile à utiliser pour les évolution futures et le plus simple à maintenir.

Si le but est, comme il le semble, de créer une bibliothèque que d'autres utiliseront, il me semble de bon ton de veiller à ce que ta bibliothèque reste le plus simple et le plus pratique possible à utiliser, quitte à perdre un peu de simplicité de ton côté

Dans ce cas le CRTP semble plus approprié, tant pour la doc (deux niveaux de classe au lieu de trois pour les méthodes offertes par DocumentManager) que pour la possibilité d'écrire d'autres classes de stockage.

Merci beaucoup pour vos réponses

[cynique]

Salut,

C'est bizarre, en le cas de CRTP, d'avoir une StoragePolicy paramétrée par DocumentManager. La StoragePolicy dit comme on stocke son paramétre, et le paramétre est une DocumentManager. Je crois que la DocumentManager gestionne les Documents (ou, peut-etre, les DocumentHandlers), et elle a besoin d'une méthode de les stocker. La StoragePolicy est cette méthode.