Discussion :

[argonath]

Bonjour, j'ai une class et ses héritières qui à la compilation me donnent une erreur "expected ';' before '(' token" dont je ne trouve pas la raison

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
 
#ifndef STATE_H_
#define STATE_H_
 
#include "wx/wx.h"
#include "ZoneDeDessin.h"
#include "DessinRectangle.h"
#include "DessinEllipse.h"
 
class State
{
    public :
        State();
        virtual ~State();
        virtual Dessin CreerDessin(ZoneDeDessin * zoneDeDessin)=0;
};
 
#endif /* STATE_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
 
#ifndef CONCRETESTATESELECTION_H_
#define CONCRETESTATESELECTION_H_
 
#include "State.h"
 
class ConcreteStateSelection : public State
{
    public :
        ConcreteStateSelection();
        virtual Dessin CreerDessin(ZoneDeDessin* zoneDeDessin);
};
 
#endif /*CONCRETESTATESELECTION_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
 
#ifndef CONCRETESTATERECTANGLE_H_
#define CONCRETESTATERECTANGLE_H_
 
#include "State.h"
 
class ConcreteStateRectangle : public State
{
    public :
        ConcreteStateRectangle();
        virtual Dessin CreerDessin(ZoneDeDessin* zoneDeDessin);
};
 
#endif /*CONCRETESTATERECTANGLE_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
 
#ifndef CONCRETESTATEELLIPSE_H_
#define CONCRETESTATEELLIPSE_H_
 
#include "State.h"
 
class ConcreteStateEllipse : public State
{
    public :
        ConcreteStateEllipse();
        virtual Dessin CreerDessin(ZoneDeDessin* zoneDeDessin);
};
 
#endif /*CONCRETESTATEELLIPSE_H_*/

J'ai l'erreur aux trois classes ainsi qu'un "`CreerDessin' declared as a `virtual' field " qui me laisse également un peu perplexe, vu que je ne vois pas pourquoi elles ne pourraient pas l'être

[Spout]

Salut,
Au début de la compilation, aucun des éléments présents dans ton fichier state.h ne sont définis.
Il va définir STATE_H_ et créer la classe State, OK.
Puis il arrive sur CONCRETESTATESELECTION_H_ qui n'est pas défini.
Donc il fait sont boulot, il le définit et arrive sur une inclusion de lui même?
Je n'ai pas testé ton code, mais déjà je pense que tu peux enlever tous les

Code :

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

#include state.h

et voir ce que ça donne.
Bon courage

[argonath]

je ne comprends pas ce que tu veux dire, ce sont trois fichiers différents (State.h,ConcreteStateSelection.h, ConcreteStateRectangle.h et ConcreteStateEllipse.h ) je les ait seulement mis ensemble ici par commodité.

Et je suis obligé d'inclure State.h dans les classes qui en héritent

non vu le message d'erreur j'ai simplement l'impression qu'il ne voit pas la dernière ligne de chaque comme un prototype de fonction

[argonath]

non non elle est bien virtuelle pure et le compilo me met une erreur dans chacun des trois fichiers, toujours à la dernière ligne oui

avec à chaque fois deux erreurs, la première complètement débile
"`CreerDessin' declared as a `virtual' field" ( oui j'ai écrit virtual merci le compilo d'enfoncer des portes ouvertes)
et ensuite
"expected ';' before '(' token"

et en remplacant "Dessin" par "void" pour voir, il me met aussi "`CreerDessin' declared as a `void' field"

d'où l'idée que peut être il ne voit pas ça comme une fonction, ce qui expliquerait aussi le rejet des parenthèses

[Spout]

OK, au temps pour moi pour les fichiers, c'était pas précisé.
Quel est le type de retour de ta fonction CreerDessin?

[argonath]

oui pas bien grave c'est vrai que j'aurais dû dire que c'était des fichiers différents ^^

le retour de la fonction est de type Dessin

il n'y a pas de #include "Dessin.h" parceque c'est aussi un héritage et qu'il est sous entendu dans les #include "DessinRectangle.h" et "DessinEllipse.h"

[gl]

As tu vérifié s'il ne manquerait pas un ; dans un des fichiers inclus (ZoneDeDessin.h, DessinRectangle.h ou DessinEllipse.h) ?

[argonath]

le cas échéant m'aurait mis l'erreur dans le fichier concerné vu qu'il a du les lire pour les include non ?

edit : j'ai vérifié par acquis de conscience, tout à l'air correct

edit 2 : il y'a bien 4 fichiers et pas 3

[gl]

le cas échéant m'aurait mis l'erreur dans le fichier concerné vu qu'il a du les lire pour les include non ?

Pas nécessairement non.

Sinon peux-tu fournir le code minimal présentant le problème

[3DArchi]

Salut,
Souvent, cette erreur sort quand le compilateur ne voit pas la déclaration pour Dessin et/ou ZoneDeDessin.
Ceci dit, rien qu'en prenant State.h, je me dit qu'il y a un problème :

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
#ifndef STATE_H_
#define STATE_H_
 
#include "wx/wx.h" //(1) 
#include "ZoneDeDessin.h"  //(2) 
#include "DessinRectangle.h"  // (3)
#include "DessinEllipse.h"  // (4)
 
class State
{
    public :
        State();
        virtual ~State();
        virtual Dessin CreerDessin(ZoneDeDessin * zoneDeDessin)=0; //(5)
};
 
#endif /* STATE_H_ */

(1) : State ne présent aucune classe wxWidget. Pourquoi cet include ?
(2) et (5) : ZoneDeDessin n'est utilisé que par pointeur. Pourquoi ne pas utiliser une déclaration anticipée simplifiant probablement les dépendances entre en-têtes.
(3) et (4) : Je ne vois rien qui mentionne Rectangle ou Ellipse. Donc cela me parait suspect.

Ceci dit, State dépend fortement de Dessin. Dans ce cas, je préfère inclure 'Dessin.h' plutôt que de supposer que celui-ci est inclue par X qui inclue Y qui inclue Dessin.h.
Voilà à quoi j'aurais aimé qu'un tel .h ressemble :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#ifndef STATE_H_
#define STATE_H_
 
#include "Dessin.h"
class ZoneDeDessin;
class State
{
    public :
        State();
        virtual ~State();
        virtual Dessin CreerDessin(ZoneDeDessin * zoneDeDessin)=0;
};
 
#endif /* STATE_H_ */

Et même, j'aurais peut être créé un ZoneDeDessinFwd.h :

Code :

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

1
2
3
4
#ifndef ZoneDeDessinFwd_H
#define ZoneDeDessinFwd_H
class ZoneDeDessinFwd;
#endif /* ZoneDeDessinFwd_H */

et State.h devient :

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
#ifndef STATE_H_
#define STATE_H_
 
#include "Dessin.h"
#include "ZoneDeDessinFwd.h"
 
class State
{
    public :
        State();
        virtual ~State();
        virtual Dessin CreerDessin(ZoneDeDessin * zoneDeDessin)=0;
};
 
#endif /* STATE_H_ */

[argonath]

Le problème venait effectivement des inclusions en cascade c'est résolu, mais effectivement tes propositions amélioreront bien la logique de la chose