Discussion :

[Monstros Velu]

Bonjour,

j'ai une question sur les factories : J'ai implémenté la mienne ainsi, par réflection :

Code :

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

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
 
 public class WSCallerFactory : Mceau_Tal_3_WSCall.IWSCallerFactory
    {
        IWSParams _iwsParams;
 
        public WSCallerFactory(IWSParams iwsParams)
        {
            _iwsParams = iwsParams; // paramètres d'environnement
        }
 
        IWSCaller IWSCallerFactory.GetInstance(WSCallerName name)
        {
            WSCallerName.Task..ToString()
 
            IWSCaller iwsc = (IWSCaller)Assembly.GetExecutingAssembly().CreateInstance(
                "Mceau_Tal_3_WSCall.WSCallerClass.WSCaller" + name.ToString("G"),
                false, 
                System.Reflection.BindingFlags.CreateInstance,
                null,
                new object[] {_iwsParams},
                null,
                null);
 
            if (iwsc == null)
            {
                throw new ArgumentOutOfRangeException(task);
            }
 
            return iwsc;
        }

Avec l'enum SWCallerName déclaré ainsi :

Code :

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

1
2
3
4
5
6
7
 
        enum WSCallerName {
            Invoices,
            Rapport,
            JobExec,
            Task
        };

Cependant, les exemples de factories sur le net utilisent des switch/cases.

Un collègue me fait remarquer qu'avec un switch case, si on veut instancier un objet d'une classe qui n'existe pas, on a une erreur à la compilation, et en plus, on bénéficie de l'intellisense.

Moi, j'aime bien ce que j'ai fait, parce qu'en cas de création d'une nouvelle classe à instancier par cette factory, il n'y a qu'à l'ajouter dans l'enum, sans toucher à la méthode GetInstance, ce qui applique l' "Open/closed principle" de SOLID.

J'ajoute qu'au début, je passais directement un string name, mais on a transformé ça en enum pour restreindre les entrées possible.

Qu'en pensez-vous ?

[Pol63]

le seul potentiel problème ici c'est que le nom de la classe et les namespaces sont en string
donc en cas de changement de ceux ci il faudra modifier dans le factory
ca reste donc un problème mineur, surtout sur un petit projet

si tu as pas trop de valeur d'enum possible un switch sur l'enum convient tout en gardant l'intellisense et en ayant de la vérification à la compilation
après si tu as beaucoup de classes possibles et que tu ne veux pas modifier le factory alors oui ta solution est bonne (il n'y a que l'enum à modifier)

[popo]

Si tu regardes en détails, il y a un couplage fort entre l’énumération et la classe.
Ce qui implique que cette énumération fait partie de l'implémentation de la fabrique.
Donc tu dois modifier la fabrique en cas d'ajout de nouvelle classe, ce qui est une violation de l'Open/Close dans ce cas là aussi.

Puisque ça ne change rien sur le principe, je suis du même avis que ton collègue.
Avec cette implémentation ton programme compilera même si tu omets de modifier l'énumération.
Tu prend donc le risque de créer un bogue que tu n'aura jamais avec une implémentation classique.

Ce n'est pas pour rien qu'on appelle cela des "patrons de conception".

A propos de SOLID, c'est très bien de vouloir le respecter.
Mais il ne faut pas se montrer rigide et comprendre que parfois pour la maintenabilité, la lisibilité et tout un tas d'autres critères, il sera nécessaire de faire autrement.
Quand à savoir quand et comment, je n'aborderai pas plus le sujet parce que ce post risque de se transformer en débat sans fin.

[Noxen]

À quoi elle te sert cette factory ?