Discussion :

[zenux]

Bonjour,

Je ne sais pas trop si je doit mettre ceci dans le forum algorithme ou C++.

Pour commencer, voici mon code :

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
#include <iostream>
#include <vector>
using namespace std;
 
class model
{
	public:
		void show()
		{
			cout<<"show"<<endl;
			//...
		}
 
		void move()
		{
			cout<<"move"<<endl;
			//...
		}
 
		//et plein d'autres méthodes
};
 
class A
{
	public:
		A()
		{
			vModel.push_back(new model());
			vModel.push_back(new model());
			vModel.push_back(new model());
		}
 
		void make()
		{
			for(int i=0;i<vModel.size();i++)
			{
				if(true) //condition qui vérifie si l'on doit faire des opération sur le modèle
					vModel[i]->show();
			}
		}
 
	private:
		vector<model *> vModel;
};
 
int main()
{
	A a;
	a.make();
 
	return 0;
}

Explication :
--------------
J'ai une classe "model" qui contient plein de méthode.
J'ai une classe "A" qui contient un vecteur de "model". Cette classe permet de faire certaines opérations sur les model si ils suivent certaines conditions ("if(true)" dans mon exemple).

Mon problème, c'est si je veux faire une opération autre que "show", je vais devoir passer un paramètre à make pour lui dire quel opération faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
void make(int op)
{
	for(int i=0;i<vModel.size();i++)
	{
		if(true)
		{
			if(op==0)
				vModel[i]->show();
			else if(op==1)
				vModel[i]->move();
			//etc
		}
	}
}

Bref ça va me faire un code rempli de "if" et "else if"...ce qui ne fera pas très beau. Et si les méthodes de model prennent différents paramètres, ça compliquera tout.
Une autre solution serait de faire autant de méthode dans la classe "A" qu'il y a de méthode dans la classe "model" mais ça ne me semble pas encore être une solution très propre.

Y a -il une solution plus correcte ?

Merci d'avance...

[koala01]

Salut,

Pourquoi ne pas partir sur l'idée d'une sorte de "machine à état"

tu créerais une énumération du genre de

Code :

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

1
2
3
4
5
6
7
enum eaction
{
    show,
    move,
    modify /* ,
    ...*/
};

et tu l'utiliserais à l'initialisation de tes modèles sous la forme de

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
 
class Model
{
    public:
        Model(eaction a):a(a){}
        ~Model(){}
        void MakeAction()
        {
             switch(a)
             {
                 case show:
                     Show();
                     break; 
                 case move:
                     Move();
                     break;
                 case modify:
                     Modify();
                     break;
                 /*...*/
            }
        }
        void Move();
        void Modify();
        void Show();
        /*...*/
    private:
        eaction a;
        /*...*/
};

et, au final, tu lance simplement la commande

Code :

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

1
2
item->MakeAction(); /*ou, si c'est un objet et non un pointeur */
item.MakeAction();

Nota:
Tu pourrais aussi envisager un arbre d'héritage pour éviter le switch, mais tu perdrais d'un coté (virtualité de la méthode) ce que tu gagnerais avec le switch

[loufoque]

un switch déjà serait mieux.

[zenux]

Merci pour vos réponse...

Mais comme je l'ait dit, avec l'idée du "if" ou "switch" si les méthodes de model prennent différents paramètres, ça devient vite du n'importe quoi.

Si l'ont reprend mon code, n'y a t-il pas moyen de faire quelque chose du genre :

Code :

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

1
2
3
4
5
6
7
int main()
{
  A a;
  a.make(model::show()); //appel la méthode "show" de model.
  a.make(model::move()); //idem mais avec "move"
  a.make(model::setSpeed(65.0)); //idem avec des paramètres
}

[Mat007]

Salut,

Il y a par exemple Boost.Bind qui permettrait de passer une fonction reflexe construite en fixant autant de parametres que tu veux.
C'est par forcement evident a manipuler les premieres fois mais ca vaut le coup de s'y plonger.

MAT.

[JolyLoic]

Idem, boost::bind pour le passage ou pas d'arguments, et un boost::function comme paramètre passé à la fonction make.

[zenux]

Merci pour vos réponses.
Je vais regarder à ça pour voir ce que je peux faire