c++ problème avec argument (ou inclusion?)

Question

Bonjour à tous, 

Je rencontre actuellement un problème dans ma programmation : en essayant de compiler j'obtiens le message suivant "'Visitor' has not been declared". 

Pourtant là je fais un .hpp, donc dans les prototypes on met juste les arguments que les fonctions vont utiliser normalement... et j'ai bien inclu "Visitor.hpp". 

Voici la déclaration de ma classe Visitable.hpp: 

     
    #ifndef VISITABLE_HPP 
    #define VISITABLE_HPP 
    #include "Visitor.hpp" 
      
    class Visitable 
    { 
    public: 
    virtual void accept(Visitor* v) const = 0; 
    }; 
      
    #endif 


Je vous remercie par avance pour votre soutien.

fiddy · Answer

Pourrait-on savoir ce que tu as mis pour déclarer et définir Visitor dans Visitor.hpp et Visitor.cpp ?

delp95 · Answer

Mon Visitor.hpp est comme ceci :


#ifndef VISITOR_HPP
#define VISITOR_HPP
#include "MyType.hpp"
#include "Op.hpp"

class Visitor
{
  public:
      virtual void visitNode(MyType* m) const = 0;
      virtual void visitOp(Op* o) const = 0;
};

#endif


Je n'ai pas de Visitor.cpp car c'est une classe virtuelle.

g723 · Answer

poste :
- le log de compilation
- "MyType.hpp" et "Op.hpp"

delp95 · Answer

Voici Op.hpp (MyType.hpp est similaire) :

#ifndef OP_H
#define OP_H
#include <string>
#include <iostream>
#include "Exp.hpp"
using namespace std;
class Op : public Exp //Op dérive de Exp
{
    public :
       virtual void display();
       virtual int accept(Visitor* v) const;
};
#endif

MyType et Op dérive de Exp que voici :

#ifndef EXPRESSION_H
#define EXPRESSION_H
#include <string>
#include <iostream>
#include "Visitable.hpp"
//#include "Visitor.hpp"
using namespace std;
class Expression : public Visitable
{
    public :
       virtual void display()=0;
       virtual int accept(AbstractVisitor* v) const = 0;
};
#endif

Qui hérite elle même de Visitable :

#ifndef VISITABLE_HPP
#define VISITABLE_HPP
#include "Visitor.hpp"
class Visitable
{
  public:
    virtual int accept(Visitor* v) const = 0;
};
#endif

g723 · Answer

Compilons exp.cpp ( on suppose qu' il inclus exp.h ) ; inclusions successives :         

 
                   #define code         
exp.h              +        -         
  visitable.h      +        -         
    visitor.h      +        -         
      mytype.h     +        -         
        exp        -        -       



Et là il y aura une erreur car le code de MyType.h va être lu et il va utiliser un visitor*, non déclaré.         

Pour éviter ça ( certains diront qu'il y a plutôt une erreur de conception blah blah blah ) utilise un fichier common.h qui déclare toutes les classes:         

#ifndef COMMON_HPP         
#define COMMON_HPP         

class Visitable;         
class Visitor;         
class Exp;         
...etc...         

#endif

delp95 · Answer

Ah oui, c'est une solution !

Mais est ce que c'est une façon "propre" de faire ?

g723 · Answer

> certains diront qu'il y a plutôt une erreur de conception

D' autre part si tu veux inliner dans tes headers, là tu seras vraiment bloqué(e)

C++ problème avec argument (ou inclusion?)

7 réponses

Discussions similaires

Newsletters