Discussion :

[Invité]

Bonjour,

Je crois avoir ce qui peut être un problème d'héritage. Je fais un programme de traitement d'images en ligne de commande. J'ai entre autre une classe Inversion qui hérite de la classe Command. J'ai prototypé une méthode run() en virtual dans Command pour que quand on la lance ce soit plutot la méthode redéfinie dans Inversion qui soit appelé. Cependant, à l'éxécution c'est la méthode run() de la classe mère qui est appelé à la place. D'où vient l'erreur ?

Voici mes deux .h :

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
 
#ifndef COMMAND_H
 
#define COMMAND_H
 
 
 
#include <string>
 
#include <vector>
 
#include <iostream>
 
 
 
using namespace std;
 
 
 
class Command
 
{
 
    public:
 
        Command();
 
        virtual ~Command();
 
        virtual void run();
 
    protected:
 
    private:
 
};
 
 
 
#endif // COMMAND_H

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
 
#ifndef INVERSION_H
 
#define INVERSION_H
 
 
#include "Command.h"
 
using namespace std;
 
 
 
class Inversion : public Command
 
{
 
    public:
 
        Inversion();
 
        virtual ~Inversion();
 
        void run();
 
    protected:
 
    private:
 
};
 
 
 
#endif // INVERSION_H

et leur .cpp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#include "Command.h"
 
 
 
Command::Command(){}
 
 
 
Command::~Command(){}
 
void Command::run(){
cout<< "classe mere" << endl;
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#include "Inversion.h"
 
Inversion::Inversion(){
	Command();
}
 
Inversion::~Inversion(){}
 
void Inversion::run(){
	cout<< "classe fille" << endl;
}

Merci d'avance !

EDIT: J'ai essayé de garder que le strict nécéssaire dans le code.

[koala01]

Salut,

Tu as tellement bien fait le ménage qu'il nous manque le principal, dit donc

Tu pourrais nous montrer le code qui fait appel à la fonction run, et, surtout, l'initialisation de la variable au départ de laquelle tu appelles cette fonction

Mais ceci dit, je te remercie quand meme de penser à nos pauvres yeux fatigués en essayant de nous donner un code minimum compilable, et je salue l'effort

[Invité]

Merci pour la réponse rapide, je suis un peu pressé par le temps justement!

Pour ne pas copier coller tout le pavé de code, l'appel de run se fait de cette manière là :

Code :

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

1
2
3
void Facade::runArg(Command pArg){
	pArg.run();
}

avec le paramètre pArg qui est une des classes filles de la classe Command, dont Inversion par exemple.

[JolyLoic]

Tu passes l'objet par copie dans ta fonction, tu es donc bien dans le cas du slicing. 2 conseils :
- Passe par référence à ta fonction
- Pour éviter ce genre de gags par la suite, dès que tu as une classe qui sert de classe de base, rend la non copiable. http://cpp.developpez.com/faq/cpp/?p...iables_syntaxe (lis aussi les questions autour de celle-ci).

[Invité]

Merci énormément, cela marche parfaitement après passage par référence! Ah les pointeurs me hanteront toujours!

[JolyLoic]

+1 à Koala.

De plus, le code suivant me semble très suspect :

[code]
#include "Inversion.h"

Inversion::Inversion(){
Command();
}

Que penses-tu qu'il fasse ?

Sinon, pour ton problème, généralement :
- Soit il y a un souci de compréhension des règles sur les fonctions virtuelles dans les constructeurs/destructeurs
- Soit à un moment ou à un autre, on manipule l'objet directement et non plus par pointeur, et on fait une copie qui provoque du slicing.