Discussion :

[YannOss]

Bonjour et merci pour votre intêret et vos réponses.

Alors j'expose le problème:
J'ai un fichier comprenant des initialisations de variables d'entrée/sortie nommé "InOut.hh"
Pour certaines initialisations, je dois lire un fichier XML en utilisant un parseur (xerces en l'occurence, obligé par mes supèrieurs). J'ai donc créer un fichier "LectureXML.cpp" avec ma fonction de lecture du fichier XML (LECTURE_FICHIER_DONNEES) que j'ai inclut dans mon fichier "InOut.hh".

Mais quand dans mon .hh j'apelle ma fonction LECTURE_FICHIER_DONNEES. Le compilo me renvoie un "LECTURE_FICHIER_DONNEES was not declared in this scope"

J'aurais aimé savoir comment le déclarer...
En résumé mon Fichier InOut.hh ressemble à ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#include <math.h>
#include "LectureXML.cpp"
 
class InOut {
public:
//constructeur
//Destructeur
//Méthode d'initialisation
void initialize (/*signature*/)
{
 alpha= LECTURE_FICHIER_DONNEES();
};
}

[imperio]

3 choses pour commencer :

- Pas de protection contre la multi-inclusion (pragma once ou ifndef/define).
- Une declaration de classe se termine par un point-virgule. Exemple :

Code :

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

1
2
3
class A
{
};

- Une fonction se termine par une accolade et n'a pas besoin d'etre suivi par un point-virgule. Exemple :

Code :

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

1
2
3
int test(void)
{
}

Ou dans ton cas :

Code :

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

1
2
3
4
5
6
7
 
class A
{
 int test(void)
 {
 }
};

Par contre une declaration de fonction se termine bien par un point virgule. Exemple :

Code :

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

1
2
 
int test(void);

[YannOss]

D'accord en fait il aurait fallu que je déclare ma fonction avant la classe:

Code :

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

1
2
3
4
5
6
7
8
9
10
int LECTURE_FICHIER_DONNEES (/*signature1*/);
classe InOut
{
  //Constructeur
  //Destructeur
  void Initialize(/*signature2*/)
  {
      alpha= LECTURE_FICHIER_DONNEES();
  }
};

Et ceux même si ma fonction est dévellopé dans un fichier inclut

[white_tentacle]

Il y a un truc que je vois qui me fait très peur :

Code :

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

#include "LectureXML.cpp"

On n’inclut jamais un .cpp, et surtout pas dans un .h .

J’espère que c’est une typo .

[YannOss]

Ben, en fait pour le coup c'est pas une Typo. Je travaille sur un logiciel assez énorme. On me demande de créer une méthode pour initialiser des données à partir d'un fichier XML, et comme je trouve que c'est plutôt sale de coder dans un .hh, je le fais dans un .cpp, même si l'inclusion d'un cpp dans un .hh est sale aussi...
Mais tu me dit qu'on ne dois *JAMAIS* faire ça, pourtant ça ne me sors ni warning ni erreur, est-ce juste qu'on ne dois pas le faire parce que c'est pas propre ou parce que le compilo ne le prendrai pas en compte?

[Bousk]

Bonjour,

C'est tout simplement la règle.
Les .cpp et .c sont compilés, les autres fichiers non.
Tu utilises toutes les extensions que tu veux, mais .cpp et .c sont réservées aux fichiers compilés.
Commencer à faire n'importe quoi n'est pas une bonne idée. Sous prétexte que ca semble marcher encore moins.
Btw, mettre du code dans un header n'a rien de grave et ca arrive. C'est même obligatoire pour les templates.

[YannOss]

Bon, ça c'est changé maintenant c'est LectureXML.hh. Donc j'include mon LectureXML.hh dans mon InOut.hh. Merci de ton conseil

[white_tentacle]

Pour compléter un peu ce qu’a dit Bousk et mon propos précédent.

- un fichier .h doit à minima respecter les contraintes suivantes :

• on peut l’inclure plusieurs fois dans le même fichier
• on peut l’inclure dans différents fichier du même projet sans que cela aboutisse à une erreur de link
• il doit pouvoir être inclus indépendamment de tout autre (ceci est plus une bonne pratique qu’une obligation stricte, mais je t’encourage fortement à la respecter, ça simplifie la vie).

C’est quasiment impossible de respecter ces contraintes avec des fichiers .cpp. Du coup, si tu inclus un .cpp dans ton header, tu casses ces propriétés.

Pour les templates, certains préfèrent séparer interface/implémentation dans deux fichiers distincts, auquel cas, en général, on inclus le fichier implémentation (.tpp) à la fin du fichier interface (.h).

Je t’invite à lire la FAQ là dessus pour plus de détails :

http://cpp.developpez.com/faq/cpp/?page=projet

[koala01]

Salut,

Et pour parfaire le tout: seule les fonctions inline doivent se retrouver dans les fichiers d'en-tête