Discussion :

[r0d]

bonjour à tous,

j'ai un problème tout bête:
J'ai une fonction toto(UINT uiType). Dans cette fonction, je voudrais créer une instance MonObjet dont la classe sera en fonction de mon paramètre uiType.
voici mon code, vous comprendrez direc:

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
fonction toto(UINT uiType)
{
swicth(uiType)
   {
   case 1:
      MaClasse1 MonObjet();
      ...
      break;
 
   case 2:
      MaClasse2 MonObjet();
      ...
      break;
 
   // etc.
   }
}

Evidemment, je ne peux pas faire ça, puisque il est interdit de faire une déclaration dans un switch.
Mes classes MaClasse1 et MaClasse2 héritent d'une classe MaClasse.

Comment faire?

P.S: en réalité, mon switch porte sur une vingtaine de valeurs.

[Azharis]

J'ai poster un problème assez similaire sauf que c'était la classe qui changé. Je le faisait avec des pointeurs, dans switch je faisait avec new mais ca marche moyen, c'est peut être du fait qu'on ne peut pas déclarer dans switch.

[Laurent Gomila]

Le moyen de contourner ça est d'englober tes 'case' comportant des déclarations, dans des blocs d'accolades.

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
fonction toto(UINT uiType)
{
swicth(uiType)
{
   case 1:
   {
      MaClasse1 MonObjet();
      ...
      break;
   }
 
   case 2:
   {
      MaClasse2 MonObjet();
      ...
      break;
   }
 
   // etc.
}
}

Cette fonction fait quoi exactement ? Ca me paraît bizarre, surtout que toutes ces classes dérivent d'une même base.

PS : attention avec les parenthèses -- là tu n'instancies pas tes classes, tu déclares à chaque fois des fonctions. Je ne pense pas que ce soit le but recherché.

[r0d]

Cette fonction fait quoi exactement ? Ca me paraît bizarre, surtout que toutes ces classes dérivent d'une même base.

Oui, moi aussi ça me parait bizzare d'ête obligé de faire ça. Je dois faire une erreur de conception.
En fait, je dois créer une carte. Je récupère des données sur une bdd. Pour chacune de ces données, j'appelle cette fonction toto(). En vérité, cette fonction est:

Code :

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

1
2
 
createPoint(UINT x, UINT y, UINT type)

et je dois créer un "Point" (un point sur une carte), qui peut être de type Entrepot, Agence, Magasin, etc... yen a une vingtaine. J'ai donc une classe Point, et plein d'autres qui en héritent.

PS : attention avec les parenthèses -- là tu n'instancies pas tes classes, tu déclares à chaque fois des fonctions. Je ne pense pas que ce soit le but recherché.

Là-dessus aussi j'ai un soucis de conception. En vérité, ma fonction createPoint est une méthode d'une classe Map. Dans cette classe Map, j'ai une liste de Point. Je voudrais donc que mes objets de type Point (ou classe qui en hérite) soient alloués dans ma classe Map. Il faut donc que je fasse un new c'est ça?

J'espère que je suis clair

[Laurent Gomila]

Oui, tu devrais plutôt avoir un truc de ce style :

Code :

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

1
2
3
4
5
6
7
8
9
10
void CMap::createPoint(UINT x, UINT y, UINT type)
{
    switch (type)
    {
        case TILE_XXX : m_Tiles[x][y] = new TileXXX(...); break;
        case TILE_YYY : m_Tiles[x][y] = new TileYYY(...); break;
        case TILE_ZZZ : m_Tiles[x][y] = new TileZZZ(...); break;
        // ...
    }
}

Mais comme dit précédemment pour Azharis, ce n'est pas le meilleur design et il vaudra mieux passer par une fabrique.

[r0d]

ok. Aurais-tu un bon lien sur le pattern factory stp?

[Laurent Gomila]

Là comme ça je n'ai rien sous la main, mais je suis certain que Google en as des tas

[r0d]

ok