Discussion :

[kipgon]

Bonjour,

je ne comprend pas bien ce qui cloche dans le code suivant (simplifié par souci de clarté) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
// A.cpp
 
#include "A.h"
#include "B.h"
 
void A::fonctionMembreDeA(C& c)
{
      B b;                               // Erreur : expected ';' before b;
 
      c.b[2].fonctionMembreDeB();        // Pas de problème
}

Code :

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

1
2
3
4
5
6
7
8
9
// A.h
 
#include "C.h";                                  // Compile avec include mais pas avec déclaration anticipée
 
class A
{
public:
      void fonctionMembreDeA(C& c);
};

Code :

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

1
2
3
4
5
6
7
// B.h
 
class B
{
public:
      void fonctionMembreDeB();
};

Code :

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

1
2
3
4
5
6
7
8
9
10
11
// C.h
 
#include <vector>
 
class B;
 
class C
{
public:
      vector<B> b;
};

Dans ma fonction principale A, je peut donc utiliser une fonction membre de la classe B mais pas créer d'instance de cette classe !

Je précise au cas ou ce serait utile que les fichiers B.h et C.h viennent toute deux d'un autre projet qui tourne très bien.

Merci d'avance pour vos suggestions.

[koala01]

Salut,

Est-ce juste une faute lors du copier coller, ou bien manque-t-il réellement le ";" après l'accolade fermante de la définition de tes classes

[kipgon]

C'est une erreur, il a bien un ';' après l'accolade fermante de chacune de mes classes.

J'ai également oublié de préciser que le constructeur de ma classe B est bien public.

Je modifie ça tout de suite.

[kipgon]

Une autre précision :

si je met (ça n'a aucun intérêt c'est juste pour le test) un ';' derrière mon type B et avant le nom de l'instance b de cette manière :

Code :

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

B; b;

Le compilateur me renvoi :
- Statement has no effect (j'imagine que c'est pour B; )
- 'b' was not declared in the scope

Si cela peut vous aiguiller...

[koala01]

Une autre précision :

si je met (ça n'a aucun intérêt c'est juste pour le test) un ';' derrière mon type B et avant le nom de l'instance b de cette manière :

Code :

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

B; b;

Le compilateur me renvoi :
- Statement has no effect (j'imagine que c'est pour B; )
- 'b' was not declared in the scope

Si cela peut vous aiguiller...

Ben non, mais ca, c'est normal!!!

le ";" est ce que l'on désigne comme le "end of statement" (fin de déclaration / d'instruction ) du C++. Avec un code comme B;, tu lui parle du type B (qu'il est sensé connaitre), mais tu ne lui dit pas quoi en faire... Et c'est pour cela qu'il te dit "cette instruction n'a aucun effet".

Puis, comme tu as mis fin à l'instruction, tu te retrouve avec une autre instruction : b;, mais il n'a jamais rencontré b jusqu'à ce jour, (enfin, dans le contexte de la paire d'accolades qui l'entourent) et il ne le connait donc absolument pas. C'est la raison pour laquelle il te dit " 'b' n'était pas déclaré dans cette portée". La portée étant l'ensemble du code qui sépare l'accolade ouvrante de l'accolade fermante, à ceci près qu'il lit le code comme tu lis n'importe quelle page : du début à la fin, dans l'ordre exacte dans lequel les instructions sont écrites, si bien que, en l'occurrence, la portée se limite à l'espace qui existe entre l'accolade ouvrante et... le terme b.

Ceci étant dit, pourrais tu nous dire l'erreur que le compilateur te donne, quand "il refuse que tu crées une instance de B, alors qu'il accepte que tu en appelle la méthode" Cela nous aidera surement à t'aider, car, bien que je me doute que tu aies tout simplifié au maximum, (ou peut être à cause de cela, justement), le code que tu présentes ne devrait pas poser de problème

PS: Ah, et, au passage : le terme "méthode" en C++ est normalement limité aux fonction membres virtuelles, pour, justement, les distinguer de celles qui ne le sont pas... si bien qu'on utilise généralement les termes

• données membres pour désigner les données qui font partie de la classe
• fonctions membres pour désigner les fonctions de la classe qui ne sont pas virtuelles
• méthodes pour désigner les fonctions virtuelles de la classe.

Les autres langages (dont UML en premier) ne font pas ce genre de distinction, généralement parce qu'ils estiment qu'une fonction membre sera toujours virtuelle. Or, c'est vrai pour de très nombreux langages (généralement ceux qui utilisent une sorte ou une autre de "garbage collector"), mais pas en C++.

D'ailleurs, en C++ les fonctions membres virtuelles sont considérées comme l'exception et non comme la règle : c'est la raison pour laquelle il faut explicitement l'indiquer quand une fonction doit l'être

[kipgon]

Pour la ligne 8 de mon A.cpp :

Code :

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

B b;

il me renvoit :

- expected ';' before b;
- statement is a reference, not call, to function 'Polygon'
- statement has no effect

Alors que la ligne 10 :

Code :

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

c.b[2].fonctionMembreDeB();

ne pose aucun problème et le code tourne très bien avec.

[Matt_Houston]

Ton erreur n'est pas reproductible avec le code que tu nous as fourni, on ne peut donc absolument rien en déduire. C'est bien de vouloir présenter un exemple minimal, encore faut-il qu'il reproduise exactement le comportement rencontré car aucun de nous ne dispose d'une boule de cristal.

[bacelar]

Je sens la MACRO bien moisie.
Utilisez les options de compilation qui permettent d'avoir le résultat du pré-processing et vérifiez que des MACRO foutent pas la grouille.

[kipgon]

Je comprend bien que ce n'est pas évident pour vous de voir ce qui peut clocher dans un code simplifié comme ça. Je me disais que peut être l'erreur vous parlerais car ce serait difficile pour moi de mettre l'ensemble du code ...

Concernant la compilation, quelle option permet d'avoir le résultat du pré-processing ?

La seule macro dont j'ai la connaissance est le Q_OBJECT dans ma classe A qui hérite de QGLWidget.

[bacelar]

Concernant la compilation, quelle option permet d'avoir le résultat du pré-processing ?

C'est fonction du compilateur utilisé.

[kipgon]

J'utilise MinGW

[bacelar]

C'est le moment d'aller voir le documentation.
Moi, je connais pas.

[Bousk]

Le seul moyen d'avoir un undefined et demander un ; avant le b dans B b; est que B ne soit pas défini.
Btw ton code minimal est minimalement faux et ne montre rien
- visibilité private par défaut dans les classes, tu n'es pas sensé pouvoir appeler fonctionMembreDeB non plus
- l'include de C dans A.h est inutile et une forward declaration suffit

Si j'emmène mon lapin chez le véto et lui demande pourquoi mon chat est malade, il aura du mal à me répondre...

[Pyramidev]

J'utilise MinGW

Dans ce cas, pour n'exécuter que le préprocesseur :

Code :

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

g++ tonFichier.cpp -E -P -o fichierDeSortie.txt

Remarques :

• C'est l'option -E qui indique de n'exécuter que le préprocesseur.
• L'option -P en complément est facultative, mais elle permet d'avoir un résultat moins lourd. Je te laisse tester avec et sans.
• Selon l'emplacement de tes ".h", il se peut que tu aies besoin d'ajouter des options -IadresseDeDossier pour que le préprocesseur réussisse à exécuter les directives #include.
• Si tu veux définir des macros en amont de la précompilation, par exemple NDEBUG en Release, il faut ajouter des options -DNOM_DE_LA_MACRO, par exemple -DNDEBUG.

[kipgon]

Merci pour vos réponses concernant le préprocesseur, je vais étudier ça et je vous dirai ce que j'en ressort .

Concernant le code simplifié, j'ai effectivement oublié de signaler que les fonctions membres et les attributs sont public. J'apporte la modification tout de suite !

Concernant l'include plutôt que la déclaration anticipée, il se trouve que le code ne compile plus sans cela. Il doit déjà y avoir un truc de ce coté la !

Je vais essayer de dégrader mon code tout en conservant mon erreur de compilation et je vous posterai ça.