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

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

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.

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

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

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

Ah oui, c'est une solution !

Mais est ce que c'est une façon "propre" de faire ?

> 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)