Discussion :

[fenelon]

Salut !

Je debute dans le C++ en m' embrouillant avec les includes: j' ai un fichier main.h contenant toutes mes includes de base:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#ifndef MAIN_H
#define MAIN_H
 
#include <iostream>
#include <string>
#include <vector>
#include <map>
#include "SCREEN.h"
 
using namespace std;
 
#endif

Mon fichier SCREEN.h est une simple declaration de base:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#ifndef SCREEN_H
#define SCREEN_H
 
#include "main.h"
 
class SCREEN
{
public:
    void tes(void);
};
#endif //SCREEN_H

Et mon fichier SCREEN.cpp contient les toutes simples declaration de cette classe:

Code :

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

1
2
3
4
5
6
7
 
#include "SCREEN.h"
 
void SCREEN::tes(void)
{
    cout << "tes" << endl;
&#125;

Mon fichier main.cpp contient simplement l' include de main.h et la fonction main:

Code :

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

1
2
3
4
5
6
7
8
 
#include "main.h"
 
int main()
{
    SCREEN test;
    return 0;
}

Voila tout. J' essaye de compiler le tout avec Borland C++ Builder X (avec GCC comme compilateur) et j' obtiens comme erreur que tes est defini plusieurs fois:

Code :

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

1
2
 
"SCREEN.cpp" : src/SCREEN.cpp définitions multiples de « SCREEN::tes()    » en ligne 4

Pourtant, si je compile exactement le même code en ligne de commande avec G++, je n' obtiens aucune erreur !

Merci

[reggae]

C'est peut-être une erreur de ma part mais... où est ton constructeur pour

Code :

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

1
2
 
SCREEN test;

?
Ton code marche sans problème quand tu l'exécute(la compilation qui réussit n'est pas forcément synonyme de bon fonctionnement du prog)?

[Zenol]

C'est peut-être une erreur de ma part mais... où est ton constructeur pour

Code :

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

1
2
 
SCREEN test;

Le constructeur n'est pas obligatoire, tout comme le destructeur.

Sinon c'est vraiment bizare, je ne voit pas la moindre chose qui puisse poser une éreur...

Edit : J'ai compiler le code sous dev c++ en rajoutent dans la fonction main test.tes(); et le programme fonctione a la perfection!!!

[reggae]

Alors là va faloir que je révise mes cours...
Si il n'y a pas de constructeur(je savais que le destructeur n'est pas obligatoire) l'objet est créé sans aucune autre action je suppose?
A priori tu as raison rien ne semple gêner la compil'...

[Laurent Gomila]

Si il n'y a pas de constructeur(je savais que le destructeur n'est pas obligatoire) l'objet est créé sans aucune autre action je suppose?

Pas exactement. Le compilo génère un constructeur par défaut (s'il n'y en a aucun autre), qui initialisera à son tour toutes les données membres via leur constructeur par défaut. Pas toujours trivial, donc.

Sinon pour le problème initial, il me semble qu'il y a des inclusions croisées entre main.h et SCREEN.h. Vire l'inclusion de main.h dans SCREEN.h, c'est inutile et ça débloquera sans doute le schmilblick.

[Zenol]

Oui mais sa ne marcheras plus, sa ne compileras plus car il n'y auras plus le #include <iostream> contenu dans .h, ni les autres inclusions.

Donc, a ta place, je mêtrais a la place de #include "main.h" les inclusons des fichier qui sont nésésaire seulement pour SCREEN.cpp
Ensuite peut importe si main.h et SCREEN.h inclue les mêmes fichier, le plus important est qu'ils ne s'inclu pas entre eux,(comme la dit loulou24)

[David Fleury]

Salut,
sinon pas trop de rapport mais évite les "using namespace std;" dans les .h
(ici main.h je crois).
préfixe tout par std:: dans les .h (ce qui est nécessaire évidement)
et fait le using dans le .cpp uniquement.
Ca évite d'ouvrir le namespace std pour tous ceux qui incluront ton .h
(même si ici c'est pas évident qu'il y en ait)